Add support for entering/exiting HTML fullscreen from OOPIFs.

content/browser/web_contents/web_contents_impl.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 "content/browser/web_contents/web_contents_impl.h"
 
 #include <stddef.h>
 
 #include <cmath>
 #include <utility>
@@ skipping 1847 matching lines @@
 
   FOR_EACH_OBSERVER(WebContentsObserver, observers_,
                     DidToggleFullscreenModeForTab(
                         IsFullscreenForCurrentTab(
                             GetRenderViewHost()->GetWidget()),
                         will_cause_resize));
 }
 
 bool WebContentsImpl::IsFullscreenForCurrentTab(
     RenderWidgetHostImpl* render_widget_host) const {
-  if (!RenderViewHostImpl::From(render_widget_host))

[alexmos, 2016/05/10 21:36:24]

Removing this allows subframe widgets to pick up the is_fullscreen_granted bit
when sending ViewMsg_Resize. (This is used in RWHI::GetResizeParams.)  

I've checked all other uses of this, and they all pass in an RVH's widget, so
this should be safe.  In fact, the |render_widget_host| param is now
unnecessary, but I'd like to remove it in a follow-up CL, since it affects a
bunch of unrelated files.

[Charlie Reis, 2016/05/12 22:59:23]

Ok.  Let's put a TODO to remove it.

-    return false;
-
   return delegate_ ? delegate_->IsFullscreenForTabOrPending(this) : false;
 }
 
 blink::WebDisplayMode WebContentsImpl::GetDisplayMode(
     RenderWidgetHostImpl* render_widget_host) const {
   if (!RenderViewHostImpl::From(render_widget_host))
     return blink::WebDisplayModeBrowser;
 
   return delegate_ ? delegate_->GetDisplayMode(this)
                    : blink::WebDisplayModeBrowser;
@@ skipping 3151 matching lines @@
   for (RenderViewHost* render_view_host : render_view_host_set)
     render_view_host->OnWebkitPreferencesChanged();
 }
 
 void WebContentsImpl::SetJavaScriptDialogManagerForTesting(
     JavaScriptDialogManager* dialog_manager) {
   dialog_manager_ = dialog_manager;
 }
 
 }  // namespace content