Rolling back r7407 to unbreak the build.
git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@7408 8db76d5a-ed1c-0410-87a9-c151d255dfc7
diff --git a/dev/core/src/com/google/gwt/dev/jjs/impl/CodeSplitter.java b/dev/core/src/com/google/gwt/dev/jjs/impl/CodeSplitter.java
index a40f2c7..0095722 100644
--- a/dev/core/src/com/google/gwt/dev/jjs/impl/CodeSplitter.java
+++ b/dev/core/src/com/google/gwt/dev/jjs/impl/CodeSplitter.java
@@ -453,7 +453,6 @@
ControlFlowAnalyzer cfa = new ControlFlowAnalyzer(jprogram);
cfa.setDependencyRecorder(dependencyRecorder);
traverseEntry(jprogram, cfa, 0);
- traverseClassArray(jprogram, cfa);
dependencyRecorder.endDependencyGraph();
return cfa;
@@ -580,28 +579,6 @@
}
/**
- * Any instance method in the magic Array class must be in the initial
- * download. The methods of that class are copied to a separate object the
- * first time class Array is touched, and any methods added later won't be
- * part of the copy.
- */
- private static void traverseClassArray(JProgram jprogram,
- ControlFlowAnalyzer cfa) {
- JDeclaredType typeArray = jprogram.getFromTypeMap("com.google.gwt.lang.Array");
- if (typeArray == null) {
- // It was pruned; nothing to do
- return;
- }
-
- cfa.traverseFromInstantiationOf(typeArray);
- for (JMethod method : typeArray.getMethods()) {
- if (!method.isStatic()) {
- cfa.traverseFrom(method);
- }
- }
- }
-
- /**
* Traverse all code in the program that is reachable via split point
* <code>splitPoint</code>.
*/
diff --git a/dev/core/src/com/google/gwt/dev/jjs/impl/ControlFlowAnalyzer.java b/dev/core/src/com/google/gwt/dev/jjs/impl/ControlFlowAnalyzer.java
index ab86a8c..6abc9de 100644
--- a/dev/core/src/com/google/gwt/dev/jjs/impl/ControlFlowAnalyzer.java
+++ b/dev/core/src/com/google/gwt/dev/jjs/impl/ControlFlowAnalyzer.java
@@ -790,14 +790,6 @@
rescuer.rescue(method);
}
- /**
- * Assume <code>type</code> is instantiated, and find out what else will
- * execute as a result.
- */
- public void traverseFromInstantiationOf(JDeclaredType type) {
- rescuer.rescue(type, true, true);
- }
-
public void traverseFromLeftoversFragmentHasLoaded() {
if (program.entryMethods.size() > 1) {
traverseFrom(program.getIndexedMethod("AsyncFragmentLoader.browserLoaderLeftoversFragmentHasLoaded"));
diff --git a/dev/core/test/com/google/gwt/dev/jjs/impl/CodeSplitterTest.java b/dev/core/test/com/google/gwt/dev/jjs/impl/CodeSplitterTest.java
deleted file mode 100644
index 0720b80..0000000
--- a/dev/core/test/com/google/gwt/dev/jjs/impl/CodeSplitterTest.java
+++ /dev/null
@@ -1,52 +0,0 @@
-/*
- * Copyright 2008 Google Inc.
- *
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not
- * use this file except in compliance with the License. You may obtain a copy of
- * the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
- * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
- * License for the specific language governing permissions and limitations under
- * the License.
- */
-package com.google.gwt.dev.jjs.impl;
-
-import com.google.gwt.core.ext.UnableToCompleteException;
-import com.google.gwt.dev.javac.impl.MockJavaResource;
-import com.google.gwt.dev.jjs.ast.JProgram;
-
-/**
- * Tests class {@link CodeSplitter}.
- */
-public class CodeSplitterTest extends OptimizerTestBase {
- /**
- * Tests that everything in the magic Array class is considered initially
- * live.
- */
- public void testArrayIsInitial() throws UnableToCompleteException {
- sourceOracle.addOrReplace(new MockJavaResource("com.google.gwt.lang.Array") {
- @Override
- protected CharSequence getContent() {
- StringBuffer code = new StringBuffer();
- code.append("package com.google.gwt.lang;\n");
- code.append("public class Array {\n");
- code.append(" private Class type;\n");
- code.append(" public Class getClass() { return type; }\n");
- code.append("}\n");
- return code;
- }
- });
-
- JProgram program = compileSnippet("void", "");
- ControlFlowAnalyzer cfa = CodeSplitter.computeInitiallyLive(program);
-
- assertTrue(cfa.getInstantiatedTypes().contains(
- findType(program, "com.google.gwt.lang.Array")));
- assertTrue(cfa.getLiveFieldsAndMethods().contains(
- findMethod(program, "com.google.gwt.lang.Array::getClass()")));;
- }
-}
diff --git a/dev/core/test/com/google/gwt/dev/jjs/impl/OptimizerTestBase.java b/dev/core/test/com/google/gwt/dev/jjs/impl/OptimizerTestBase.java
index 37b9202..4c437a3 100644
--- a/dev/core/test/com/google/gwt/dev/jjs/impl/OptimizerTestBase.java
+++ b/dev/core/test/com/google/gwt/dev/jjs/impl/OptimizerTestBase.java
@@ -26,8 +26,6 @@
import com.google.gwt.dev.jjs.ast.JField;
import com.google.gwt.dev.jjs.ast.JMethod;
import com.google.gwt.dev.jjs.ast.JProgram;
-import com.google.gwt.dev.jjs.impl.JsniRefLookup.ErrorReporter;
-import com.google.gwt.dev.util.JsniRef;
import com.google.gwt.dev.util.log.AbstractTreeLogger;
import com.google.gwt.dev.util.log.PrintWriterTreeLogger;
@@ -82,24 +80,9 @@
}
/**
- * Find a method, given a JSNI reference to it.
- */
- public static JMethod findMethod(JProgram program, final String methodJsniRef) {
- JsniRef ref = JsniRef.parse(methodJsniRef);
- assertNotNull(ref);
- assertTrue(ref.isMethod());
- return (JMethod) JsniRefLookup.findJsniRefTarget(ref, program,
- new ErrorReporter() {
- public void reportError(String error) {
- // ignore errors and return null
- }
- });
- }
-
- /**
* Finds a type by name. The type name may be short, e.g. <code>"Foo"</code>,
- * or fully-qualified, e.g. <code>"com.google.example.Foo"</code>. If a short
- * name is used, it must be unambiguous.
+ * or fully-qualified, e.g. <code>"com.google.example.Foo"</code>. If a
+ * short name is used, it must be unambiguous.
*/
public static JDeclaredType findType(JProgram program, String typeName) {
JDeclaredType type = program.getFromTypeMap(typeName);
