[gwt-contrib] Re: Code Review Request: Addition of the DevModeService Server

2009-10-20 Thread rdayal
On 2009/10/20 17:00:27, mmendez wrote: > LGTM Thanks, committed as r6424. http://gwt-code-reviews.appspot.com/80805 --~--~-~--~~~---~--~~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~--~~~~--~~--~--~---

[gwt-contrib] Re: Code Review Request: Addition of the DevModeService Server

2009-10-20 Thread mmendez
LGTM http://gwt-code-reviews.appspot.com/80805 --~--~-~--~~~---~--~~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~--~~~~--~~--~--~---

[gwt-contrib] Re: Code Review Request: Addition of the DevModeService Server

2009-10-20 Thread rdayal
Added a new patch set. Converts the DevModeServiceServer to a DevModeServiceRequestProcessor (which interacts with the updated transport API). http://gwt-code-reviews.appspot.com/80805 --~--~-~--~~~---~--~~ http://groups.google.com/group/Google-Web-Toolkit-Contr

[gwt-contrib] Re: Code Review Request: Addition of the DevModeService Server

2009-10-18 Thread mmendez
http://gwt-code-reviews.appspot.com/80805/diff/1/2 File dev/core/src/com/google/gwt/dev/shell/remoteui/DevModeServiceServer.java (right): http://gwt-code-reviews.appspot.com/80805/diff/1/2#newcode55 Line 55: public void handleRequest(Request request) { This queues a request, it doesn't necessari