commit 55b0628763232beafb9dd0687d37be02c6e1efff
author Ray Cromwell <cromwellian@google.com> Mon Feb 23 14:36:41 2015 -0800
committer Ray Cromwell <cromwellian@google.com> Mon Feb 23 14:36:54 2015 -0800
tree 4bf966c62177db81c8a3336bc78fd12314ae79ec
parent abaf42b86871de6a2b9754f6ac13d1c9f886df1f

Fix bug in -XclosureFormattedOutput

WHen -XjsInteropMode CLOSURE was removed, the if statement
ended up with a bug where the else clause is never taken

Change-Id: I3ca2460d4a96c1b0f9015375ef956e748d55decf

dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaScriptAST.java

diff --git a/dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaScriptAST.java b/dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaScriptAST.java
index 7148b4e..e845c4f 100644
--- a/dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaScriptAST.java
+++ b/dev/core/src/com/google/gwt/dev/jjs/impl/GenerateJavaScriptAST.java
@@ -2846,7 +2846,7 @@
     private String exportProvidedNamespace(JDeclaredType x, List<JsStatement> globalStmts,
         String lastProvidedNamespace, Pair<String, String> exportNamespacePair) {
       if (!exportNamespacePair.getLeft().equals(lastProvidedNamespace)) {
-        if (jsInteropMode == OptionJsInteropMode.Mode.JS) {
+        if (!jsExportClosureStyle) {
           JsName provideFunc = indexedFunctions.get("JavaClassHierarchySetupUtil.provide").getName();
           JsNameRef provideFuncRef = provideFunc.makeRef(x.getSourceInfo());
           JsInvocation provideCall = new JsInvocation(x.getSourceInfo());
@@ -2859,7 +2859,7 @@
           JsExprStmt provideStat = createAssignment(globalTemp.makeRef(x.getSourceInfo()),
               provideCall).makeStmt();
           globalStmts.add(provideStat);
-        } else if (jsInteropMode == OptionJsInteropMode.Mode.JS && jsExportClosureStyle) {
+        } else {
           // goog.provide statements prepended by linker, so namespace already exists
           // but enclosing constructor exports may have overwritten them
           // so write foo.bar.Baz = foo.bar.Baz || {}