Implement fullscreen info bubble on Win and Mac

chrome/browser/ui/cocoa/browser_window_cocoa.h

Index: chrome/browser/ui/cocoa/browser_window_cocoa.h
diff --git a/chrome/browser/ui/cocoa/browser_window_cocoa.h b/chrome/browser/ui/cocoa/browser_window_cocoa.h
index ab3c8325e855e89ee9acc6c40db52d5f863fe248..f632fef89d2bab6cc05213c642dc186a69c56e40 100644
--- a/chrome/browser/ui/cocoa/browser_window_cocoa.h
+++ b/chrome/browser/ui/cocoa/browser_window_cocoa.h
@@ -55,7 +55,8 @@ class BrowserWindowCocoa : public BrowserWindow,
   virtual gfx::Rect GetBounds() const;
   virtual bool IsMaximized() const;
   virtual bool IsMinimized() const;
-  virtual void SetFullscreen(bool fullscreen);
+  virtual void SetFullscreen(bool fullscreen, const GURL& url,
+      bool ask_permission);

[Nico, 2011/10/12 16:23:09]

Consider using an enum instead of a bool – makes the calling sites more
readable. Also below.

[jeremya, 2011/10/13 05:16:54]

I think the problem was mainly that the SetFullscreen(false, ...) case was
confusing, since the GURL and the bool were unused in that case. I've split it
into EnterFullscreen and ExitFullscreen, which I think makes the callsites more
readable even without an enum.

   virtual bool IsFullscreen() const;
   virtual bool IsFullscreenBubbleVisible() const;
   virtual LocationBar* GetLocationBar() const;
@@ -105,7 +106,9 @@ class BrowserWindowCocoa : public BrowserWindow,
   virtual void Copy();
   virtual void Paste();
   virtual void OpenTabpose();
-  virtual void SetPresentationMode(bool presentation_mode);
+  virtual void SetPresentationMode(bool presentation_mode,
+                                   const GURL& url,
+                                   bool ask_permission);
   virtual bool InPresentationMode();
   virtual void PrepareForInstant();
   virtual void ShowInstant(TabContentsWrapper* preview);