Support fullscreen video for content shell

content/shell/android/java/src/org/chromium/content_shell/Shell.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_shell;
 
 import android.content.Context;
 import android.graphics.drawable.ClipDrawable;
 import android.text.TextUtils;
 import android.util.AttributeSet;
@@ skipping 45 matching lines @@
     private ImageButton mStopButton;
     private ImageButton mReloadButton;
 
     private ClipDrawable mProgressDrawable;
 
     private long mNativeShell;
     private ContentViewRenderView mContentViewRenderView;
     private WindowAndroid mWindow;
 
     private boolean mLoading = false;
+    private boolean mIsFullscreen = false;
 
     /**
      * Constructor for inflating via XML.
      */
     public Shell(Context context, AttributeSet attrs) {
         super(context, attrs);
     }
 
     /**
      * Set the SurfaceView being renderered to as soon as it is available.
@@ skipping 178 matching lines @@
     @SuppressWarnings("unused")
     @CalledByNative
     private void onLoadProgressChanged(double progress) {
         removeCallbacks(mClearProgressRunnable);
         mProgressDrawable.setLevel((int) (10000.0 * progress));
         if (progress == 1.0) postDelayed(mClearProgressRunnable, COMPLETED_PROGRESS_TIMEOUT_MS);
     }
 
     @CalledByNative
     private void toggleFullscreenModeForTab(boolean enterFullscreen) {
+        mIsFullscreen = enterFullscreen;
+        mWebContents.wasResized();

[Ted C, 2014/10/02 21:52:45]

Why is this necessary?

Could we just do:
mUrlTextView.setVisibility(enterFullscreen ? GONE : VISIBLE);

That should trigger a resize on the content view, which I think is more correct
and avoids adding the wasResized function which already exists on
ContentViewCore, and I think shouldn't be added to WebContentsAndroid.

[qinmin, 2014/10/02 22:44:30]

Done. Set the whole toolbar's visibility to false when we enter fullscreen,
setting textview alone will still show the navigation buttons.

On 2014/10/02 21:52:45, Ted C wrote:

     }
 
     @CalledByNative
     private boolean isFullscreenForTabOrPending() {
-        return false;
+        return mIsFullscreen;
     }
 
     @SuppressWarnings("unused")
     @CalledByNative
     private void setIsLoading(boolean loading) {
         mLoading = loading;
     }
 
     /**
      * Initializes the ContentView based on the native tab contents pointer passed in.
@@ skipping 56 matching lines @@
                 Context.INPUT_METHOD_SERVICE);
         if (visible) {
             imm.showSoftInput(mUrlTextView, InputMethodManager.SHOW_IMPLICIT);
         } else {
             imm.hideSoftInputFromWindow(mUrlTextView.getWindowToken(), 0);
         }
     }
 
     private static native void nativeCloseShell(long shellPtr);
 }