Refactor ContentViewClient (1/6)

components/web_contents_delegate_android/android/java/src/org/chromium/components/web_contents_delegate_android/WebContentsDelegateAndroid.java

Index: components/web_contents_delegate_android/android/java/src/org/chromium/components/web_contents_delegate_android/WebContentsDelegateAndroid.java
diff --git a/components/web_contents_delegate_android/android/java/src/org/chromium/components/web_contents_delegate_android/WebContentsDelegateAndroid.java b/components/web_contents_delegate_android/android/java/src/org/chromium/components/web_contents_delegate_android/WebContentsDelegateAndroid.java
index fe55dcebf2e7313c903b07f7626569f22028cead..e8c26f9ff22f80b672c413cf4081eac845c123de 100644
--- a/components/web_contents_delegate_android/android/java/src/org/chromium/components/web_contents_delegate_android/WebContentsDelegateAndroid.java
+++ b/components/web_contents_delegate_android/android/java/src/org/chromium/components/web_contents_delegate_android/WebContentsDelegateAndroid.java
@@ -8,6 +8,7 @@ import android.view.KeyEvent;
 
 import org.chromium.base.annotations.CalledByNative;
 import org.chromium.base.annotations.JNINamespace;
+import org.chromium.content.browser.ContentVideoViewEmbedder;
 import org.chromium.content_public.browser.WebContents;
 import org.chromium.content_public.common.ResourceRequestBody;
 
@@ -153,4 +154,8 @@ public class WebContentsDelegateAndroid {
     public boolean isFullscreenForTabOrPending() {
         return false;
     }
+
+    public ContentVideoViewEmbedder getContentVideoViewEmbedder() {

[boliu, 2016/09/26 21:25:24]

is this needed at all?

actually this actually raises another question, is chrome also subclassing this?
then maybe the native plumbing can be shared in this component instead?

[Jinsuk Kim, 2016/09/27 10:08:58]

WDITOT - moved the native plumbing of both chrome/webview to this class.

+        return null;
+    }
 }