Remove unused java package checks from CompilingClassLoader. This is obsolete since we moved the implementation into findClass instead of loadClass. findClass never gets called for bootstrap classes anyway.
Removed comment from SerializableTypeOracleBuilder that tied it to CompilingClassLoader.
Review by: spoon (desk)
git-svn-id: https://google-web-toolkit.googlecode.com/svn/releases/1.6@4417 8db76d5a-ed1c-0410-87a9-c151d255dfc7
diff --git a/dev/core/src/com/google/gwt/dev/shell/CompilingClassLoader.java b/dev/core/src/com/google/gwt/dev/shell/CompilingClassLoader.java
index 777c868..eaa813e 100644
--- a/dev/core/src/com/google/gwt/dev/shell/CompilingClassLoader.java
+++ b/dev/core/src/com/google/gwt/dev/shell/CompilingClassLoader.java
@@ -591,13 +591,6 @@
new NullPointerException());
}
- // Don't mess with anything in the standard Java packages.
- //
- if (isInStandardJavaPackage(className)) {
- // make my superclass load it
- throw new ClassNotFoundException(className);
- }
-
// Check for a bridge class that spans hosted and user space.
if (BRIDGE_CLASS_NAMES.containsKey(className)) {
return BRIDGE_CLASS_NAMES.get(className);
@@ -719,22 +712,6 @@
}
/**
- * NOTE: This logic is also used in SerializableTypeOracleBuilder; don't
- * change this one without changing the other.
- */
- private boolean isInStandardJavaPackage(String className) {
- if (className.startsWith("java.")) {
- return true;
- }
-
- if (className.startsWith("javax.")) {
- return true;
- }
-
- return false;
- }
-
- /**
* For safety, we only allow synthetic classes to be loaded this way. Accepts
* any classes whose names match .+$\d+
* <p>
diff --git a/user/src/com/google/gwt/user/rebind/rpc/SerializableTypeOracleBuilder.java b/user/src/com/google/gwt/user/rebind/rpc/SerializableTypeOracleBuilder.java
index 3bd7a0f..4a13edb 100644
--- a/user/src/com/google/gwt/user/rebind/rpc/SerializableTypeOracleBuilder.java
+++ b/user/src/com/google/gwt/user/rebind/rpc/SerializableTypeOracleBuilder.java
@@ -339,10 +339,6 @@
/**
* Returns <code>true</code> if this type is part of the standard java
* packages.
- *
- * NOTE: This code is copied from
- * {@link com.google.gwt.dev.shell.CompilingClassLoader CompilingClassLoader};
- * don't change one without changing the other.
*/
static boolean isInStandardJavaPackage(String className) {
if (className.startsWith("java.")) {