Refactor away the Browser* dependency in exclusive_access

chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc

Index: chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc
diff --git a/chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc b/chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc
index b960ce187b799c89dc697a0afddd49dcd40b7d22..58d396b79b1f4fc089cc163404eaa918c534c23f 100644
--- a/chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc
+++ b/chrome/browser/ui/exclusive_access/exclusive_access_bubble.cc
@@ -7,8 +7,8 @@
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/profiles/profile.h"
-#include "chrome/browser/ui/browser.h"
-#include "chrome/browser/ui/browser_commands.h"
+#include "chrome/browser/ui/exclusive_access/exclusive_access_context.h"
+#include "chrome/browser/ui/exclusive_access/exclusive_access_manager.h"
 #include "chrome/browser/ui/exclusive_access/fullscreen_controller.h"
 #include "chrome/grit/generated_resources.h"
 #include "extensions/browser/extension_registry.h"
@@ -32,10 +32,10 @@ const int ExclusiveAccessBubble::kSlideOutDurationMs = 700;
 const int ExclusiveAccessBubble::kPopupTopPx = 15;
 
 ExclusiveAccessBubble::ExclusiveAccessBubble(
-    Browser* browser,
+    ExclusiveAccessManager* manager,
     const GURL& url,
     ExclusiveAccessBubbleType bubble_type)
-    : browser_(browser), url_(url), bubble_type_(bubble_type) {
+    : manager_(manager), url_(url), bubble_type_(bubble_type) {
   DCHECK_NE(EXCLUSIVE_ACCESS_BUBBLE_TYPE_NONE, bubble_type_);
 }
 
@@ -115,23 +115,22 @@ void ExclusiveAccessBubble::CheckMousePosition() {
 }
 
 void ExclusiveAccessBubble::ToggleFullscreen() {

[scheib, 2015/02/25 19:30:13]

While here, would you fix this too please: Rename
ExclusiveAccessBubble::ToggleFullscreen ->
ExclusiveAccessBubble::ExitExclusiveAccess

[Sriram, 2015/02/25 21:53:27]

Actually this is explicitly dealing with fullscreen and not exiting exclusive
access so leaving this one as such.

[scheib, 2015/02/25 22:31:51]

It shouldn't be "Toggle", it is definitely not Toggling.

The only impl is Fullscreen now, but for keyboard lock without pointer lock
won't we want it to exit keyboard lock?

At this point in code I think the abstracted concept ExitExclusiveAccess is
fitting.

[Sriram, 2015/02/25 23:15:59]

Done.

-  browser_->exclusive_access_manager()
-      ->fullscreen_controller()
-      ->ExitExclusiveAccessToPreviousState();
+  manager_->fullscreen_controller()->ExitExclusiveAccessToPreviousState();
 }
 
 void ExclusiveAccessBubble::Accept() {
-  browser_->exclusive_access_manager()->OnAcceptExclusiveAccessPermission();
+  manager_->OnAcceptExclusiveAccessPermission();
 }
 
 void ExclusiveAccessBubble::Cancel() {
-  browser_->exclusive_access_manager()->OnDenyExclusiveAccessPermission();
+  manager_->OnDenyExclusiveAccessPermission();
 }
 
 base::string16 ExclusiveAccessBubble::GetCurrentMessageText() const {
   return exclusive_access_bubble::GetLabelTextForType(
       bubble_type_, url_,
-      extensions::ExtensionRegistry::Get(browser_->profile()));
+      extensions::ExtensionRegistry::Get(
+          manager_->exclusive_access_context()->GetProfile()));
 }
 
 base::string16 ExclusiveAccessBubble::GetCurrentDenyButtonText() const {