Revision: 8465
Author: rj...@google.com
Date: Mon Aug 2 18:45:49 2010
Log: Fixes UiBinder MenuBar parser to allow MenuItemSeparator, and custom
subclasses of MenuItem.
Patch by konstantin.scheg...@gmail.com
Review by rj...@google.com
http://gwt-code-reviews.appspot.com/702803
Fixes gwt issue
Revision: 8455
Author: rj...@google.com
Date: Mon Aug 2 10:03:54 2010
Log: Remove 1.6isms from RequestFactory, and fix DynaTableRf (id's
still must be Long, and at the moment it actually counts).
Review at http://gwt-code-reviews.appspot.com/721804
Review by: robertvaw...@google.com
Revision: 8454
Author: tomer...@google.com
Date: Mon Aug 2 09:47:24 2010
Log: Adds integral BiDi support to Label.java and its descendants.
Introduces a new interface, HasAutoHorizontalAlignment.
Review at http://gwt-code-reviews.appspot.com/642803
Revision: 8461
Author: jlaba...@google.com
Date: Mon Aug 2 14:09:21 2010
Log: Disabling part of HTMLTest because it fails cross browser.
http://code.google.com/p/google-web-toolkit/source/detail?r=8461
Modified:
/trunk/user/test/com/google/gwt/user/client/ui/HTMLTest.java
Revision: 8463
Author: unn...@google.com
Date: Mon Aug 2 15:57:18 2010
Log: Change the way we include the User Auth files, keeping more of them
checked into GWT code rather than generated by roo
Review at http://gwt-code-reviews.appspot.com/721803
Review by: rj...@google.com
Revision: 8459
Author: jlaba...@google.com
Date: Mon Aug 2 13:10:45 2010
Log: Fixing HTMLTest so that it doesn't assume tag names returned from
getInnerHtml are lowercase. IE automatically converts tag names to
uppercase.
http://code.google.com/p/google-web-toolkit/source/detail?r=8459
Revision: 8462
Author: con...@google.com
Date: Mon Aug 2 14:49:52 2010
Log: SpeedTracerLogger: Don't throw exceptions on missed events. Instead,
try to recover gracefully.
Review at http://gwt-code-reviews.appspot.com/674805
Review by: zun...@google.com
Revision: 8460
Author: con...@google.com
Date: Mon Aug 2 13:26:01 2010
Log: Add a file check to handle edge cases like broken symbolic links when
scanning directories for resources.
Review at http://gwt-code-reviews.appspot.com/711803
Review by: sco...@google.com
Revision: 8456
Author: rj...@google.com
Date: Mon Aug 2 10:40:29 2010
Log: Fixes many compiler warnings in JsonRequestProcessor.
http://code.google.com/p/google-web-toolkit/source/detail?r=8456
Modified:
/trunk/user/src/com/google/gwt/requestfactory/server/JsonRequestProcessor.java
Revision: 8453
Author: con...@google.com
Date: Sat Jul 31 10:16:08 2010
Log: Reintroduces changes from r8440
Review at http://gwt-code-reviews.appspot.com/707804
Review by: unn...@google.com
http://code.google.com/p/google-web-toolkit/source/detail?r=8453
Deleted:
Revision: 8464
Author: b...@google.com
Date: Mon Aug 2 18:09:28 2010
Log: Finish modernizing the DynaTableRF sample to use an event-based
application model.
Patch by: bobv
Review by: rjrjr
Review at http://gwt-code-reviews.appspot.com/733802
LGTM, just a couple of code style nits I saw re-reading the patch.
http://gwt-code-reviews.appspot.com/714801/diff/48001/6003
File user/src/com/google/gwt/resources/client/DataResource.java (right):
http://gwt-code-reviews.appspot.com/714801/diff/48001/6003#newcode35
On Tuesday, August 03, 2010 00:15:14 codesite-nore...@google.com wrote:
Revision: 8452
Author: cromwell...@google.com
Date: Fri Jul 30 17:29:09 2010
Log: Implement ROO-933 and ROO-935. Handle
java.lang.{Byte/Short/Character/Integer/Long/BigDecimal/BigInteger/Date/Flo
at/Double} as return
On 2010/08/02 18:38:29, toms wrote:
On Mon, Aug 2, 2010 at 2:14 PM, mailto:j...@google.com wrote:
http://gwt-code-reviews.appspot.com/719802/diff/1/2
File user/src/com/google/gwt/user/client/ui/DockLayoutPanel.java
(right):
http://gwt-code-reviews.appspot.com/719802/diff/1/2#newcode315
http://gwt-code-reviews.appspot.com/734801/diff/1/2
File user/src/com/google/gwt/junit/JUnitShell.java (right):
http://gwt-code-reviews.appspot.com/734801/diff/1/2#newcode1019
user/src/com/google/gwt/junit/JUnitShell.java:1019: + Try increasing
this timeout using the '-testMethodTimeout number'
committed as r8457
http://gwt-code-reviews.appspot.com/720801/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Reviewers: Ray Ryan,
Description:
Public: Add default validators for the standard constraints.
Include super source version of the Pattern constraint
that does not reference java.util.regex.Pattern
Please review this at http://gwt-code-reviews.appspot.com/735801/show
Affected files:
M
LGTM
http://gwt-code-reviews.appspot.com/708803/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
On 2010/08/03 16:07:07, Nick Chalko wrote:
Found some spelling errors and pushed a new version for review.
http://gwt-code-reviews.appspot.com/735801/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
I don't see a new patch
http://gwt-code-reviews.appspot.com/735801/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
On 2010/08/03 17:03:03, Ray Ryan wrote:
I don't see a new patch
rietveld was waiting for me to enter a online description on its way
now.
http://gwt-code-reviews.appspot.com/735801/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
http://gwt-code-reviews.appspot.com/735801/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
On 2010/08/03 17:04:19, Nick Chalko wrote:
On 2010/08/03 17:03:03, Ray Ryan wrote:
I don't see a new patch
rietveld was waiting for me to enter a online description on its
way now.
Ok new patch uploaded.
http://gwt-code-reviews.appspot.com/735801/show
--
Holy combinatorial explosion!
I assume the stuff that is copyright Google is either by you or by Chris
R?
A few nits, and one real concern about DefaultConstraints.
http://gwt-code-reviews.appspot.com/735801/diff/1/2
File tools/api-checker/config/gwt20_21userApi.conf (right):
Holy combinatorial explosion!
I assume the stuff that is copyright Google is either by you or by Chris
R?
A few nits, and one real concern about DefaultConstraints.
http://gwt-code-reviews.appspot.com/735801/diff/1/2
File tools/api-checker/config/gwt20_21userApi.conf (right):
Reviewers: Ray Ryan,
Description:
De-bounce built-in selection models
Please review this at http://gwt-code-reviews.appspot.com/722802/show
Affected files:
M user/src/com/google/gwt/view/client/DefaultSelectionModel.java
M user/src/com/google/gwt/view/client/MultiSelectionModel.java
M
If I do this:
sm.setSelected(!foo, sm.isSelected());
sm.setSelected(!foo, sm.isSelected());
I'll still see an event. Given your nifty event coalescing, I wouldn't
have to. Should you do it there instead?
Also, I wonder if NoSelectionModel is the exception to the no bounces
please rule.
http://gwt-code-reviews.appspot.com/722802/diff/1/2
File user/src/com/google/gwt/view/client/DefaultSelectionModel.java
(right):
http://gwt-code-reviews.appspot.com/722802/diff/1/2#newcode84
user/src/com/google/gwt/view/client/DefaultSelectionModel.java:84:
scheduleSelectionChangeEvent();
http://gwt-code-reviews.appspot.com/725801/diff/12001/13004
File dev/core/src/com/google/gwt/dev/shell/GWTBridgeImpl.java (right):
http://gwt-code-reviews.appspot.com/725801/diff/12001/13004#newcode26
dev/core/src/com/google/gwt/dev/shell/GWTBridgeImpl.java:26: protected
static
http://gwt-code-reviews.appspot.com/725801/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Reviewers: ,
Description:
When a record has a property (for instance a Long property) set to
'null', the client side parser will throw out an exception instead of
gracefully returning null.
Please review this at http://gwt-code-reviews.appspot.com/712802/show
Affected files:
Reviewers: Ray Ryan,
Description:
1. Problems in assertFirstHeader().
1.1. invalid condition;
1.2. invalid reference on second argument.
2. Use addStatement() instead of addInitStatement(), because same is
done everywhere.
3. Small tweaks.
4. Set of tests, including all bad cases.
Please review
Reviewers: cromwellian,
Description:
Fixes various warnings in RequestFactory, most of them generics
related.
Review by: cromwell...@google.com
Please review this at http://gwt-code-reviews.appspot.com/737801/show
Affected files:
M
LGTM
http://gwt-code-reviews.appspot.com/725801/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Thanks, Bob
http://gwt-code-reviews.appspot.com/714801/diff/48001/6003
File user/src/com/google/gwt/resources/client/DataResource.java (right):
http://gwt-code-reviews.appspot.com/714801/diff/48001/6003#newcode35
user/src/com/google/gwt/resources/client/DataResource.java:35: *
Specifies the
http://gwt-code-reviews.appspot.com/737801/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
Reviewers: Ray Ryan,
Description:
Add login widgets to Expenses and the Mobile versions
Please review this at http://gwt-code-reviews.appspot.com/713802/show
Affected files:
M bikeshed/src/com/google/gwt/sample/expenses/gwt/client/Expenses.java
M
LGTM
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
http://gwt-code-reviews.appspot.com/735801/diff/1/2
File tools/api-checker/config/gwt20_21userApi.conf (right):
http://gwt-code-reviews.appspot.com/735801/diff/1/2#newcode81
tools/api-checker/config/gwt20_21userApi.conf:81:
:user/src/com/google/gwt/validation/client/constraints/*.java\
On
http://gwt-code-reviews.appspot.com/735801/show
--
http://groups.google.com/group/Google-Web-Toolkit-Contributors
http://gwt-code-reviews.appspot.com/735801/diff/1/2
File tools/api-checker/config/gwt20_21userApi.conf (right):
http://gwt-code-reviews.appspot.com/735801/diff/1/2#newcode81
tools/api-checker/config/gwt20_21userApi.conf:81:
:user/src/com/google/gwt/validation/client/constraints/*.java\
Well I
41 matches
Mail list logo