http://gwt-code-reviews.appspot.com/1385801/diff/1/dev/core/src/com/google/gwt/core/ext/linker/impl/HostedModeTemplate.js
File
dev/core/src/com/google/gwt/core/ext/linker/impl/HostedModeTemplate.js
(right):

http://gwt-code-reviews.appspot.com/1385801/diff/1/dev/core/src/com/google/gwt/core/ext/linker/impl/HostedModeTemplate.js#newcode358
dev/core/src/com/google/gwt/core/ext/linker/impl/HostedModeTemplate.js:358:
var url = computeUrlForResource("__MODULE_NAME__.nocache.js");
I don't think you want to touch this file - I don't recognize it, so I
think it's for one of the old linkers, which won't have
computeUrlForResource defined (note how it uses the 'base' variable,
which is part of the legacy linkers).

http://gwt-code-reviews.appspot.com/1385801/diff/1/dev/core/src/com/google/gwt/core/ext/linker/impl/loadExternalStylesheets.js
File
dev/core/src/com/google/gwt/core/ext/linker/impl/loadExternalStylesheets.js
(right):

http://gwt-code-reviews.appspot.com/1385801/diff/1/dev/core/src/com/google/gwt/core/ext/linker/impl/loadExternalStylesheets.js#newcode13
dev/core/src/com/google/gwt/core/ext/linker/impl/loadExternalStylesheets.js:13:
l.setAttribute('href', computeUrlForResource(stylesheetUrl));
I think you should leave this file as is, and instead change
ResourceInjectionUtil - line 85.  Otherwise we calculate the URL in 2
places.

http://gwt-code-reviews.appspot.com/1385801/diff/1/dev/core/src/com/google/gwt/core/ext/linker/impl/permutations.js
File dev/core/src/com/google/gwt/core/ext/linker/impl/permutations.js
(right):

http://gwt-code-reviews.appspot.com/1385801/diff/1/dev/core/src/com/google/gwt/core/ext/linker/impl/permutations.js#newcode45
dev/core/src/com/google/gwt/core/ext/linker/impl/permutations.js:45:
return "__HOSTED_FILENAME__";
On 2011/03/11 16:53:17, zundel wrote:
these 2 changes make me a bit queasy. They change the API for
getCompiledCodeFilename().. but if I don't leave these resources
un-based, then
we lose flexibility in the computeUrlForResource() method.


I'm confused - why don't you just return
computeUrlForResource("__HOSTED_FILENAME__") here?

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

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

Reply via email to