[gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-18 Thread rjrjr
I had to bring back the deprecated methods for HandlManager. HandlerManager needs to be JRE compatible, so violator is not an option. Duh. The patch is also updated to catch up with the AutoBean migration, and to move the RF test classes to the new event package. All tests pass now.

[gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-18 Thread jlabanca
LGTM http://gwt-code-reviews.appspot.com/1394803/diff/17008/user/test/com/google/web/bindery/requestfactory/gwt/client/RequestFactoryTestBase.java File user/test/com/google/web/bindery/requestfactory/gwt/client/RequestFactoryTestBase.java (right):

[gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-18 Thread rjrjr
http://gwt-code-reviews.appspot.com/1394803/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-18 Thread Ray Ryan
Done, should submit soon. On Mon, Apr 18, 2011 at 11:13 AM, rj...@google.com wrote: http://gwt-code-reviews.appspot.com/1394803/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-18 Thread rjrjr
r10014 http://gwt-code-reviews.appspot.com/1394803/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-15 Thread jlabanca
LGTM http://gwt-code-reviews.appspot.com/1394803/diff/10001/user/src/com/google/gwt/event/shared/UmbrellaException.java File user/src/com/google/gwt/event/shared/UmbrellaException.java (right):

[gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-15 Thread bobv
LGTM http://gwt-code-reviews.appspot.com/1394803/diff/10001/user/src/com/google/gwt/event/shared/HandlerManager.java File user/src/com/google/gwt/event/shared/HandlerManager.java (right):

[gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-14 Thread rjrjr
http://gwt-code-reviews.appspot.com/1394803/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-14 Thread rjrjr
John, thanks for all the feedback. I think this is done. Bob, I've expanded it to make the relocated RequestFactory classes use the new event package, along with DynaTableRF. Could you pay particular attention to RequestFactoryJarExtractor? I'm not sure if I did the right thing there. Even with

Re: [gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-01 Thread Ray Ryan
We want to be able to experiment with non-GWT clients of web services, particularly via RequestFactory. But I have to put emphasis on the word experiment. Non-GWT won't be a supported path soon, if ever. On Thu, Mar 31, 2011 at 8:06 AM, Andrés Testi andres.a.te...@gmail.comwrote: Why bindery

Re: [gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-01 Thread Stephen Haberman
We want to be able to experiment with non-GWT clients of web services, particularly via RequestFactory. But I have to put emphasis on the word experiment. Non-GWT won't be a supported path soon, if ever. Is it worth moving packages at the current time then? You could tease out a non-GWT jar

Re: [gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-01 Thread John LaBanca
I don't think Andrés was asking why they weren't in the gwt package. He's sking why they are in the com.google.web package if they are usable outside of the web domain. It seems like we are moving from a very limited package scope to a slightly less limited package scope. I'm sure you've

Re: [gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-01 Thread Ray Ryan
On Fri, Apr 1, 2011 at 11:16 AM, John LaBanca jlaba...@google.com wrote: On Fri, Apr 1, 2011 at 2:13 PM, Ray Ryan rj...@google.com wrote: On Fri, Apr 1, 2011 at 10:38 AM, John LaBanca jlaba...@google.comwrote: I don't think Andrés was asking why they weren't in the gwt package. He's

Re: [gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-01 Thread Ray Ryan
On Fri, Apr 1, 2011 at 10:38 AM, John LaBanca jlaba...@google.com wrote: I don't think Andrés was asking why they weren't in the gwt package. He's sking why they are in the com.google.web package if they are usable outside of the web domain. It seems like we are moving from a very limited

Re: [gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-01 Thread John LaBanca
On Fri, Apr 1, 2011 at 2:13 PM, Ray Ryan rj...@google.com wrote: On Fri, Apr 1, 2011 at 10:38 AM, John LaBanca jlaba...@google.com wrote: I don't think Andrés was asking why they weren't in the gwt package. He's sking why they are in the com.google.web package if they are usable outside

[gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-01 Thread Andrés Testi
You're reading web to mean HTML. I'm reading it as app that talks to a web service, regardless of what it's written in. I really like the GWT event model and want to use it in Guice applications to raise bussiness rules, dispatch entity lifecycle events, etc.. I think this events API is useful

[gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-04-01 Thread jlabanca
http://gwt-code-reviews.appspot.com/1394803/diff/1/user/src/com/google/gwt/activity/shared/Activity.java File user/src/com/google/gwt/activity/shared/Activity.java (right): http://gwt-code-reviews.appspot.com/1394803/diff/1/user/src/com/google/gwt/activity/shared/Activity.java#newcode19

[gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-03-31 Thread Andrés Testi
Why bindery package is nested in a web package? Are these APIs not available for non web applications? Regards. - Andrés On 31 mar, 01:19, rj...@google.com wrote: Ready for review. John, can you keep me honest on the treatment of com.google.gwt.event.shared, and the choices made in the new

[gwt-contrib] Re: Makes EventBus available outside of the gwt package, in (issue1394803)

2011-03-30 Thread rjrjr
Ready for review. John, can you keep me honest on the treatment of com.google.gwt.event.shared, and the choices made in the new event package? Bob, does this fit what you have in mind for the bindery organization? Note that I've updated Activity and Place to use the new classes, but not