Fullscreen: make fullscreen requests come from RenderFrame

content/public/browser/web_contents.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
 
 #include <set>
 
 #include "base/basictypes.h"
@@ skipping 571 matching lines @@
   virtual void InsertCSS(const std::string& css) = 0;
 
   // Returns true if audio has recently been audible from the WebContents.
   virtual bool WasRecentlyAudible() = 0;
 
   typedef base::Callback<void(const Manifest&)> GetManifestCallback;
 
   // Requests the Manifest of the main frame's document.
   virtual void GetManifest(const GetManifestCallback&) = 0;
 
+  // Returns whether the WebContents is associated with a fullscreen tab.
+  virtual bool IsFullscreenForCurrentTab() const = 0;

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

This doesn't look like it's used outside of content/, so we shouldn't put it in
the public interface.

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

Hmm, I did not meant to have this in this CL. It's a remaining of trying to move
IsFullscreenForCurrentTab out of RVHDelegate and directly to the  WebContents.

+
 #if defined(OS_ANDROID)
   CONTENT_EXPORT static WebContents* FromJavaWebContents(
       jobject jweb_contents_android);
   virtual base::android::ScopedJavaLocalRef<jobject> GetJavaWebContents() = 0;
 #elif defined(OS_MACOSX)
   // Allowing other views disables optimizations which assume that only a single
   // WebContents is present.
   virtual void SetAllowOtherViews(bool allow) = 0;
 
   // Returns true if other views are allowed, false otherwise.
   virtual bool GetAllowOtherViews() = 0;
 #endif  // OS_ANDROID
 
  private:
   // This interface should only be implemented inside content.
   friend class WebContentsImpl;
   WebContents() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_