LGTM

On 2011/04/15 21:25:03, zundel wrote:
Obviously, my testing before uploading the previous patch left
something to be
desired.  I uploaded some of the compiled samples to app engine and
looked at
them this time and made sure I wasn't looking at cached resources.

Sorry, the diff got messed up switching between home and work working
on this.


http://gwt-code-reviews.appspot.com/1370808/diff/14001/gadgets/src/com/google/gwt/gadgets/linker/computeUrlForGadgetResource.js
File
gadgets/src/com/google/gwt/gadgets/linker/computeUrlForGadgetResource.js
(right):


http://gwt-code-reviews.appspot.com/1370808/diff/14001/gadgets/src/com/google/gwt/gadgets/linker/computeUrlForGadgetResource.js#newcode31

gadgets/src/com/google/gwt/gadgets/linker/computeUrlForGadgetResource.js:31:
return __MODULE_FUNC__.__moduleBase + resource;
On 2011/04/06 21:51:50, unnurg wrote:
> How is this file different than the computeUrlForResource.js that
the linker
> uses?

Umm, that isn't the file I intended to upload.



http://gwt-code-reviews.appspot.com/1370808/

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to