http://gwt-code-reviews.appspot.com/890802/diff/1/2 File user/src/com/google/gwt/requestfactory/client/DefaultRequestTransport.java (right):
http://gwt-code-reviews.appspot.com/890802/diff/1/2#newcode98 user/src/com/google/gwt/requestfactory/client/DefaultRequestTransport.java:98: builder.setHeader("pageurl", Location.getHref()); Set the magic permutation header here? Or is that being set elsewhere? http://gwt-code-reviews.appspot.com/890802/diff/1/4 File user/src/com/google/gwt/requestfactory/shared/HasRequestTransport.java (right): http://gwt-code-reviews.appspot.com/890802/diff/1/4#newcode20 user/src/com/google/gwt/requestfactory/shared/HasRequestTransport.java:20: * the manner in which requests are sent to the backend. Why do it via cast? Why not just make RequestFactory extend it? Or make this part of RF proper? Perhaps add an overload of RequestFactory#init that accepts an RT? http://gwt-code-reviews.appspot.com/890802/diff/1/5 File user/src/com/google/gwt/requestfactory/shared/RequestFactory.java (right): http://gwt-code-reviews.appspot.com/890802/diff/1/5#newcode91 user/src/com/google/gwt/requestfactory/shared/RequestFactory.java:91: * DefaultRequestTransport}. This makes me think that if I call init after calling setTransport I'll clobber myself. http://gwt-code-reviews.appspot.com/890802/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
