Fullscreen: make fullscreen requests come from RenderFrame

components/web_contents_delegate_android/web_contents_delegate_android.cc

 // Copyright (c) 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.
 
 #include "components/web_contents_delegate_android/web_contents_delegate_android.h"
 
 #include <android/keycodes.h>
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
@@ skipping 355 matching lines @@
 
 void WebContentsDelegateAndroid::ShowRepostFormWarningDialog(
     WebContents* source) {
   JNIEnv* env = AttachCurrentThread();
   ScopedJavaLocalRef<jobject> obj = GetJavaDelegate(env);
   if (obj.is_null())
     return;
   Java_WebContentsDelegateAndroid_showRepostFormWarningDialog(env, obj.obj());
 }
 
-void WebContentsDelegateAndroid::ToggleFullscreenModeForTab(
+void WebContentsDelegateAndroid::EnterFullscreenModeForTab(
     WebContents* web_contents,
-    bool enter_fullscreen) {
+    const GURL& origin) {
   JNIEnv* env = AttachCurrentThread();
   ScopedJavaLocalRef<jobject> obj = GetJavaDelegate(env);
   if (obj.is_null())
     return;
   Java_WebContentsDelegateAndroid_toggleFullscreenModeForTab(
-      env, obj.obj(), enter_fullscreen);
+      env, obj.obj(), true);

[blundell, 2014/12/10 20:47:15]

Isn't this change a no-op? I would expect it be doing something with |origin|.

[mlamouri (slow - plz ping), 2014/12/10 20:52:35]

It is indeed a no-op. We don't use the origin on Android so there is no point in
passing in trough for now.

+}
+
+void WebContentsDelegateAndroid::ExitFullscreenModeForTab(
+    WebContents* web_contents) {
+  JNIEnv* env = AttachCurrentThread();
+  ScopedJavaLocalRef<jobject> obj = GetJavaDelegate(env);
+  if (obj.is_null())
+    return;
+  Java_WebContentsDelegateAndroid_toggleFullscreenModeForTab(
+      env, obj.obj(), false);
 }
 
 bool WebContentsDelegateAndroid::IsFullscreenForTabOrPending(
     const WebContents* web_contents) const {
   JNIEnv* env = AttachCurrentThread();
   ScopedJavaLocalRef<jobject> obj = GetJavaDelegate(env);
   if (obj.is_null())
     return false;
   return Java_WebContentsDelegateAndroid_isFullscreenForTabOrPending(
       env, obj.obj());
@@ skipping 34 matching lines @@
 // Native JNI methods
 // ----------------------------------------------------------------------------
 
 // Register native methods
 
 bool RegisterWebContentsDelegateAndroid(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace web_contents_delegate_android