[gwt-contrib] Change in gwt[master]: Introduces a common SerializableThrowable and refactors Stac...

2013-02-27 Thread Goktug Gokdogan
Goktug Gokdogan has uploaded a new change for review. https://gwt-review.googlesource.com/2040 Change subject: Introduces a common SerializableThrowable and refactors StackTraceDeobfuscator as public API. .. Introduces a

[gwt-contrib] Re: Exposed ability for implmentations of Animation.java to check if the animation is currently running. (issue1891804)

2013-02-27 Thread goktug
http://gwt-code-reviews.appspot.com/1891804/diff/7001/user/src/com/google/gwt/animation/client/Animation.java File user/src/com/google/gwt/animation/client/Animation.java (right): http://gwt-code-reviews.appspot.com/1891804/diff/7001/user/src/com/google/gwt/animation/client/Animation.java#newcod

[gwt-contrib] Change in gwt[master]: Fix SpeedTracer Log when compilation is spread across differ...

2013-02-27 Thread Roberto Lublinerman
Roberto Lublinerman has abandoned this change. Change subject: Fix SpeedTracer Log when compilation is spread across different processes. .. Abandoned Submitted, Thanks! -- To view, visit https://gwt-review.googlesource.co

[gwt-contrib] Change in gwt[master]: Fix SpeedTracer Log when compilation is spread across differ...

2013-02-27 Thread Brian Slesinsky
Brian Slesinsky has posted comments on this change. Change subject: Fix SpeedTracer Log when compilation is spread across different processes. .. Patch Set 1: Code-Review+1 Sure, makes sense. -- To view, visit https://gwt-

[gwt-contrib] Re: Exposed ability for implmentations of Animation.java to check if the animation is currently running. (issue1891804)

2013-02-27 Thread mdempsky
On 2013/02/28 01:45:15, jtam wrote: Switched to Public. Not sure what you would like me to do in terms of unit tests. This is just exposing an internal which should be already tested. I think just adding appropriate asserts for isRunning()'s return value to AnimationTest would be sufficient.

[gwt-contrib] Re: Exposed ability for implmentations of Animation.java to check if the animation is currently running. (issue1891804)

2013-02-27 Thread mdempsky
http://gwt-code-reviews.appspot.com/1891804/diff/1/user/src/com/google/gwt/animation/client/Animation.java File user/src/com/google/gwt/animation/client/Animation.java (right): http://gwt-code-reviews.appspot.com/1891804/diff/1/user/src/com/google/gwt/animation/client/Animation.java#newcode203 u

[gwt-contrib] Re: Exposed ability for implmentations of Animation.java to check if the animation is currently running. (issue1891804)

2013-02-27 Thread skybrian
Why protected? Also, shouldn't the unit tests be updated? http://gwt-code-reviews.appspot.com/1891804/ -- -- http://groups.google.com/group/Google-Web-Toolkit-Contributors --- You received this message because you are subscribed to the Google Groups "Google Web Toolkit Contributors" group. T

[gwt-contrib] Change in gwt[master]: Fix SpeedTracer Log when compilation is spread across differ...

2013-02-27 Thread Matthew Dempsky
Matthew Dempsky has posted comments on this change. Change subject: Fix SpeedTracer Log when compilation is spread across different processes. .. Patch Set 1: Code-Review+1 Seems to make sense to me. -- To view, visit http

[gwt-contrib] Change in gwt[master]: Fix SpeedTracer Log when compilation is spread across differ...

2013-02-27 Thread Roberto Lublinerman
Roberto Lublinerman has uploaded a new change for review. https://gwt-review.googlesource.com/2031 Change subject: Fix SpeedTracer Log when compilation is spread across different processes. .. Fix SpeedTracer Log when com

[gwt-contrib] Change in gwt[master]: Remove unnecessary wrapping of IOException in DiskCacheToken...

2013-02-27 Thread Matthew Dempsky
Matthew Dempsky has posted comments on this change. Change subject: Remove unnecessary wrapping of IOException in DiskCacheToken. This just increases stack trace verbosity, since the callers either throw IOException already or could easily do so. .

[gwt-contrib] Change in gwt[master]: Removes deprecated methods from JUnitShell.

2013-02-27 Thread Matthew Dempsky
Matthew Dempsky has posted comments on this change. Change subject: Removes deprecated methods from JUnitShell. .. Patch Set 1: Code-Review+1 -- To view, visit https://gwt-review.googlesource.com/2030 To unsubscribe, visit htt

[gwt-contrib] Change in gwt[master]: Removes deprecated methods from JUnitShell.

2013-02-27 Thread Goktug Gokdogan
Goktug Gokdogan has uploaded a new change for review. https://gwt-review.googlesource.com/2030 Change subject: Removes deprecated methods from JUnitShell. .. Removes deprecated methods from JUnitShell. Change-Id: I0e3fc2b07

[gwt-contrib] Change in gwt[master]: Remove unnecessary wrapping of IOException in DiskCacheToken...

2013-02-27 Thread Matthew Dempsky
Matthew Dempsky has posted comments on this change. Change subject: Remove unnecessary wrapping of IOException in DiskCacheToken. This just increases stack trace verbosity, since the callers either throw IOException already or could easily do so. .

[gwt-contrib] Change in gwt[master]: Remove unnecessary wrapping of IOException in DiskCacheToken...

2013-02-27 Thread Brian Slesinsky
Brian Slesinsky has uploaded a new change for review. https://gwt-review.googlesource.com/2020 Change subject: Remove unnecessary wrapping of IOException in DiskCacheToken. This just increases stack trace verbosity, since the callers either throw IOException already or could easily do so.