commit 0d84721e175734a9efc00e0ed6f1d01310a5ea2a
author Goktug Gokdogan <goktug@google.com> Fri Nov 13 17:16:28 2015 -0800
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Sat Nov 14 21:17:05 2015 +0000
tree 5f2cebf499672967131364562dc014c50e1bef24
parent 5d563e5b246a3ae5758f9daf9f7830f1b6bf030d

Rollback method pruning change for new flags.

Recently JMethod.canBeReferencedExternally updated so that
it doesn't rely on exporting flag. Actually that wasn't the
plan with exporting flag. We want to keep everything pruned
if the flag is not set. By this way, it will prevent common
code from effecting apps who doesn't need exporting.
And without exporting native JsTypes and JsFunction still work
which covers the use case of a lot of apps just looking for
simple interation with JavaScript.

Laters exporting flag will be enhanced to take parameters for
fine tuning what is exported.

Change-Id: I2fad8977ed471a75c7cc5348c8ab74d014e6b1a7

dev/core/src/com/google/gwt/dev/jjs/ast/JMethod.java

diff --git a/dev/core/src/com/google/gwt/dev/jjs/ast/JMethod.java b/dev/core/src/com/google/gwt/dev/jjs/ast/JMethod.java
index 26382d9..6d8cf25 100644
--- a/dev/core/src/com/google/gwt/dev/jjs/ast/JMethod.java
+++ b/dev/core/src/com/google/gwt/dev/jjs/ast/JMethod.java
@@ -76,16 +76,11 @@
 
   @Override
   public boolean canBeReferencedExternally() {
-    if (getJsMemberType() != JsMemberType.NONE) {
+    if (exported || isJsFunctionMethod()) {
       return true;
     }
-
-    if (isJsFunctionMethod()) {
-      return true;
-    }
-
     for (JMethod overriddenMethod : getOverriddenMethods()) {
-      if (overriddenMethod.isJsFunctionMethod()) {
+      if (overriddenMethod.exported || overriddenMethod.isJsFunctionMethod()) {
         return true;
       }
     }