Don't use webmediaplayer to FullscreenController about fullscreen change

content/public/browser/render_view_host.h

Index: content/public/browser/render_view_host.h
diff --git a/content/public/browser/render_view_host.h b/content/public/browser/render_view_host.h
index 559c0ef4c69de1b91464fa869ee3fbbaae38edf9..35dcc06269ed8e078f268887e2b5222c8e043fa4 100644
--- a/content/public/browser/render_view_host.h
+++ b/content/public/browser/render_view_host.h
@@ -223,6 +223,9 @@ class CONTENT_EXPORT RenderViewHost : virtual public RenderWidgetHost {
 
   // Asks the renderer to send the rects of the current find matches.
   virtual void RequestFindMatchRects(int current_version) = 0;
+
+  // The page has entered or exited a fullscreen state.
+  virtual void FullscreenChanged() = 0;

[jam, 2014/06/16 20:09:06]

why add this method to RVH when all it does is call RWH::WasResized() which is
public? i.e. just have the calling code call WasResized()

[boliu, 2014/06/16 20:12:45]

I asked for it in the review because it made no sense to call WasResized from
webview code where it's clearly *not* resized. WasResized does more than just
updating the size. It also handles the full screen state, and that's the bit
webview is interested in.

[jam, 2014/06/16 22:44:16]

I don't understand what you mean. This code adds a new method to RVH that just
calls WasResized(). So I don't see how that's different from calling WasResized
directly?

Unless I'm missing something above, please just use WasResized. Especially since
RVH is going away as part of site isolation, while RWH is not.

[qinmin, 2014/06/16 23:16:18]

Done. Reverting back to WasResized() now.

On 2014/06/16 22:44:16, jam wrote:

 #endif
 
  private: