Refactor FullscreenController removing TogglePresentationMode & adding ToggleFullscreenWithChrome.

chrome/browser/ui/fullscreen/fullscreen_controller.h

Index: chrome/browser/ui/fullscreen/fullscreen_controller.h
diff --git a/chrome/browser/ui/fullscreen/fullscreen_controller.h b/chrome/browser/ui/fullscreen/fullscreen_controller.h
index 6b094f0a53fc9867428ebc587c654655dfbf093e..f1e9ae15baa6faa330ebd145a68756e0738480d6 100644
--- a/chrome/browser/ui/fullscreen/fullscreen_controller.h
+++ b/chrome/browser/ui/fullscreen/fullscreen_controller.h
@@ -74,7 +74,7 @@ class FullscreenController : public content::NotificationObserver {
 #endif
 
 #if defined(OS_MACOSX)
-  void TogglePresentationMode();
+  void ToggleFullscreenWithChrome();
 #endif
 
   // Mouse Lock ////////////////////////////////////////////////////////////////
@@ -142,10 +142,9 @@ class FullscreenController : public content::NotificationObserver {
   void NotifyMouseLockChange();
 
   // TODO(koz): Change |for_tab| to an enum.
-  void ToggleFullscreenModeInternal(bool for_tab);
-#if defined(OS_MACOSX)
-  void TogglePresentationModeInternal(bool for_tab);
-#endif
+  void ToggleFullscreenModeInternal(bool for_tab, bool with_chrome);

[Robert Sesek, 2013/01/28 19:46:22]

Since there's already a TODO about using an enum, why don't you do that for
this?

[scheib, 2013/01/28 20:40:02]

Done.

+  void EnterFullscreenModeInternal(bool for_tab, bool with_chrome);
+  void ExitFullscreenModeInternal();
   void SetFullscreenedTab(content::WebContents* tab);
   void SetMouseLockTab(content::WebContents* tab);