Refactor away the Browser* dependency in exclusive_access

chrome/browser/ui/exclusive_access/exclusive_access_context.h

+// Copyright (c) 2015 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 CHROME_BROWSER_UI_EXCLUSIVE_ACCESS_EXCLUSIVE_ACCESS_CONTEXT_H_

[miu, 2015/02/26 20:07:57]

Please add a newline above this line.

[Sriram, 2015/02/26 23:58:03]

Done.

+#define CHROME_BROWSER_UI_EXCLUSIVE_ACCESS_EXCLUSIVE_ACCESS_CONTEXT_H_
+
+#include "chrome/browser/ui/exclusive_access/exclusive_access_bubble_type.h"
+
+class GURL;
+class Profile;
+
+namespace content {
+class WebContents;
+}
+
+// Context in which exclusive access operation is being performed. This
+// interface is implemented once in Browser context and in Platform Application
+// context.
+class ExclusiveAccessContext {
+ public:
+  virtual ~ExclusiveAccessContext() {}
+
+  // Returns the current profile associated with the window.
+  virtual Profile* GetProfile() = 0;
+
+  // Returns true if the window hosting the exclusive access bubble is
+  // fullscreen.
+  virtual bool IsFullscreen() const = 0;
+
+  // Returns true if fullscreen with toolbar is supported.
+  virtual bool SupportsFullscreenWithToolbar() const;
+
+  // Shows or hides the tab strip, toolbar and bookmark bar with in browser
+  // fullscreen.
+  // Currently only supported on Mac.
+  virtual void UpdateFullscreenWithToolbar(bool with_toolbar);
+  // Returns true if the window is fullscreen with additional UI elements. See

[miu, 2015/02/26 20:07:56]

Please add a newline above this line.

[Sriram, 2015/02/26 23:58:03]

Done.

+  // EnterFullscreen |with_toolbar|.
+  virtual bool IsFullscreenWithToolbar() const = 0;
+
+  // Enters fullscreen and update exit bubble.
+  // On Mac, the tab strip and toolbar will be shown if |with_toolbar| is true,
+  // |with_toolbar| is ignored on other platforms.
+  virtual void EnterFullscreen(const GURL& url,
+                               ExclusiveAccessBubbleType bubble_type,
+                               bool with_toolbar) = 0;
+
+  // Exits fullscreen and update exit bubble.
+  virtual void ExitFullscreen() = 0;
+
+  // Updates the content of exclusive access exit bubble content.
+  virtual void UpdateExclusiveAccessExitBubbleContent(
+      const GURL& url,
+      ExclusiveAccessBubbleType bubble_type) = 0;
+
+#if defined(OS_WIN)
+  // Sets state for entering or exiting Win8 Metro snap mode.
+  virtual void SetMetroSnapMode(bool enable);
+
+  // Returns whether the window is currently in Win8 Metro snap mode.
+  virtual bool IsInMetroSnapMode() const;
+#endif  // defined(OS_WIN)
+
+  // Returns the currently active WebContents, or nullptr if there is none.
+  virtual content::WebContents* GetActiveWebContents() = 0;
+
+  // Displays the download shelf associated with currently active window.
+  virtual void UnhideDownloadShelf();

[miu, 2015/02/26 20:07:57]

The hide/unhide download shelf methods are an abstraction violation.  Meaning,
the concept only applies to the Browser and not a platform app window.  So,
please add a TODO comment here to remove them in a later change.  Ideally, the
BrowserView should hide its download shelf widget when it is instructed to enter
fullscreen mode.

[Sriram, 2015/02/26 23:58:03]

Done.

+
+  // Hides download shelf associated with currently active window.
+  virtual void HideDownloadShelf();
+};
+
+#endif  // CHROME_BROWSER_UI_EXCLUSIVE_ACCESS_EXCLUSIVE_ACCESS_CONTEXT_H_