Public glue layer plumbing for View#startActivityForResult

android_webview/java/src/org/chromium/android_webview/FullScreenView.java

Index: android_webview/java/src/org/chromium/android_webview/FullScreenView.java
diff --git a/android_webview/java/src/org/chromium/android_webview/FullScreenView.java b/android_webview/java/src/org/chromium/android_webview/FullScreenView.java
index 1eb96a4eda3dd97c99e511031b4607bcfac1b2dd..b25347e99f7d2941153d93348b4db0296bc0bd82 100644
--- a/android_webview/java/src/org/chromium/android_webview/FullScreenView.java
+++ b/android_webview/java/src/org/chromium/android_webview/FullScreenView.java
@@ -5,6 +5,7 @@
 package org.chromium.android_webview;
 
 import android.content.Context;
+import android.content.Intent;
 import android.content.res.Configuration;
 import android.graphics.Canvas;
 import android.graphics.Paint;
@@ -230,6 +231,13 @@ public class FullScreenView extends FrameLayout {
         }
 
         @Override
+        public void super_startActivityForResult(Intent intent, int requestCode) {
+            // Intentional no-op. startActivityForResult will only go through

[boliu, 2015/10/21 00:22:17]

Could just raise a NotImplemented exception or something? This should be a
NOTREACHED, not no-op.

[hush (inactive), 2015/10/21 01:23:24]

yes this is supposed to be not reached. I can raise a run time exception with a
error message.

+            // the initial InternalAccessDelegate, which fires the intent
+            // through WebView.
+        }
+
+        @Override
         public boolean awakenScrollBars() {
             return FullScreenView.this.awakenScrollBars(0);
         }