Having IE9 fallback to Safari sounds dangerous to me. If I put a fix in a Safari Impl class, am I supposed to know that the change will also apply to IE9? Normally, I don't test other browsers if I'm working on a permutation specified fix. And just because Showcase runs doesn't mean that all of the IE workarounds have gone away, or that all of the Safari workarounds make sense in IE9.
Our plan of record is to have IE9 fallback to IE8. That way, IE9 will continue to work as it does today. In GWT user code, we will update deferred bindings to fix all GWT bugs, which should cover most of the low level stuff. User code is more likely to be making a view decision, such as using a different look in IE versus Safari. I don't like the idea of using a soft perm by default because it can't be undone by users who are willing to wait out a full compile for smaller code size. I'd be happier if GPE offered a fast compile option where users could specify user agents to compile if they just want to test in web mode. We just got rid of the gecko permutation, so we can replace it with IE9 and be back to where we were. Users can choose to soft perm however they want. And, once IE9 is released, it probably makes more sense to soft perm IE6/IE8 to reduce permutations. Lets get the best possible experience on modern browsers. Certainly we don't want to increase Safari code size to carry IE9! On 2011/03/04 20:56:48, fabiomfv wrote: http://gwt-code-reviews.appspot.com/1369807/diff/1/dev/core/src/com/google/gwt/core/ext/DefaultSelectionProperty.java
File
dev/core/src/com/google/gwt/core/ext/DefaultSelectionProperty.java (right): http://gwt-code-reviews.appspot.com/1369807/diff/1/dev/core/src/com/google/gwt/core/ext/DefaultSelectionProperty.java#newcode89
dev/core/src/com/google/gwt/core/ext/DefaultSelectionProperty.java:89:
On 2011/03/04 19:04:54, jlabanca wrote: > extra spaces
Done.
http://gwt-code-reviews.appspot.com/1369807/diff/1/dev/core/src/com/google/gwt/dev/cfg/ModuleDefSchema.java
File dev/core/src/com/google/gwt/dev/cfg/ModuleDefSchema.java (right):
http://gwt-code-reviews.appspot.com/1369807/diff/1/dev/core/src/com/google/gwt/dev/cfg/ModuleDefSchema.java#newcode163
dev/core/src/com/google/gwt/dev/cfg/ModuleDefSchema.java:163: On 2011/03/04 19:04:54, jlabanca wrote: > spaces
Done.
http://gwt-code-reviews.appspot.com/1369807/diff/1/dev/core/src/com/google/gwt/dev/shell/StandardRebindOracle.java
File dev/core/src/com/google/gwt/dev/shell/StandardRebindOracle.java
(right): http://gwt-code-reviews.appspot.com/1369807/diff/1/dev/core/src/com/google/gwt/dev/shell/StandardRebindOracle.java#newcode137
dev/core/src/com/google/gwt/dev/shell/StandardRebindOracle.java:137: On 2011/03/04 19:04:54, jlabanca wrote: > spaces
Done.
http://gwt-code-reviews.appspot.com/1369807/diff/1/user/src/com/google/gwt/user/rebind/UserAgentPropertyGenerator.java
File
user/src/com/google/gwt/user/rebind/UserAgentPropertyGenerator.java
(right):
http://gwt-code-reviews.appspot.com/1369807/diff/1/user/src/com/google/gwt/user/rebind/UserAgentPropertyGenerator.java#newcode50 user/src/com/google/gwt/user/rebind/UserAgentPropertyGenerator.java:50: // Opera
On 2011/03/04 19:04:54, jlabanca wrote: > Move // Opera comment down a line.
Done.
http://gwt-code-reviews.appspot.com/1369807/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
