[Android] Add JavaBridgeBasicsTest#testReplaceJavascriptInterface

content/public/android/javatests/src/org/chromium/content/browser/JavaBridgeBasicsTest.java

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.content.browser;
 
 import android.test.suitebuilder.annotation.SmallTest;
 
 import org.chromium.base.test.util.Feature;
 import org.chromium.content.browser.test.util.TestCallbackHelperContainer;
@@ skipping 120 matching lines @@
             @Override
             public void run() {
                 getContentViewCore().getWebContents().getNavigationController().reload(true);
             }
         });
         onPageFinishedHelper.waitForCallback(currentCallCount);
     }
 
     // Note that this requires that we can pass a JavaScript boolean to Java.
     private void assertRaisesException(String script) throws Throwable {
-        executeJavaScript("try {" +
-                          script + ";" +
-                          "  testController.setBooleanValue(false);" +
-                          "} catch (exception) {" +
-                          "  testController.setBooleanValue(true);" +
-                          "}");
+        executeJavaScript("try {"
+                + script + ";"
+                + "  testController.setBooleanValue(false);"
+                + "} catch (exception) {"
+                + "  testController.setBooleanValue(true);"
+                + "}");
         assertTrue(mTestController.waitForBooleanValue());
     }
 
     @SmallTest
     @Feature({"AndroidWebView", "Android-JavaBridge"})
     public void testTypeOfInjectedObject() throws Throwable {
         assertEquals("object", executeJavaScriptAndGetStringResult("typeof testController"));
     }
 
     @SmallTest
@@ skipping 297 matching lines @@
                 return inner;
             }
             // A weak reference is used to check InnerObject instance reachability.
             WeakReference<InnerObject> mWeakRefForInner;
         }
         TestObject object = new TestObject();
         injectObjectAndReload(object, "testObject");
         // Initially, store a reference to the inner object in JS to make sure it's not
         // garbage-collected prematurely.
         assertEquals("object", executeJavaScriptAndGetStringResult(
-                        "(function() { " +
-                        "globalInner = testObject.getInnerObject(); return typeof globalInner; " +
-                        "})()"));
+                        "(function() { "
+                        + "globalInner = testObject.getInnerObject(); return typeof globalInner; "
+                        + "})()"));
         assertTrue(object.mWeakRefForInner.get() != null);
         // Check that returned Java object is being held by the Java bridge, thus it's not
         // collected.  Note that despite that what JavaDoc says about invoking "gc()", both Dalvik
         // and ART actually run the collector.
         Runtime.getRuntime().gc();
         assertTrue(object.mWeakRefForInner.get() != null);
         // Now dereference the inner object in JS and run GC to collect the interface object.
         assertEquals("true", executeJavaScriptAndGetStringResult(
-                        "(function() { " +
-                        "delete globalInner; gc(); return (typeof globalInner == 'undefined'); " +
-                        "})()"));
+                        "(function() { "
+                        + "delete globalInner; gc(); return (typeof globalInner == 'undefined'); "
+                        + "})()"));
         // Force GC on the Java side again. The bridge had to release the inner object, so it must
         // be collected this time.
         Runtime.getRuntime().gc();
         assertEquals(null, object.mWeakRefForInner.get());
     }
 
     @SmallTest
     @Feature({"AndroidWebView", "Android-JavaBridge"})
     public void testSameReturnedObjectUsesSameWrapper() throws Throwable {
         class InnerObject {
@@ skipping 81 matching lines @@
     @SmallTest
     @Feature({"AndroidWebView", "Android-JavaBridge"})
     public void testEnumerateMembers() throws Throwable {
         injectObjectAndReload(new Object() {
             public void method() {}
             private void privateMethod() {}
             public int field;
             private int mPrivateField;
         }, "testObject");
         executeJavaScript(
-                "var result = \"\"; " +
-                "for (x in testObject) { result += \" \" + x } " +
-                "testController.setStringValue(result);");
+                "var result = \"\"; "
+                + "for (x in testObject) { result += \" \" + x } "
+                + "testController.setStringValue(result);");
         assertEquals(" equals getClass hashCode method notify notifyAll toString wait",
                 mTestController.waitForStringValue());
     }
 
     @SmallTest
     @Feature({"AndroidWebView", "Android-JavaBridge"})
     public void testReflectPublicMethod() throws Throwable {
         injectObjectAndReload(new Object() {
             public Class<?> myGetClass() {
                 return getClass();
             }
 
             public String method() {
                 return "foo";
             }
         }, "testObject");
         assertEquals("foo", executeJavaScriptAndGetStringResult(
-                "testObject.myGetClass().getMethod('method', null).invoke(testObject, null)" +
-                ".toString()"));
+                "testObject.myGetClass().getMethod('method', null).invoke(testObject, null)"
+                + ".toString()"));
     }
 
     @SmallTest
     @Feature({"AndroidWebView", "Android-JavaBridge"})
     public void testReflectPublicField() throws Throwable {
         injectObjectAndReload(new Object() {
             public Class<?> myGetClass() {
                 return getClass();
             }
 
@@ skipping 10 matching lines @@
             public Class<?> myGetClass() {
                 return getClass();
             }
 
             private void method() {};
         }, "testObject");
         assertRaisesException("testObject.myGetClass().getMethod('method', null)");
         // getDeclaredMethod() is able to access a private method, but invoke()
         // throws a Java exception.
         assertRaisesException(
-                "testObject.myGetClass().getDeclaredMethod('method', null)." +
-                "invoke(testObject, null)");
+                "testObject.myGetClass().getDeclaredMethod('method', null)."
+                + "invoke(testObject, null)");
     }
 
     @SmallTest
     @Feature({"AndroidWebView", "Android-JavaBridge"})
     public void testReflectPrivateFieldRaisesException() throws Throwable {
         injectObjectAndReload(new Object() {
             public Class<?> myGetClass() {
                 return getClass();
             }
 
@@ skipping 223 matching lines @@
             }
 
             @JavascriptInterface
             public String m2(int x) {
                 return "bar " + x;
             }
         }
 
         final String jsObjectKeysTestTemplate = "Object.keys(%s).toString()";
         final String jsForInTestTemplate =
-                "(function(){" +
-                "  var s=[]; for(var m in %s) s.push(m); return s.join(\",\")" +
-                "})()";
+                "(function(){"
+                + "  var s=[]; for(var m in %s) s.push(m); return s.join(\",\")"
+                + "})()";
         final String inspectableObjectName = "testObj1";
         final String nonInspectableObjectName = "testObj2";
 
         // Inspection is enabled by default.
         injectObjectAndReload(new Test(), inspectableObjectName, JavascriptInterface.class);
 
         assertEquals("m1,m2", executeJavaScriptAndGetStringResult(
                         String.format(jsObjectKeysTestTemplate, inspectableObjectName)));
         assertEquals("m1,m2", executeJavaScriptAndGetStringResult(
                         String.format(jsForInTestTemplate, inspectableObjectName)));
@@ skipping 13 matching lines @@
                         String.format(jsForInTestTemplate, nonInspectableObjectName)));
     }
 
     @SmallTest
     @Feature({"AndroidWebView", "Android-JavaBridge"})
     public void testAccessToObjectGetClassIsBlocked() throws Throwable {
         injectObjectAndReload(new Object(), "testObject");
         assertEquals("function", executeJavaScriptAndGetStringResult("typeof testObject.getClass"));
         assertRaisesException("testObject.getClass()");
     }
+
+    @SmallTest
+    @Feature({"AndroidWebView", "Android-JavaBridge"})
+    public void testReplaceJavascriptInterface() throws Throwable {
+        class Test {
+            public Test(int value) {
+                mValue = value;
+            }
+            @JavascriptInterface
+            public int getValue() {
+                return mValue;
+            }
+            private int mValue;
+        }
+        injectObjectAndReload(new Test(13), "testObject");
+        assertEquals("13", executeJavaScriptAndGetStringResult("testObject.getValue()"));
+        // The documentation doesn't specify, what happens if the embedder is trying
+        // to inject a different object under the same name. The current implementation
+        // simply replaces the old object with the new one.
+        injectObjectAndReload(new Test(42), "testObject");
+        assertEquals("42", executeJavaScriptAndGetStringResult("testObject.getValue()"));
+    }
 }