[gwt-contrib] Re: bug in permutations.js?

2014-02-14 Thread Stephen Haberman
Here is the difference in the generated JS. From a ~10/2013 trunk build: function com_google_gwt_dom_client_StyleInjector_StyleInjectorImpl(){ switch (permutationId) { case 1: case 2: case 3: return new StyleInjector$StyleInjectorImplIE_0; } return new

[gwt-contrib] Re: bug in permutations.js?

2014-02-13 Thread Colin Alworth
In CrossSiteIframeTemplate.js this is handled by assigning __MODULE_FUNC__.__softPermutationId to 0 to begin with, and then only change that value if : was present in the permutation string. I'm not seeing any other js files that init __softPermutationId to 0, and only permutations.js assigns

Re: [gwt-contrib] Re: bug in permutations.js?

2014-02-13 Thread Stephen Haberman
and only permutations.js assigns it to another value (but does so conditionally, so it never *assigns* undefined to __softPermutationId, it just leaves it not defined). But line 61: https://gwt.googlesource.com/gwt/+/master/dev/core/src/com/google/gwt/core/ext/linker/impl/permutations.js