[gwt-contrib] Handle composite constraints. (issue1231801)

2010-12-16 Thread nchalko
Reviewers: rchandia, Description: Handle composite constraints. [JSR 303 TCK Result] 36 of 258 (13.95%) Pass with 15 Failures and 16 Errors. Please review this at http://gwt-code-reviews.appspot.com/1231801/show Affected files: M samples/validation/src/com/google/gwt/sample/validation/shar

[gwt-contrib] [google-web-toolkit] r9460 committed - Link to CL

2010-12-16 Thread codesite-noreply
Revision: 9460 Author: ncha...@google.com Date: Thu Dec 16 21:53:00 2010 Log: Link to CL http://code.google.com/p/google-web-toolkit/source/detail?r=9460 Modified: /wiki/BeanValidation.wiki === --- /wiki/BeanValidation.wiki Thu Dec 16 09:47:49 2010 +++ /wik

[gwt-contrib] Basic MessageIntrpolation. (issue1230801)

2010-12-16 Thread nchalko
Reviewers: rchandia, Description: Basic MessageIntrpolation. Refactored the bootstrap to hook via ValidatorFactory [JSR 303 TCK Result] 44 of 258 (17.05%) Pass with 12 Failures and 12 Errors. Please review this at http://gwt-code-reviews.appspot.com/1230801/show Affected files: M samples/comm

[gwt-contrib] Re: New Google Groups apparent with GWT ?

2010-12-16 Thread Chris
I second this request for info :) We are using TinyMCE right now and worked hard to get it to integrate; but we don't really like it and there are oddities that we still deal with sometimes. If someone has sample integration code that could be shared, we would appreciate being able to see it to

[gwt-contrib] Re: Added the no-arg add() widget method to the HTMLPanel. (issue1226801)

2010-12-16 Thread jlabanca
LGTM http://gwt-code-reviews.appspot.com/1226801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Make the logging framework provide a default uncaught exception handler. (issue1223802)

2010-12-16 Thread rjrjr
Unnur, is there a test I can extend for this? http://gwt-code-reviews.appspot.com/1223802/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Make the logging framework provide a default uncaught exception handler. (issue1223802)

2010-12-16 Thread rjrjr
Reviewers: unnurg, burdon_google.com, Description: Make the logging framework provide a default uncaught exception handler. Please review this at http://gwt-code-reviews.appspot.com/1223802/show Affected files: M user/src/com/google/gwt/logging/client/LogConfiguration.java Index: user/src/

[gwt-contrib] Re: Re-enable a few style tests now that old linux hosted mode is gone (issue1228801)

2010-12-16 Thread zundel
So this patch just backs out r5374. Can we be more specific in the comment and say that the error was pre gecko 1.8? (Mozilla 1.7 was linux hosted mode) (checked with svn diff -c 5374 user/test/com/google/gwt/dom/client/StyleTest.java) here is the log --

[gwt-contrib] Re: Re-enable XML parse test, which was previously failing in one Safari configuration in Production... (issue1229801)

2010-12-16 Thread zundel
Did you try running this test on a Mac? On 2010/12/16 21:57:24, fredsa wrote: http://gwt-code-reviews.appspot.com/1229801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Replace most user/developer visible references to hosted/web mode with Development/Production Mode (issue1221801)

2010-12-16 Thread zundel
LGTM thanks for looking into the tests as well On 2010/12/16 21:59:57, fredsa wrote: Addressed feedback and moved StyleTest and XMLTest to separate issues: Re-enable a few style tests now that old linux hosted mode is gone http://gwt-code-reviews.appspot.com/1228801 Re-enable XML pars

[gwt-contrib] Re: Replace most user/developer visible references to hosted/web mode with Development/Production Mode (issue1221801)

2010-12-16 Thread fredsa
Addressed feedback and moved StyleTest and XMLTest to separate issues: Re-enable a few style tests now that old linux hosted mode is gone http://gwt-code-reviews.appspot.com/1228801 Re-enable XML parse test, which was previously failing in one Safari configuration in Production Mode in the 1.

[gwt-contrib] Re-enable XML parse test, which was previously failing in one Safari configuration in Production... (issue1229801)

2010-12-16 Thread fredsa
Reviewers: zundel, Description: Re-enable XML parse test, which was previously failing in one Safari configuration in Production Mode in the 1.5 release branch Please review this at http://gwt-code-reviews.appspot.com/1229801/show Affected files: M user/test/com/google/gwt/xml/client/XMLTest

[gwt-contrib] Re-enable a few style tests now that old linux hosted mode is gone (issue1228801)

2010-12-16 Thread fredsa
Reviewers: zundel, Description: Re-enable a few style tests now that old linux hosted mode is gone Please review this at http://gwt-code-reviews.appspot.com/1228801/show Affected files: M user/test/com/google/gwt/dom/client/StyleTest.java Index: user/test/com/google/gwt/dom/client/StyleTes

[gwt-contrib] Re: Replace most user/developer visible references to hosted/web mode with Development/Production Mode (issue1221801)

2010-12-16 Thread fredsa
http://gwt-code-reviews.appspot.com/1221801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Experimental framework for generator result caching in dev mode. (issue1227801)

2010-12-16 Thread jbrosenberg
Reviewers: tobyr, zundel, robertvawter, Description: Experimental framework for generator result caching in dev mode. Please review this at http://gwt-code-reviews.appspot.com/1227801/show Affected files: A dev/core/src/com/google/gwt/core/ext/GeneratorContextExt.java A dev/core/src/com/go

[gwt-contrib] [google-web-toolkit] r9458 committed - Adding the 2.1.1 tag.

2010-12-16 Thread codesite-noreply
Revision: 9458 Author: p...@google.com Date: Thu Dec 16 12:57:29 2010 Log: Adding the 2.1.1 tag. http://code.google.com/p/google-web-toolkit/source/detail?r=9458 Added: /tags/2.1.1 -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] [google-web-toolkit] r9457 committed - Removing the 2.1.1 tag.

2010-12-16 Thread codesite-noreply
Revision: 9457 Author: p...@google.com Date: Thu Dec 16 12:56:48 2010 Log: Removing the 2.1.1 tag. http://code.google.com/p/google-web-toolkit/source/detail?r=9457 Deleted: /tags/2.1.1 -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] [google-web-toolkit] r9456 committed - Change era names in pt_PT to match pt (a Portuguese specialist replied...

2010-12-16 Thread codesite-noreply
Revision: 9456 Author: j...@google.com Date: Thu Dec 16 09:22:20 2010 Log: Change era names in pt_PT to match pt (a Portuguese specialist replied it should also be changed). Patch by: jat Review by: pdr http://code.google.com/p/google-web-toolkit/source/detail?r=9456 Modified: /trunk/user

[gwt-contrib] [google-web-toolkit] r9455 committed - Merge pt_PT fix for "AD" era name into 2.1 release branch.

2010-12-16 Thread codesite-noreply
Revision: 9455 Author: j...@google.com Date: Thu Dec 16 09:38:49 2010 Log: Merge pt_PT fix for "AD" era name into 2.1 release branch. http://code.google.com/p/google-web-toolkit/source/detail?r=9455 Modified: /releases/2.1/user/src/com/google/gwt/i18n/client/constants/DateTimeConstantsImpl_pt

[gwt-contrib] Re: Add a warning in GWTTestCase to help users that accidentally run a GWT JUnit Test as a JUnit Test. (issue1224801)

2010-12-16 Thread rdayal
Thanks for looking into this. A couple of questions/comments before I give the LGTM.. http://gwt-code-reviews.appspot.com/1224801/diff/1/2 File user/src/com/google/gwt/junit/client/GWTTestCase.java (right): http://gwt-code-reviews.appspot.com/1224801/diff/1/2#newcode445 user/src/com/google/gwt

[gwt-contrib] [google-web-toolkit] r9454 committed - Cherry picking r9453 into release branch.

2010-12-16 Thread codesite-noreply
Revision: 9454 Author: jlaba...@google.com Date: Thu Dec 16 07:24:08 2010 Log: Cherry picking r9453 into release branch. http://code.google.com/p/google-web-toolkit/source/detail?r=9454 Modified: /releases/2.1/user/src/com/google/gwt/user/cellview/client/CellList.java /releases/2.1/user/src/co

[gwt-contrib] Re: Fixing a bug in CellTable and CellList where we get an index out of bounds exception if we try t... (issue1225801)

2010-12-16 Thread jlabanca
committed as r9453 http://gwt-code-reviews.appspot.com/1225801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] [google-web-toolkit] r9453 committed - Fixing a bug in CellTable and CellList where we get an index out of bo...

2010-12-16 Thread codesite-noreply
Revision: 9453 Author: jlaba...@google.com Date: Thu Dec 16 06:19:53 2010 Log: Fixing a bug in CellTable and CellList where we get an index out of bounds exception if we try to access the keyboard selected element when keyboard selection is disabled. Review at http://gwt-code-reviews.appspot

[gwt-contrib] [google-web-toolkit] r9451 committed - escape wikilink

2010-12-16 Thread codesite-noreply
Revision: 9451 Author: ncha...@google.com Date: Thu Dec 16 09:47:49 2010 Log: escape wikilink http://code.google.com/p/google-web-toolkit/source/detail?r=9451 Modified: /wiki/BeanValidation.wiki === --- /wiki/BeanValidation.wiki Thu Dec 16 09:46:38 2010 +++

[gwt-contrib] [google-web-toolkit] r9450 committed - warning bootstrap is broken

2010-12-16 Thread codesite-noreply
Revision: 9450 Author: ncha...@google.com Date: Thu Dec 16 09:46:38 2010 Log: warning bootstrap is broken http://code.google.com/p/google-web-toolkit/source/detail?r=9450 Modified: /wiki/BeanValidation.wiki === --- /wiki/BeanValidation.wiki Wed Dec 15 22:17

Re: [gwt-contrib] Re: Initial version of HTML5 Audio and Video (issue1195801)

2010-12-16 Thread Ray Ryan
On Wed, Dec 15, 2010 at 5:40 PM, wrote: > LGTM > > Don't forget test cases. > By which I'm sure John meant "before you submit this." > > You guys are doing an awesome job with the HTML5 stuff! I can't wait to > see this stuff in action. > > > http://gwt-code-reviews.appspot.com/1195801/diff/30

[gwt-contrib] Re: Copying the DOMImplIE6 setOpacity implementation into DOMImplIE8 because IE8 does not support th... (issue1211801)

2010-12-16 Thread jlabanca
committed as r9434 http://gwt-code-reviews.appspot.com/1211801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Fixing a bug in CellTable and CellList where we get an index out of bounds exception if we try t... (issue1225801)

2010-12-16 Thread pdr
On 2010/12/16 16:51:04, jlabanca wrote: LGTM http://gwt-code-reviews.appspot.com/1225801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Fixing a bug in CellTable and CellList where we get an index out of bounds exception if we try t... (issue1225801)

2010-12-16 Thread jlabanca
Reviewers: pdr, Description: Fixing a bug in CellTable and CellList where we get an index out of bounds exception if we try to access the keyboard selected element when keyboard selection is disabled. Please review this at http://gwt-code-reviews.appspot.com/1225801/show Affected files: M us

[gwt-contrib] Re: Add a warning in GWTTestCase to help users that accidentally run a GWT JUnit Test as a JUnit Test. (issue1224801)

2010-12-16 Thread jlabanca
LGTM http://gwt-code-reviews.appspot.com/1224801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Some of the unit tests in CreateEventTest have listeners (issue1223801)

2010-12-16 Thread jlabanca
LGTM http://gwt-code-reviews.appspot.com/1223801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Some of the unit tests in CreateEventTest have listeners (issue1223801)

2010-12-16 Thread zundel
http://gwt-code-reviews.appspot.com/1223801/diff/1/2 File user/test/com/google/gwt/user/client/ui/CreateEventTest.java (right): http://gwt-code-reviews.appspot.com/1223801/diff/1/2#newcode738 user/test/com/google/gwt/user/client/ui/CreateEventTest.java:738: listener.assertReceived(); On 2010/12/

[gwt-contrib] Re: Some of the unit tests in CreateEventTest have listeners (issue1223801)

2010-12-16 Thread zundel
http://gwt-code-reviews.appspot.com/1223801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Add a warning in GWTTestCase to help users that accidentally run a GWT JUnit Test as a JUnit Test. (issue1224801)

2010-12-16 Thread pdr
Reviewers: jlabanca, rdayal, Description: Add a warning in GWTTestCase to help users that accidentally run a GWT JUnit Test as a JUnit Test. This mistake is easy to make (especially in Eclipse.) Therefore, we display a helpful error suggesting the user run as a GWTJUnitTest if an exception is th

[gwt-contrib] Re: Some of the unit tests in CreateEventTest have listeners (issue1223801)

2010-12-16 Thread jlabanca
LGTM http://gwt-code-reviews.appspot.com/1223801/diff/1/2 File user/test/com/google/gwt/user/client/ui/CreateEventTest.java (right): http://gwt-code-reviews.appspot.com/1223801/diff/1/2#newcode738 user/test/com/google/gwt/user/client/ui/CreateEventTest.java:738: listener.assertReceived(); Shoul

[gwt-contrib] Some of the unit tests in CreateEventTest have listeners (issue1223801)

2010-12-16 Thread zundel
Reviewers: jlabanca, Description: Some of the unit tests in CreateEventTest have listeners that fire multiple times. This occasionally causes a failure in an unrelated test. Assuming that the extra firings are spurious, I've added an explicit 'cancel()' method to bypass the assertions if they f

[gwt-contrib] Re: Comment on DefaultLocaleBinding in google-web-toolkit

2010-12-16 Thread codesite-noreply
Comment by eplischke: ... or if you have multi-locales than you can do this way: For more information: http://code.google.com/p/google-web-toolkit/wiki/DefaultLocaleBinding -- http://groups.google.com/group/Google-Web-Toolkit-Contributors