[gwt-contrib] Re: PlaceHistoryManager

2010-08-23 Thread Thomas Broyer
On 21 août, 20:13, Patrick Julien pjul...@gmail.com wrote: I've been experimenting with PlaceHistoryManager and company.  So far, I have the following comments: First, I've only read through the design-doc Wave and the code, I haven't yet experimented with it (as I first have to migrate my

Re: [gwt-contrib] Re: PlaceHistoryManager

2010-08-23 Thread Patrick Julien
On Mon, Aug 23, 2010 at 5:16 AM, Thomas Broyer t.bro...@gmail.com wrote: On 21 août, 20:13, Patrick Julien pjul...@gmail.com wrote: I've been experimenting with PlaceHistoryManager and company.  So far, I have the following comments: First, I've only read through the design-doc Wave and the

[gwt-contrib] Re: Added all safehtml packages. (issue771801)

2010-08-23 Thread pdr
http://gwt-code-reviews.appspot.com/771801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Added all safehtml packages. (issue771801)

2010-08-23 Thread pdr
http://gwt-code-reviews.appspot.com/771801/diff/11002/15007 File user/src/com/google/gwt/safehtml/shared/OnlyToBeUsedInGeneratedCodeStringBlessedAsSafeHtml.java (right): http://gwt-code-reviews.appspot.com/771801/diff/11002/15007#newcode26

[gwt-contrib] Re: Public: Support classes for a future generating GWT validators. (issue766802)

2010-08-23 Thread bobv
http://gwt-code-reviews.appspot.com/766802/diff/1/2 File user/src/com/google/gwt/validation/client/AbstractBeanDescriptor.java (right): http://gwt-code-reviews.appspot.com/766802/diff/1/2#newcode27 user/src/com/google/gwt/validation/client/AbstractBeanDescriptor.java:27: * Abstract

[gwt-contrib] Re: Public: Start of a sample application showing GWT validation. (issue760802)

2010-08-23 Thread bobv
http://gwt-code-reviews.appspot.com/760802/diff/6001/1029 File samples/validation/src/com/google/gwt/sample/validation/Validation.gwt.xml (right): http://gwt-code-reviews.appspot.com/760802/diff/6001/1029#newcode3 samples/validation/src/com/google/gwt/sample/validation/Validation.gwt.xml:3: !--

[gwt-contrib] Re: Added all safehtml packages. (issue771801)

2010-08-23 Thread t . broyer
http://gwt-code-reviews.appspot.com/771801/diff/28002/6006 File user/src/com/google/gwt/safehtml/shared/OnlyToBeUsedInGeneratedCodeStringBlessedAsSafeHtml.java (right): http://gwt-code-reviews.appspot.com/771801/diff/28002/6006#newcode32

[gwt-contrib] Removing the Expenses sample from bikeshed now that it has been converted to a sample. I'm leav... (issue791801)

2010-08-23 Thread jlabanca
Reviewers: cramsdale, Description: Removing the Expenses sample from bikeshed now that it has been converted to a sample. I'm leaving the Scaffold app for now until I know that its safe to delete it. Please review this at http://gwt-code-reviews.appspot.com/791801/show Affected files: M

Re: [gwt-contrib] Removing the Expenses sample from bikeshed now that it has been converted to a sample. I'm leav... (issue791801)

2010-08-23 Thread Ray Ryan
LGTM On Mon, Aug 23, 2010 at 8:31 AM, jlaba...@google.com wrote: Reviewers: cramsdale, Description: Removing the Expenses sample from bikeshed now that it has been converted to a sample. I'm leaving the Scaffold app for now until I know that its safe to delete it. Please review this at

[gwt-contrib] Re: PlaceHistoryManager

2010-08-23 Thread Thomas Broyer
On 23 août, 14:49, Patrick Julien pjul...@gmail.com wrote: On Mon, Aug 23, 2010 at 5:16 AM, Thomas Broyer t.bro...@gmail.com wrote: On 21 août, 20:13, Patrick Julien pjul...@gmail.com wrote: I've been experimenting with PlaceHistoryManager and company.  So far, I have the following

[gwt-contrib] Re: Removing the Expenses sample from bikeshed now that it has been converted to a sample. I'm leav... (issue791801)

2010-08-23 Thread cramsdale
LGTM http://gwt-code-reviews.appspot.com/791801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: ClientBundleWithLookup.getResource(String) now uses a HashMap in DevMode. (issue789801)

2010-08-23 Thread scottb
http://gwt-code-reviews.appspot.com/789801/diff/1/2 File user/src/com/google/gwt/resources/rebind/context/AbstractClientBundleGenerator.java (right): http://gwt-code-reviews.appspot.com/789801/diff/1/2#newcode702

[gwt-contrib] [google-web-toolkit] r8622 committed - Tag 2.1.0-ms3

2010-08-23 Thread codesite-noreply
Revision: 8622 Author: gwt.mirror...@gmail.com Date: Mon Aug 23 09:33:42 2010 Log: Tag 2.1.0-ms3 http://code.google.com/p/google-web-toolkit/source/detail?r=8622 Added: /tags/2.1.0-ms3 -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

Re: [gwt-contrib] Re: PlaceHistoryManager

2010-08-23 Thread Patrick Julien
On Mon, Aug 23, 2010 at 12:07 PM, Thomas Broyer t.bro...@gmail.com wrote: implementation you want, even with the stock Proxy{,List}Place places. No it's not.  This is the tokens returns by RequestFactory based on your record classes.  The *Place objects that you use have no bearing on what

[gwt-contrib] Re: Removing the Expenses sample from bikeshed now that it has been converted to a sample. I'm leav... (issue791801)

2010-08-23 Thread jlabanca
committed as r8624 http://gwt-code-reviews.appspot.com/791801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] [google-web-toolkit] r8625 committed - ClientBundleWithLookup.getResource(String) now uses a HashMap in DevMo...

2010-08-23 Thread codesite-noreply
Revision: 8625 Author: sco...@google.com Date: Mon Aug 23 07:33:33 2010 Log: ClientBundleWithLookup.getResource(String) now uses a HashMap in DevMode. This is the first of a couple of patches that will remove certain JSNI code paths in generated code from being traversed in DevMode. This

[gwt-contrib] [google-web-toolkit] r8626 committed - svn merge https://google-web-toolkit.googlecode.com/svn/trunk -r8599:8...

2010-08-23 Thread codesite-noreply
Revision: 8626 Author: gwt.mirror...@gmail.com Date: Mon Aug 23 11:11:04 2010 Log: svn merge https://google-web-toolkit.googlecode.com/svn/trunk -r8599:8600 . Plus, updated branch-info.txt with this information http://code.google.com/p/google-web-toolkit/source/detail?r=8626 Modified:

[gwt-contrib] [google-web-toolkit] r8627 committed - Put the commit number in branch-info.txt

2010-08-23 Thread codesite-noreply
Revision: 8627 Author: gwt.mirror...@gmail.com Date: Mon Aug 23 11:12:23 2010 Log: Put the commit number in branch-info.txt http://code.google.com/p/google-web-toolkit/source/detail?r=8627 Modified: /branches/2.1M3/branch-info.txt === ---

[gwt-contrib] UiBinder. Design time tweaks for @UiField (issue792801)

2010-08-23 Thread Konstantin . Scheglov
Reviewers: rjrjr, Description: This patch contains changes required for reliable editing of ui:field attribute in template and corresponding @UiField in Java. Solved problems: 1. When GWT Designer calls createAndBindUi(), it can not provide owner, because we can not instantiate it, and it is

[gwt-contrib] Re: Make xsiframe linker use a .js file for hosted mode so that cross site hosted mode will work (issue784801)

2010-08-23 Thread unnurg
http://gwt-code-reviews.appspot.com/784801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Fix memory leaks in DevMode (issue766803)

2010-08-23 Thread conroy
Discussed in person with jat and scottb: Tests are failing for this change because the ThreadLocal assumptions don't actually hold under JUnit. As a result, I'm going to be dropping this change in favor of more explicitly caching based on the generator context lifecycle. On 2010/08/20 20:46:05,

[gwt-contrib] Re: Make xsiframe linker use a .js file for hosted mode so that cross site hosted mode will work (issue784801)

2010-08-23 Thread spoon
LGTM http://gwt-code-reviews.appspot.com/784801/diff/3002/7001 File dev/core/src/com/google/gwt/core/ext/linker/impl/hosted.js (right): http://gwt-code-reviews.appspot.com/784801/diff/3002/7001#newcode4 dev/core/src/com/google/gwt/core/ext/linker/impl/hosted.js:4: var $moduleName =

[gwt-contrib] [google-web-toolkit] r8628 committed - Turn down noise level on AbstractLocalizableImplCreator from INFO to T...

2010-08-23 Thread codesite-noreply
Revision: 8628 Author: gwt.mirror...@gmail.com Date: Mon Aug 23 10:16:50 2010 Log: Turn down noise level on AbstractLocalizableImplCreator from INFO to TRACE. Review by: j...@google.com http://code.google.com/p/google-web-toolkit/source/detail?r=8628 Modified:

[gwt-contrib] [google-web-toolkit] r8629 committed - Cleanups on NameFactory and AbstractClientBundleGenerator....

2010-08-23 Thread codesite-noreply
Revision: 8629 Author: gwt.mirror...@gmail.com Date: Mon Aug 23 11:34:16 2010 Log: Cleanups on NameFactory and AbstractClientBundleGenerator. Suggested by: bobv Review by: bobv http://code.google.com/p/google-web-toolkit/source/detail?r=8629 Modified:

[gwt-contrib] Re: Public: Start of a sample application showing GWT validation. (issue760802)

2010-08-23 Thread nchalko
Bob do you think the Contacts App from http://code.google.com/webtoolkit/articles/mvp-architecture.html#model Is a better starting place http://gwt-code-reviews.appspot.com/760802/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] SafeHtml Templates (issue793801)

2010-08-23 Thread pdr
Reviewers: jat, rice, knorton, xtof, Description: SafeHtml Templates The SafeHtml Templates were removed from the main SafeHtml review, but are being added again here so that widgets can start using this api, even while a newer parser is in the works. This is a first pass at templates, and the

[gwt-contrib] [google-web-toolkit] r8630 committed - Fix for bad create requests. Get rid of the squirrely logic in Request...

2010-08-23 Thread codesite-noreply
Revision: 8630 Author: amitman...@google.com Date: Mon Aug 23 16:33:22 2010 Log: Fix for bad create requests. Get rid of the squirrely logic in RequestFactory tracking creates. Patch by: amitmanjhi Review by: rjrjr (desk review)

[gwt-contrib] AbstractEditView now requests all the paths it needs (issue786802)

2010-08-23 Thread rjrjr
Reviewers: amitmanjhi, Description: AbstractEditView now requests all the paths it needs Review by: amitman...@google.com Please review this at http://gwt-code-reviews.appspot.com/786802/show Affected files: M

[gwt-contrib] Re: AbstractEditView now requests all the paths it needs (issue786802)

2010-08-23 Thread amitmanjhi
LGTM http://gwt-code-reviews.appspot.com/786802/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] [google-web-toolkit] r8631 committed - AbstractEditView now requests all the paths it needs...

2010-08-23 Thread codesite-noreply
Revision: 8631 Author: gwt.mirror...@gmail.com Date: Mon Aug 23 18:21:12 2010 Log: AbstractEditView now requests all the paths it needs Review at http://gwt-code-reviews.appspot.com/786802 Review by: amitman...@google.com http://code.google.com/p/google-web-toolkit/source/detail?r=8631

[gwt-contrib] [google-web-toolkit] r8633 committed - Fix infinite recursion in JsonRequestProcessor

2010-08-23 Thread codesite-noreply
Revision: 8633 Author: cromwell...@google.com Date: Mon Aug 23 19:35:44 2010 Log: Fix infinite recursion in JsonRequestProcessor http://code.google.com/p/google-web-toolkit/source/detail?r=8633 Modified: /trunk/user/src/com/google/gwt/requestfactory/server/JsonRequestProcessor.java

[gwt-contrib] [google-web-toolkit] r8634 committed - svn merge https://google-web-toolkit.googlecode.com/svn/trunk -r8629:8...

2010-08-23 Thread codesite-noreply
Revision: 8634 Author: gwt.mirror...@gmail.com Date: Mon Aug 23 22:48:15 2010 Log: svn merge https://google-web-toolkit.googlecode.com/svn/trunk -r8629:8633 . Merged r8630, 8631, 8632, 8633 http://code.google.com/p/google-web-toolkit/source/detail?r=8634 Modified:

[gwt-contrib] [google-web-toolkit] r8635 committed - Updated branch-info.txt

2010-08-23 Thread codesite-noreply
Revision: 8635 Author: gwt.mirror...@gmail.com Date: Mon Aug 23 22:49:44 2010 Log: Updated branch-info.txt http://code.google.com/p/google-web-toolkit/source/detail?r=8635 Modified: /branches/2.1M3/branch-info.txt === --- /branches/2.1M3/branch-info.txt