Android: Deletes super_onConfigurationChanged delegate

The call order of |super.onConfigurationChanged| doesn't seem to matter. This
CL deletes the interface method from ContentViewCore.InternalAccessDelegate
and let the embedders call it directly.

Bug: 598880
Change-Id: If0ef9eeb583036846130accbbdd0e71b19ac48b6
Reviewed-on: https://chromium-review.googlesource.com/1053327Reviewed-by: Bo <boliu@chromium.org>
Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#557436}

android_webview/java/src/org/chromium/android_webview/AwContents.java

@@ -203,6 +203,11 @@ public class AwContents implements SmartClipProvider {
          * @see View#startActivityForResult(Intent, int)
          */
         void super_startActivityForResult(Intent intent, int requestCode);
+
+        /**
+         * @see View#onConfigurationChanged(Configuration)
+         */
+        void super_onConfigurationChanged(Configuration newConfig);
     }
 
     /**
@@ -3432,6 +3437,7 @@ public class AwContents implements SmartClipProvider {
         public void onConfigurationChanged(Configuration newConfig) {
             if (!isDestroyedOrNoOperation(NO_WARN)) {
                 mContentViewCore.onConfigurationChanged(newConfig);
+                mInternalAccessAdapter.super_onConfigurationChanged(newConfig);
             }
         }
 

components/content_view/java/src/org/chromium/components/content_view/ContentView.java

@@ -230,6 +230,7 @@ public class ContentView
     protected void onConfigurationChanged(Configuration newConfig) {
         ContentViewCore cvc = getContentViewCore();
         if (cvc != null) cvc.onConfigurationChanged(newConfig);
+        super.onConfigurationChanged(newConfig);
     }
 
     /**
@@ -340,11 +341,6 @@ public class ContentView
         return super.onGenericMotionEvent(event);
     }
 
-    @Override
-    public void super_onConfigurationChanged(Configuration newConfig) {
-        super.onConfigurationChanged(newConfig);
-    }
-
     ///////////////////////////////////////////////////////////////////////////////////////////////
     //                End Implementation of ContentViewCore.InternalAccessDelegate               //
     ///////////////////////////////////////////////////////////////////////////////////////////////

content/public/android/java/src/org/chromium/content/browser/ContentViewCoreImpl.java

@@ -199,7 +199,6 @@ public class ContentViewCoreImpl implements ContentViewCore {
         try {
             TraceEvent.begin("ContentViewCore.onConfigurationChanged");
             getImeAdapter().onKeyboardConfigurationChanged(newConfig);
-            mContainerViewInternals.super_onConfigurationChanged(newConfig);
             // To request layout has side effect, but it seems OK as it only happen in
             // onConfigurationChange and layout has to be changed in most case.
             ViewAndroidDelegate viewDelegate = mWebContents.getViewAndroidDelegate();

content/public/android/java/src/org/chromium/content_public/browser/ContentViewCore.java

@@ -69,11 +69,6 @@ public interface ContentViewCore {
          */
         boolean super_onGenericMotionEvent(MotionEvent event);
 
-        /**
-         * @see View#onConfigurationChanged(Configuration)
-         */
-        void super_onConfigurationChanged(Configuration newConfig);
-
         /**
          * @see View#onScrollChanged(int, int, int, int)
          */

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

@@ -4,7 +4,6 @@
 
 package org.chromium.content.browser;
 
-import android.content.res.Configuration;
 import android.os.SystemClock;
 import android.support.test.InstrumentationRegistry;
 import android.support.test.filters.SmallTest;
@@ -67,10 +66,6 @@ public class ContentViewScrollingTest {
             return false;
         }
 
-        @Override
-        public void super_onConfigurationChanged(Configuration newConfig) {
-        }
-
         @Override
         public void onScrollChanged(int lPix, int tPix, int oldlPix, int oldtPix) {
             synchronized (mLock) {