[gwt-contrib] Status of JavaScript InterOp library (JSIO) project

2009-10-26 Thread Hilbrand
I was wondering what the status is of the JavaScript InterOp library (JSIO) project: http://code.google.com/p/gwt-api-interop/. It does look interesting and it's also referred to from the gwt-google-apis library. But it seems abandoned; the last commit was Oct 1, 2008, it's tagged Experimental, an

[gwt-contrib] ServletValidator, -noserver and logs

2009-10-26 Thread Sami Jaber
Hi, I'm experiencing a few problems with the way ServletValidator works. My webapp is very common, I have a war providing a web.xml. This web.xml contains a Forms login authentication following this approach : Protected Page /* * FORM myRealm /TestSample.jsp /error.jsp When I la

[gwt-contrib] Re: Status of JavaScript InterOp library (JSIO) project

2009-10-26 Thread BobV
It's pretty much defunct at this point. The functionality of that project can be replicated using GWT's built-in JavaScriptObject overlay types and/or the GWT-exporter project. -- Bob Vawter Google Web Toolkit Team --~--~-~--~~~---~--~~ http://groups.google.com/

[gwt-contrib] Re: Status of JavaScript InterOp library (JSIO) project

2009-10-26 Thread Hilbrand
ok. thanks for the update. On Oct 26, 3:03 pm, BobV wrote: > It's pretty much defunct at this point.  The functionality of that > project can be replicated using GWT's built-in JavaScriptObject > overlay types and/or the GWT-exporter project. > > -- > Bob Vawter > Google Web Toolkit Team --~--~-

[gwt-contrib] Re: Fix sundry warnings

2009-10-26 Thread rice
http://gwt-code-reviews.appspot.com/86801/diff/1/11 File dev/core/src/com/google/gwt/dev/shell/CompilingClassLoader.java (left): http://gwt-code-reviews.appspot.com/86801/diff/1/11#oldcode558 Line 558: + intfMethod.toString(); On 2009/10/24 01:08:05, Ray Ryan wrote: > Why is it okay to delete th

[gwt-contrib] Re: Fix sundry warnings

2009-10-26 Thread rjrjr
http://gwt-code-reviews.appspot.com/86801/diff/1/26 File user/src/com/google/gwt/core/client/GWT.java (right): http://gwt-code-reviews.appspot.com/86801/diff/1/26#newcode212 Line 212: @SuppressWarnings("unused") // name parameter will be used following replacement No, 80. One of many arbitrary d

[gwt-contrib] Re: Status of JavaScript InterOp library (JSIO) project

2009-10-26 Thread Ray Ryan
Can a project owner edit its home page to say so? --~--~-~--~~~---~--~~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~--~~~~--~~--~--~---

[gwt-contrib] [google-web-toolkit] r6463 committed - Recommitting r6449 (Adds basic test case for SOYC) with fix. Compiled ...

2009-10-26 Thread codesite-noreply
Revision: 6463 Author: jlaba...@google.com Date: Mon Oct 26 09:31:34 2009 Log: Recommitting r6449 (Adds basic test case for SOYC) with fix. Compiled user files are now added to javac.junit.out instead of javac.out to prevent rebuilding of dev. Patch by: kprobst Review by: jlabanca (pair progr

[gwt-contrib] Re: ServletValidator, -noserver and logs

2009-10-26 Thread Scott Blum
On Mon, Oct 26, 2009 at 7:21 AM, Sami Jaber wrote: > So, my questions are : > 1) Why is it required to validate web.xml with jetty validator when we run > in -noserver mode ? > Looks like either an oversight, or else we left it in thinking it'd be net more helpful than harmful. > 2) Why JettyN

[gwt-contrib] Re: Support more structured GWT versions for 2.0+

2009-10-26 Thread rjrjr
http://gwt-code-reviews.appspot.com/72802/diff/2001/3006 File dev/core/src/com/google/gwt/dev/GwtVersion.java (right): http://gwt-code-reviews.appspot.com/72802/diff/2001/3006#newcode30 Line 30: implements Comparable { What's the value of the ReleaseType enum? Where do we ever actually check for

[gwt-contrib] [google-web-toolkit] r6464 committed - Exluding BrowserManagerTest from the dev tests. It is brought when we...

2009-10-26 Thread codesite-noreply
Revision: 6464 Author: jlaba...@google.com Date: Mon Oct 26 10:25:56 2009 Log: Exluding BrowserManagerTest from the dev tests. It is brought when we compile the user files (we need to compile hello sample for dev tests) because BrowserManagerTest is actually in user/src, not user/test. Patch

[gwt-contrib] Re: Fix sundry warnings

2009-10-26 Thread rjrjr
LGTM But, hey, other reviewers: ping http://gwt-code-reviews.appspot.com/86801 --~--~-~--~~~---~--~~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~--~~~~--~~--~--~---

[gwt-contrib] Move Plural Forms example to Showcase

2009-10-26 Thread jlabanca
Reviewers: jat, Description: Moving the Plural Forms example to Showcase so we can remote the I18N sample. Showcase and I18N currently have a lot of overlap. I will remove the I18N sample in another patch. Please review this at http://gwt-code-reviews.appspot.com/86804 Affected files: samp

[gwt-contrib] Re: Support more structured GWT versions for 2.0+

2009-10-26 Thread jat
Thanks for the review. http://gwt-code-reviews.appspot.com/72802/diff/2001/3006 File dev/core/src/com/google/gwt/dev/GwtVersion.java (right): http://gwt-code-reviews.appspot.com/72802/diff/2001/3006#newcode30 Line 30: implements Comparable { On 2009/10/26 17:01:01, Ray Ryan wrote: > What's the

[gwt-contrib] Remove I18N sample

2009-10-26 Thread jlabanca
Reviewers: jat, Description: The I18N sample has been rolled into Showcase. See: http://gwt-code-reviews.appspot.com/86804/show Now we can delete it to avoid having to maintain two sets of code. Please review this at http://gwt-code-reviews.appspot.com/87807 Affected files: eclipse/samples

[gwt-contrib] Re: Move Plural Forms example to Showcase

2009-10-26 Thread jat
Mostly LGTM, with only wording/translation issues. http://gwt-code-reviews.appspot.com/86804/diff/1/7 File samples/showcase/src/com/google/gwt/sample/showcase/client/ShowcaseConstants.properties (right): http://gwt-code-reviews.appspot.com/86804/diff/1/7#newcode187 Line 187: cwPluralFormsExampl

[gwt-contrib] Re: Fix sundry warnings

2009-10-26 Thread scottb
I was thinking we didn't necessarily need 4 people to review all the same (mostly trivial) changes, so I synced up with Dan at his desk on the few remaining items he had questions about. Do we really need more review at this point? If it LGTY, it LGTM. http://gwt-code-reviews.appspot.com/86801

[gwt-contrib] [google-web-toolkit] r6465 committed - Create change branch for Sam's single-XPI patch.

2009-10-26 Thread codesite-noreply
Revision: 6465 Author: j...@google.com Date: Mon Oct 26 11:45:11 2009 Log: Create change branch for Sam's single-XPI patch. http://code.google.com/p/google-web-toolkit/source/detail?r=6465 Added: /changes/jat/single-xpi --~--~-~--~~~---~--~~ http://groups.goo

[gwt-contrib] Re: Fix sundry warnings

2009-10-26 Thread Ray Ryan
LGTM! On Mon, Oct 26, 2009 at 11:45 AM, wrote: > I was thinking we didn't necessarily need 4 people to review all the > same (mostly trivial) changes, so I synced up with Dan at his desk on > the few remaining items he had questions about. Do we really need more > review at this point? If it L

[gwt-contrib] [google-web-toolkit] r6466 committed - Checkin first cut of polishing off Sam's single-XPI work. I haven't f...

2009-10-26 Thread codesite-noreply
Revision: 6466 Author: j...@google.com Date: Mon Oct 26 11:51:28 2009 Log: Checkin first cut of polishing off Sam's single-XPI work. I haven't fixed the MSVC project files, and I haven't rebuilt all the plugins yet (many of them are just moving around previously compiled versions). http://code

[gwt-contrib] [google-web-toolkit] r6467 committed - Back port of FF3.5 disabling for HTTPSuite test to GWT 1.7. Original p...

2009-10-26 Thread codesite-noreply
Revision: 6467 Author: zun...@google.com Date: Mon Oct 26 12:41:12 2009 Log: Back port of FF3.5 disabling for HTTPSuite test to GWT 1.7. Original patch is at r6249 in trunk. Review by: jlabanca http://code.google.com/p/google-web-toolkit/source/detail?r=6467 Modified: /releases/1.7/user/tes

[gwt-contrib] Re: Fix for issue 3374 (URL.encodeComponent limited to query-string uses)

2009-10-26 Thread rjrjr
Thank you for this, and for your incredible patience. I'll submit it this afternoon. One JavaDoc nit below, which I'll address. Please say something if I'm confused there. http://gwt-code-reviews.appspot.com/87806/diff/1/2 File user/src/com/google/gwt/http/client/URL.java (right): http://gwt-c

[gwt-contrib] Re: Fix for issue 3374 (URL.encodeComponent limited to query-string uses)

2009-10-26 Thread Ray Ryan
Oh, I did get the javadoc wrong, didn't I? I'll leave it alone. On Mon, Oct 26, 2009 at 1:30 PM, wrote: > Thank you for this, and for your incredible patience. I'll submit it > this afternoon. > > One JavaDoc nit below, which I'll address. Please say something if I'm > confused there. > > > http

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

2009-10-26 Thread codesite-noreply
Revision: 6469 Author: tamplinjohn Date: Mon Oct 26 14:11:20 2009 Log: Edited wiki page through web user interface. http://code.google.com/p/google-web-toolkit/source/detail?r=6469 Modified: /wiki/DesignOOPHM.wiki === --- /wiki/DesignOOPHM.wiki Tue May

[gwt-contrib] [google-web-toolkit] r6470 committed - Fix for issue 3374. This patch adds overloads of encodeComponent and...

2009-10-26 Thread codesite-noreply
Revision: 6470 Author: rj...@google.com Date: Mon Oct 26 15:00:20 2009 Log: Fix for issue 3374. This patch adds overloads of encodeComponent and decodeComponent to deal differently with spaces (keeping them as %20 rather than replacing them with +) and therefore allow their use for path segments (

[gwt-contrib] [google-web-toolkit] r6471 committed - Merging tr...@6462:6464 trunk....

2009-10-26 Thread codesite-noreply
Revision: 6471 Author: jlaba...@google.com Date: Mon Oct 26 16:09:24 2009 Log: Merging tr...@6462:6464 trunk. - Adds basic test case for SOYC. svn merge --ignore-ancestry -r6462:6464 https://google-web-toolkit.googlecode.com/svn/trunk . http://code.google.com/p/google-web-toolkit/source/detai

[gwt-contrib] [google-web-toolkit] r6472 committed - Patch updates the collection of tests that are ignored on HtmlUnit, to...

2009-10-26 Thread codesite-noreply
Revision: 6472 Author: amitman...@google.com Date: Mon Oct 26 16:46:02 2009 Log: Patch updates the collection of tests that are ignored on HtmlUnit, to reflect HtmlUnit being updated from 2.5 to a 2.7-snapshot. The occurrence count for the annotation is down from 95 to 64. Patch by: flin Revi

[gwt-contrib] [google-web-toolkit] r6473 committed - Add nsIXULRuntime.h for FF2/FF3.

2009-10-26 Thread codesite-noreply
Revision: 6473 Author: j...@google.com Date: Mon Oct 26 17:16:39 2009 Log: Add nsIXULRuntime.h for FF2/FF3. http://code.google.com/p/google-web-toolkit/source/detail?r=6473 Added: /plugin-sdks/gecko-sdks/gecko-1.8/include/nsIXULRuntime.h /plugin-sdks/gecko-sdks/gecko-1.9.0/include/nsIXULRunt

[gwt-contrib] [google-web-toolkit] r6474 committed - Update Linux plugins for single XPI.

2009-10-26 Thread codesite-noreply
Revision: 6474 Author: j...@google.com Date: Mon Oct 26 17:20:07 2009 Log: Update Linux plugins for single XPI. http://code.google.com/p/google-web-toolkit/source/detail?r=6474 Added: /changes/jat/single-xpi/plugins/xpcom/prebuilt/gwt-dmp.xpi Deleted: /changes/jat/single-xpi/plugins/xpcom/pr

[gwt-contrib] [google-web-toolkit] r6475 committed - Update OSX libraries....

2009-10-26 Thread codesite-noreply
Revision: 6475 Author: j...@google.com Date: Mon Oct 26 18:06:40 2009 Log: Update OSX libraries. Makefile changes need more work. http://code.google.com/p/google-web-toolkit/source/detail?r=6475 Modified: /changes/jat/single-xpi/plugins/xpcom/Makefile /changes/jat/single-xpi/plugins/xpco

[gwt-contrib] [google-web-toolkit] r6477 committed - tr...@6470 was merged into this branch...

2009-10-26 Thread codesite-noreply
Revision: 6477 Author: rj...@google.com Date: Mon Oct 26 20:26:34 2009 Log: tr...@6470 was merged into this branch Fix for issue 3374, URL.encodeComponent replaces %20's with +'s svn merge --ignore-ancestry -c 6470 https://google-web-toolkit.googlecode.com/svn/trunk . http://code.googl

[gwt-contrib] [google-web-toolkit] r6479 committed - This allows multiple source files to be specified for a ui:style, just...

2009-10-26 Thread codesite-noreply
Revision: 6479 Author: rj...@google.com Date: Mon Oct 26 23:33:41 2009 Log: This allows multiple source files to be specified for a ui:style, just like any other CssResource. It also allows you to put body text on a ui:style that has sources. The body becomes just one more source file, the last o