[gwt-contrib] Issue 5752: ListEditor.setValue fails once the underlying AutoBean has been frozen (issue1284801)

2011-01-13 Thread t . broyer
Reviewers: bobv, rjrjr, Description: Copied from the issue tracker: The real issue seems to be in how com.google.gwt.requestfactory.client.impl.RequestFactoryEditorDelegate determines whether it shouldFlush() or not: it only deals with am I in 'read-only' mode or not (if request==null, edited

[gwt-contrib] GWT 2.2 ?

2011-01-13 Thread Thomas Broyer
Hi G-men, You seem to be working actively on preparing a 2.2 release very soon, and I'm hoping that some issues (some of which I've provided a patch for) will be resolved in this release. One issue that's a blocker for us is related to ListEditor when used with

[gwt-contrib] Tone down the experimental api warning. (issue1285801)

2011-01-13 Thread pdr
Reviewers: rjrjr, Description: Tone down the experimental api warning. Please review this at http://gwt-code-reviews.appspot.com/1285801/show Affected files: M user/src/com/google/gwt/canvas/client/Canvas.java M user/src/com/google/gwt/canvas/dom/client/CanvasGradient.java M

[gwt-contrib] [google-web-toolkit] r9529 committed - Spelling

2011-01-13 Thread codesite-noreply
Revision: 9529 Author: ncha...@google.com Date: Thu Jan 13 08:23:15 2011 Log: Spelling http://code.google.com/p/google-web-toolkit/source/detail?r=9529 Modified: /wiki/BeanValidation.wiki === --- /wiki/BeanValidation.wiki Mon Jan 3 14:43:03 2011 +++

[gwt-contrib] Re: Tone down the experimental api warning. (issue1285801)

2011-01-13 Thread jlabanca
LGTM http://gwt-code-reviews.appspot.com/1285801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] [google-web-toolkit] r9530 committed - Cherry picking r9527 into releases/2.2 branch for http://code.google....

2011-01-13 Thread codesite-noreply
Revision: 9530 Author: gwt.mirror...@gmail.com Date: Thu Jan 13 09:45:23 2011 Log: Cherry picking r9527 into releases/2.2 branch for http://code.google.com/p/google-web-toolkit/issues/detail?id=5869 http://code.google.com/p/google-web-toolkit/source/detail?r=9530 Modified:

[gwt-contrib] [google-web-toolkit] r9531 committed - Removing client-side validation from GWT 2.2 as it is not yet in a wor...

2011-01-13 Thread codesite-noreply
Revision: 9531 Author: rchan...@google.com Date: Thu Jan 13 04:56:44 2011 Log: Removing client-side validation from GWT 2.2 as it is not yet in a working state. http://code.google.com/p/google-web-toolkit/source/detail?r=9531 Deleted: /releases/2.2/samples/validation

[gwt-contrib] [google-web-toolkit] r9533 committed - Add TCK jars to user .classpath...

2011-01-13 Thread codesite-noreply
Revision: 9533 Author: ncha...@google.com Date: Thu Jan 13 05:39:51 2011 Log: Add TCK jars to user .classpath Review by: knor...@google.com http://code.google.com/p/google-web-toolkit/source/detail?r=9533 Modified: /trunk/eclipse/user/.classpath === ---

[gwt-contrib] Re: This change adds couple of things: (issue1251801)

2011-01-13 Thread xtof
http://gwt-code-reviews.appspot.com/1251801/diff/44001/45010 File user/src/com/google/gwt/user/server/Util.java (right): http://gwt-code-reviews.appspot.com/1251801/diff/44001/45010#newcode76 user/src/com/google/gwt/user/server/Util.java:76: * @throws IllegalStateException if duplicate cookies

[gwt-contrib] Re: Issue 5752: ListEditor.setValue fails once the underlying AutoBean has been frozen (issue1284801)

2011-01-13 Thread BobV
Attached to http://code.google.com/p/google-web-toolkit/issues/detail?id=5873 -- Bob Vawter Google Web Toolkit Team -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] [google-web-toolkit] r9534 committed - Cherry picking r9528 into the release branch

2011-01-13 Thread codesite-noreply
Revision: 9534 Author: p...@google.com Date: Thu Jan 13 07:07:25 2011 Log: Cherry picking r9528 into the release branch http://code.google.com/p/google-web-toolkit/source/detail?r=9534 Modified: /releases/2.2/user/src/com/google/gwt/canvas/client/Canvas.java

[gwt-contrib] Re: This change adds couple of things: (issue1251801)

2011-01-13 Thread jat
http://gwt-code-reviews.appspot.com/1251801/diff/44001/45010 File user/src/com/google/gwt/user/server/Util.java (right): http://gwt-code-reviews.appspot.com/1251801/diff/44001/45010#newcode76 user/src/com/google/gwt/user/server/Util.java:76: * @throws IllegalStateException if duplicate cookies

[gwt-contrib] Re: This change adds couple of things: (issue1251801)

2011-01-13 Thread Christoph Kern
On Thu, Jan 13, 2011 at 10:53, j...@google.com wrote: http://gwt-code-reviews.appspot.com/1251801/diff/44001/45010 File user/src/com/google/gwt/user/server/Util.java (right): http://gwt-code-reviews.appspot.com/1251801/diff/44001/45010#newcode76

[gwt-contrib] Add optimizations for redundant and trivial 'switch' statements (issue1286801)

2011-01-13 Thread rice
Reviewers: scottb, robertvawter, Description: Add optimizations for redundant and trivial 'switch' statements Please review this at http://gwt-code-reviews.appspot.com/1286801/show Affected files: M dev/core/src/com/google/gwt/dev/jjs/JavaToJavaScriptCompiler.java A

[gwt-contrib] [google-web-toolkit] r9535 committed - Change the gecko 2.0 dynamic linux libs to be stripped so that codesit...

2011-01-13 Thread codesite-noreply
Revision: 9535 Author: gwt.mirror...@gmail.com Date: Thu Jan 13 12:31:52 2011 Log: Change the gecko 2.0 dynamic linux libs to be stripped so that codesite doesn't barf on them and to save space) http://code.google.com/p/google-web-toolkit/source/detail?r=9535 Added:

[gwt-contrib] [google-web-toolkit] r9536 committed - Public (t.bro...@gmail.com):...

2011-01-13 Thread codesite-noreply
Revision: 9536 Author: rj...@google.com Date: Thu Jan 13 12:25:22 2011 Log: Public (t.bro...@gmail.com): Uses the java.beans.Introspector#decapitalize instead of manual decapitalization of setter-method names to attribute names. This means setHTML() can be set using an all-caps HTML= attribute,

[gwt-contrib] [google-web-toolkit] r9537 committed - Cherry picking r9536 releases/[rel_#] for issue 4781

2011-01-13 Thread codesite-noreply
Revision: 9537 Author: rj...@google.com Date: Thu Jan 13 09:40:51 2011 Log: Cherry picking r9536 releases/[rel_#] for issue 4781 http://code.google.com/p/google-web-toolkit/source/detail?r=9537 Modified: /releases/2.2/user/src/com/google/gwt/uibinder/rebind/model/OwnerFieldClass.java

[gwt-contrib] [google-web-toolkit] r9538 committed - Cache ModuleDefs by context ClassLoader....

2011-01-13 Thread codesite-noreply
Revision: 9538 Author: sco...@google.com Date: Thu Jan 13 09:40:26 2011 Log: Cache ModuleDefs by context ClassLoader. GWT Designer need separated class loader for every open project. http://gwt-code-reviews.appspot.com/1274801/show Patch by: amitin Review by: me,zundel

[gwt-contrib] Fixes a bug in JavaScriptObject support in DevMode that prevents dispatching (issue1287801)

2011-01-13 Thread knorton
Reviewers: bobv, scottb, Description: Fixes a bug in JavaScriptObject support in DevMode that prevents dispatching through an interface to a method defined in a super-interface. The change also includes a new test to serve as a regression test. Please review this at

[gwt-contrib] Re: Fixes a bug in JavaScriptObject support in DevMode that prevents dispatching (issue1287801)

2011-01-13 Thread bobv
LGTM http://gwt-code-reviews.appspot.com/1287801/diff/1/2 File dev/core/src/com/google/gwt/dev/shell/rewrite/RewriteSingleJsoImplDispatches.java (right): http://gwt-code-reviews.appspot.com/1287801/diff/1/2#newcode272

[gwt-contrib] Re: Fixes a bug in JavaScriptObject support in DevMode that prevents dispatching (issue1287801)

2011-01-13 Thread cromwellian
http://gwt-code-reviews.appspot.com/1287801/diff/1/3 File user/test/com/google/gwt/dev/jjs/test/singlejso/TypeHierarchyTest.java (right): http://gwt-code-reviews.appspot.com/1287801/diff/1/3#newcode205 user/test/com/google/gwt/dev/jjs/test/singlejso/TypeHierarchyTest.java:205: Element element =

[gwt-contrib] [google-web-toolkit] r9539 committed - Cherry picking r9538 into releases/2.2

2011-01-13 Thread codesite-noreply
Revision: 9539 Author: rj...@google.com Date: Thu Jan 13 10:46:47 2011 Log: Cherry picking r9538 into releases/2.2 http://code.google.com/p/google-web-toolkit/source/detail?r=9539 Modified: /releases/2.2/dev/core/src/com/google/gwt/dev/cfg/ModuleDefLoader.java

[gwt-contrib] Re: Fixes a bug in JavaScriptObject support in DevMode that prevents dispatching (issue1287801)

2011-01-13 Thread knorton
http://gwt-code-reviews.appspot.com/1287801/diff/1/3 File user/test/com/google/gwt/dev/jjs/test/singlejso/TypeHierarchyTest.java (right): http://gwt-code-reviews.appspot.com/1287801/diff/1/3#newcode205 user/test/com/google/gwt/dev/jjs/test/singlejso/TypeHierarchyTest.java:205: Element element =

Re: [gwt-contrib] Re: Fixes a bug in JavaScriptObject support in DevMode that prevents dispatching (issue1287801)

2011-01-13 Thread Ray Cromwell
On Thu, Jan 13, 2011 at 2:24 PM, knor...@google.com wrote: To be honest, I don't know what our JSO tests currently cover. My intent was to put my case in as a regression and try to follow up with some more robust testing strategies. Do you anticipate a problem that is not covered with our

[gwt-contrib] Re: Fixes a bug in JavaScriptObject support in DevMode that prevents dispatching (issue1287801)

2011-01-13 Thread knorton
No, it's a good point. I'm realizing we probably need some more tests covering a lot fo the patterns that are possible after Joel's change to loosen some of the interface restrictions. On 2011/01/13 22:32:43, cromwellian wrote: On Thu, Jan 13, 2011 at 2:24 PM, mailto:knor...@google.com wrote:

[gwt-contrib] Re: Fixes a bug in JavaScriptObject support in DevMode that prevents dispatching (issue1287801)

2011-01-13 Thread knorton
http://gwt-code-reviews.appspot.com/1287801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: At application startup, assert that the specified user.agent selection property value indeed mat... (issue1278801)

2011-01-13 Thread fredsa
http://gwt-code-reviews.appspot.com/1278801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Add optimizations for redundant and trivial 'switch' statements (issue1286801)

2011-01-13 Thread scottb
Mostly LG. http://gwt-code-reviews.appspot.com/1286801/diff/1/3 File dev/core/src/com/google/gwt/dev/js/JsDuplicateCaseFolder.java (right): http://gwt-code-reviews.appspot.com/1286801/diff/1/3#newcode130 dev/core/src/com/google/gwt/dev/js/JsDuplicateCaseFolder.java:130: String body =

[gwt-contrib] Re: This change adds couple of things: (issue1251801)

2011-01-13 Thread meder
http://gwt-code-reviews.appspot.com/1251801/diff/44001/45010 File user/src/com/google/gwt/user/server/Util.java (right): http://gwt-code-reviews.appspot.com/1251801/diff/44001/45010#newcode76 user/src/com/google/gwt/user/server/Util.java:76: * @throws IllegalStateException if duplicate cookies

[gwt-contrib] Re: This change adds couple of things: (issue1251801)

2011-01-13 Thread meder
http://gwt-code-reviews.appspot.com/1251801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Move the TCK report task to user/build.xml and (issue1288801)

2011-01-13 Thread nchalko
Reviewers: rchandia, Description: Move the TCK report task to user/build.xml and delete samples/validationtck Please review this at http://gwt-code-reviews.appspot.com/1288801/show Affected files: D samples/validationtck/README.txt D samples/validationtck/build.xml D

[gwt-contrib] [google-web-toolkit] r9540 committed - Adds a hook to invalidate a rebind within dev mode (from Alexander...

2011-01-13 Thread codesite-noreply
Revision: 9540 Author: gwt.mirror...@gmail.com Date: Thu Jan 13 13:31:09 2011 Log: Adds a hook to invalidate a rebind within dev mode (from Alexander Mitin) Review at http://gwt-code-reviews.appspot.com/1275801 Review by: rj...@google.com

[gwt-contrib] Re: Fixes a bug in JavaScriptObject support in DevMode that prevents dispatching (issue1287801)

2011-01-13 Thread knorton
http://gwt-code-reviews.appspot.com/1287801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Fix a generics compile error on OSX java 1.6 (issue1289801)

2011-01-13 Thread nchalko
Reviewers: rchandia, Description: Fix a generics compile error on OSX java 1.6 Please review this at http://gwt-code-reviews.appspot.com/1289801/show Affected files: M user/test/com/google/gwt/requestfactory/client/RequestFactoryTestBase.java Index:

[gwt-contrib] Re: Fixes a bug in JavaScriptObject support in DevMode that prevents dispatching (issue1287801)

2011-01-13 Thread knorton
Thanks Ray, I didn't actually change the logic in writeTrampoline in the last patch, but I realized that an identical change there actually fixes this problem for the Java objects as well. Can you give it another quick look? http://gwt-code-reviews.appspot.com/1287801/show --

[gwt-contrib] Re: Fixes a bug in JavaScriptObject support in DevMode that prevents dispatching (issue1287801)

2011-01-13 Thread cromwellian
LGTM http://gwt-code-reviews.appspot.com/1287801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] [google-web-toolkit] r9541 committed - Cherry picking r9540 into releases/2.2, Adds a hook to invalidate a re...

2011-01-13 Thread codesite-noreply
Revision: 9541 Author: gwt.mirror...@gmail.com Date: Thu Jan 13 13:57:26 2011 Log: Cherry picking r9540 into releases/2.2, Adds a hook to invalidate a rebind within dev mode http://code.google.com/p/google-web-toolkit/source/detail?r=9541 Modified:

[gwt-contrib] Re: Fix a generics compile error on OSX java 1.6 (issue1289801)

2011-01-13 Thread rchandia
LGTM On 2011/01/14 01:17:55, Nick Chalko wrote: http://gwt-code-reviews.appspot.com/1289801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] [google-web-toolkit] r9542 committed - Fixes a bug in JavaScriptObject support in DevMode that prevents dispa...

2011-01-13 Thread codesite-noreply
Revision: 9542 Author: knor...@google.com Date: Thu Jan 13 15:08:15 2011 Log: Fixes a bug in JavaScriptObject support in DevMode that prevents dispatching through an interface to a method defined in a super-interface. The change also includes a new test to serve as a regression test. Review

[gwt-contrib] typo in documentation?

2011-01-13 Thread Arthur Kalmenson
Hey everyone, Just a quick question, I was looking through the MVP docs: http://code.google.com/webtoolkit/doc/latest/DevGuideMvpActivitiesAndPlaces.html#Places and in the Place section there is a paragraph that eludes to a BasicPlace class. It says: Many Places in your app might not save any