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

2011-05-09 Thread codesite-noreply
Comment by kujans...@googlemail.com: released in 2.3 http://code.google.com/webtoolkit/doc/latest/DevGuideSecurityRpcXsrf.html For more information: http://code.google.com/p/google-web-toolkit/wiki/RpcAuth -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Calculates result of getSourceName() one time, so reuse because of CU cache. (issue1438801)

2011-05-09 Thread zundel
http://gwt-code-reviews.appspot.com/1438801/diff/3001/dev/core/test/com/google/gwt/dev/javac/CompiledClassTest.java File dev/core/test/com/google/gwt/dev/javac/CompiledClassTest.java (right):

[gwt-contrib] Re: Calculates result of getSourceName() one time, so reuse because of CU cache. (issue1438801)

2011-05-09 Thread zundel
LGTM http://gwt-code-reviews.appspot.com/1438801/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

Re: [gwt-contrib] rfc on 6068 fix

2011-05-09 Thread Ray Ryan
Bob V is the authority here, but he's on vacation this week. On Sun, May 8, 2011 at 7:08 AM, Patrick Julien pjul...@gmail.com wrote: Here is what I am planning to do to fix issue 6068 http://code.google.com/p/google-web-toolkit/issues/detail?id=6068 1. Add to BeanMethod.java (package

[gwt-contrib] [google-web-toolkit] r10162 committed - Calculates result of getSourceName() one time, so reuse because of CU ...

2011-05-09 Thread codesite-noreply
Revision: 10162 Author: scheg...@google.com Date: Mon May 9 07:42:14 2011 Log: Calculates result of getSourceName() one time, so reuse because of CU cache. Prepares SAXParserFActory one time, so avoids expensive ClassLoader lookups. Review at

[gwt-contrib] Disabling test for null on getBrowserById for DOM elements without DTD when running on Chrome 12 (issue1443801)

2011-05-09 Thread rchandia
Reviewers: jat, Frank, Description: Disabling test for null on getBrowserById for DOM elements without DTD when running on Chrome 12 Please review this at http://gwt-code-reviews.appspot.com/1443801/ Affected files: M user/test/com/google/gwt/xml/client/XMLTest.java Index:

[gwt-contrib] Re: Minor refactoring of dashboard interfaces in GWT. I'd made these changes this week in another ... (issue1439801)

2011-05-09 Thread jhumphries
http://gwt-code-reviews.appspot.com/1439801/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] [google-web-toolkit] r10164 committed - Edited wiki page BeanValidation through web user interface.

2011-05-09 Thread codesite-noreply
Revision: 10164 Author: ncha...@google.com Date: Mon May 9 12:10:10 2011 Log: Edited wiki page BeanValidation through web user interface. http://code.google.com/p/google-web-toolkit/source/detail?r=10164 Modified: /wiki/BeanValidation.wiki ===

[gwt-contrib] [google-web-toolkit] r10165 committed - Introduces read only task view with an edit button. Gets around...

2011-05-09 Thread codesite-noreply
Revision: 10165 Author: rj...@google.com Date: Mon May 9 09:40:33 2011 Log: Introduces read only task view with an edit button. Gets around problems when we try to edit a TaskProxyImpl, which is unpossible. http://code.google.com/p/google-web-toolkit/source/detail?r=10165 Added:

[gwt-contrib] [google-web-toolkit] r10166 committed - Fix the transition direction between read and task views

2011-05-09 Thread codesite-noreply
Revision: 10166 Author: rj...@google.com Date: Mon May 9 10:10:43 2011 Log: Fix the transition direction between read and task views http://code.google.com/p/google-web-toolkit/source/detail?r=10166 Modified:

[gwt-contrib] Better error reporting (issue1444801)

2011-05-09 Thread rjrjr
Reviewers: jlabanca, Description: Better error reporting Please review this at http://gwt-code-reviews.appspot.com/1444801/ Affected files: M samples/mobilewebapp/src/main/com/google/gwt/sample/mobilewebapp/client/App.java M

[gwt-contrib] [google-web-toolkit] r10167 committed - Implementing custom TaskReadViews for tablet, desktop, and mobile.

2011-05-09 Thread codesite-noreply
Revision: 10167 Author: jlaba...@google.com Date: Mon May 9 10:25:15 2011 Log: Implementing custom TaskReadViews for tablet, desktop, and mobile. http://code.google.com/p/google-web-toolkit/source/detail?r=10167 Added:

[gwt-contrib] Re: Better error reporting (issue1444801)

2011-05-09 Thread rjrjr
Not quite ready for review http://gwt-code-reviews.appspot.com/1444801/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Better error reporting (issue1444801)

2011-05-09 Thread rjrjr
http://gwt-code-reviews.appspot.com/1444801/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Serialization of Final Fields in RPC (issue1380807)

2011-05-09 Thread zhuyi
I don't have the full context, but adding flakiness is not a good idea. According to the current code structure, if we want to make the variable non-static, we may need to pass the propertyOracle parameter to many methods, because shouldConsiderFieldsForSerialization method in

[gwt-contrib] Re: Disabling test for null on getBrowserById for DOM elements without DTD when running on Chrome 12 (issue1443801)

2011-05-09 Thread tobyr
LGTM http://gwt-code-reviews.appspot.com/1443801/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Better error reporting (issue1444801)

2011-05-09 Thread Ray Ryan
Verbal LGTM from John. r10167 On Mon, May 9, 2011 at 1:40 PM, rj...@google.com wrote: http://gwt-code-reviews.appspot.com/1444801/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Remove JProgram.jsniMap in favor of local accounting. (issue1445801)

2011-05-09 Thread scottb
Reviewers: jbrosenberg, cromwellian, Please review this at http://gwt-code-reviews.appspot.com/1445801/ Affected files: M dev/core/src/com/google/gwt/dev/jjs/ast/JProgram.java A dev/core/src/com/google/gwt/dev/jjs/ast/js/JsniClassLiteral.java M