Changes LG. found a few places where more changes could be made and some comments that seem to indicate some tests we could turn back on now.
http://gwt-code-reviews.appspot.com/1221801/diff/1/35 File user/src/com/google/gwt/rpc/client/impl/SimplePayloadSink.java (right): http://gwt-code-reviews.appspot.com/1221801/diff/1/35#newcode195 user/src/com/google/gwt/rpc/client/impl/SimplePayloadSink.java:195: * In hosted-mode, the field's declaring class is written to the stream to s/hosted-mode/Development Mode http://gwt-code-reviews.appspot.com/1221801/diff/1/40 File user/src/com/google/gwt/safehtml/shared/SafeHtmlHostedModeUtils.java (right): http://gwt-code-reviews.appspot.com/1221801/diff/1/40#newcode25 user/src/com/google/gwt/safehtml/shared/SafeHtmlHostedModeUtils.java:25: * SafeHtml utilities whose implementation differs between hosted and Production s/hosted/Development http://gwt-code-reviews.appspot.com/1221801/diff/1/40#newcode29 user/src/com/google/gwt/safehtml/shared/SafeHtmlHostedModeUtils.java:29: * This class has a super-source peer that provides the web-mode implementation. s/web-mode/Production Mode http://gwt-code-reviews.appspot.com/1221801/diff/1/41 File user/src/com/google/gwt/safehtml/shared/SafeHtmlUtils.java (right): http://gwt-code-reviews.appspot.com/1221801/diff/1/41#newcode71 user/src/com/google/gwt/safehtml/shared/SafeHtmlUtils.java:71: * details). For performance reasons, this check is not performed in prod mode s/prod/Production Mode ? http://gwt-code-reviews.appspot.com/1221801/diff/1/41#newcode76 user/src/com/google/gwt/safehtml/shared/SafeHtmlUtils.java:76: * @throws IllegalArgumentException if not running in prod mode and {...@code s/prod/Production Mode ? http://gwt-code-reviews.appspot.com/1221801/diff/1/52 File user/test/com/google/gwt/core/ext/linker/impl/SelectionScriptLinkerUnitTest.java (right): http://gwt-code-reviews.appspot.com/1221801/diff/1/52#newcode207 user/test/com/google/gwt/core/ext/linker/impl/SelectionScriptLinkerUnitTest.java:207: // hosted Development Mode... although this isn't a javadoc comment http://gwt-code-reviews.appspot.com/1221801/diff/1/54 File user/test/com/google/gwt/dev/jjs/scriptonly/ScriptOnlyTest.java (right): http://gwt-code-reviews.appspot.com/1221801/diff/1/54#newcode60 user/test/com/google/gwt/dev/jjs/scriptonly/ScriptOnlyTest.java:60: // Try something you can't do in web-mode (JRE code) s/web-mode/Production Mode http://gwt-code-reviews.appspot.com/1221801/diff/1/54#newcode63 user/test/com/google/gwt/dev/jjs/scriptonly/ScriptOnlyTest.java:63: // Try something you can't do in web-mode ("server" code) s/web-mode/Production Mode http://gwt-code-reviews.appspot.com/1221801/diff/1/59 File user/test/com/google/gwt/dev/jjs/test/JsStaticEvalTest.java (right): http://gwt-code-reviews.appspot.com/1221801/diff/1/59#newcode24 user/test/com/google/gwt/dev/jjs/test/JsStaticEvalTest.java:24: * Development Mode due to browser inconsistencies; however it should run in web s/web/Production http://gwt-code-reviews.appspot.com/1221801/diff/1/61 File user/test/com/google/gwt/dom/client/StyleTest.java (right): http://gwt-code-reviews.appspot.com/1221801/diff/1/61#newcode110 user/test/com/google/gwt/dom/client/StyleTest.java:110: // TODO: re-enable these cases when we finally drop linux Development Mode. Hey, we can probably re-enable these now! http://gwt-code-reviews.appspot.com/1221801/diff/1/76 File user/test/com/google/gwt/xml/client/XMLTest.java (right): http://gwt-code-reviews.appspot.com/1221801/diff/1/76#newcode328 user/test/com/google/gwt/xml/client/XMLTest.java:328: */ umm, maybe time to re-enable this test too? http://gwt-code-reviews.appspot.com/1221801/show -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
