Fullscreen: make fullscreen requests come from RenderFrame

content/public/browser/web_contents_delegate.h

Index: content/public/browser/web_contents_delegate.h
diff --git a/content/public/browser/web_contents_delegate.h b/content/public/browser/web_contents_delegate.h
index 17bed7fbf401461207400c972c1e4f6892923b55..9b1eaf33566761fe96e19a2d19d5199d09953e56 100644
--- a/content/public/browser/web_contents_delegate.h
+++ b/content/public/browser/web_contents_delegate.h
@@ -372,7 +372,11 @@ class CONTENT_EXPORT WebContentsDelegate {
   virtual bool EmbedsFullscreenWidget() const;
 
   // Called when the renderer puts a tab into or out of fullscreen mode.
+  // |origin| will always be empty when |enter_fullscreen| is false.

[Charlie Reis, 2014/12/10 00:51:21]

These comments help, but please first say what |origin| is for (i.e., what it
means when it isn't empty).

[mlamouri (slow - plz ping), 2014/12/10 15:47:02]

Done.

+  // |origin| can be empty when |enter_fullscreen| in which case the

[Charlie Reis, 2014/12/10 00:51:21]

when |enter_fullscreen| is true, in which case...

[mlamouri (slow - plz ping), 2014/12/10 15:47:02]

No longer exists.

+  // implementation is expected to use |web_contents|' origin instead.
   virtual void ToggleFullscreenModeForTab(WebContents* web_contents,
+                                          const GURL& origin,
                                           bool enter_fullscreen) {}
   virtual bool IsFullscreenForTabOrPending(
       const WebContents* web_contents) const;