Refactor fullscreen_controller.

chrome/browser/ui/exclusive_access/exclusive_access_controller_base.h

-// Copyright 2014 The Chromium Authors. All rights reserved.
+// 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_SYNC_SESSIONS_NOTIFICATION_SERVICE_SESSIONS_ROUTER_H_
-#define CHROME_BROWSER_SYNC_SESSIONS_NOTIFICATION_SERVICE_SESSIONS_ROUTER_H_
+#ifndef CHROME_BROWSER_UI_EXCLUSIVE_ACCESS_EXCLUSIVE_ACCESS_CONTROLLER_BASE_H_
+#define CHROME_BROWSER_UI_EXCLUSIVE_ACCESS_EXCLUSIVE_ACCESS_CONTROLLER_BASE_H_
 
-#include <set>
-
-#include "base/callback_list.h"
+#include "base/basictypes.h"
 #include "base/memory/weak_ptr.h"
-#include "chrome/browser/sync/sessions/sessions_sync_manager.h"
+#include "chrome/browser/ui/exclusive_access/exclusive_access_bubble_type.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 
+class Browser;
+class BrowserWindow;
+class ExclusiveAccessManager;
 class GURL;
-class HistoryService;
 class Profile;
 
 namespace content {
 class WebContents;
 }
 
-namespace browser_sync {
+// The base class for the different exclusive access controller like the

[scheib, 2015/01/22 22:54:04]

controller -> controllers

[Sriram, 2015/01/23 07:13:43]

Done.

+// FullscreenController, and MouseLockController which controls lifetime for
+// which the resource (screen/mouse) is held exclusively.
+class ExclusiveAccessControllerBase : public content::NotificationObserver {
+ public:
+  explicit ExclusiveAccessControllerBase(ExclusiveAccessManager* manager,
+                                         Browser* browser);
+  ~ExclusiveAccessControllerBase() override;
 
-// A SessionsSyncManager::LocalEventRouter that drives session sync via
-// the NotificationService.
-class NotificationServiceSessionsRouter
-    : public LocalSessionEventRouter,
-      public content::NotificationObserver {
- public:
-  NotificationServiceSessionsRouter(
-      Profile* profile,
-      const syncer::SyncableService::StartSyncFlare& flare);
-  ~NotificationServiceSessionsRouter() override;
+  GURL GetExclusiveAccessBubbleURL() const;
+  virtual GURL GetURLForExclusiveAccessBubble() const;
 
-  // content::NotificationObserver implementation.
-  // BrowserSessionProvider -> sync API model change application.
+  content::WebContents* exclusive_access_tab() const {
+    return tab_with_exclusive_access_;
+  }
+
+  // Functions implemented by derived classes:
+
+  // Control behavior when escape is pressed returning true if it was handled.
+  virtual bool HandleUserPressedEscape() = 0;
+
+  // Called by Browser in response to call from ExclusiveAccessBubble.
+  virtual void ExitExclusiveAccessToPreviousState() = 0;
+
+  // Return true if bubble content should be updated.

[scheib, 2015/01/22 22:54:04]

Return -> Returns
 
http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Function_Comm...
"These comments should be descriptive ("Opens the file") rather than imperative
("Open the file"); "
Probably prefix with explanation as well. e.g.
// Called to indicate that the permission for this exclusive access has been
granted. Returns true of the bubble content should be updated.

[Sriram, 2015/01/23 07:13:43]

Done.

+  virtual bool OnAcceptExclusiveAccessPermission() = 0;
+
+  // Return true if bubble content should be updated.
+  virtual bool OnDenyExclusiveAccessPermission() = 0;
+
+  // Called by ExclusiveAccessManager in response to calls from Browser.
+  virtual void OnTabDeactivated(content::WebContents* web_contents);
+  virtual void OnTabDetachedFromView(content::WebContents* web_contents);
+  virtual void OnTabClosing(content::WebContents* web_contents);
+
+  // Callbacks ////////////////////////////////////////////////////////////////
+
+  // content::NotificationObserver to detect page navigation and exit exclusive
+  // access.
   void Observe(int type,
                const content::NotificationSource& source,
                const content::NotificationDetails& details) override;
 
-  // SessionsSyncManager::LocalEventRouter implementation.
-  void StartRoutingTo(LocalSessionEventHandler* handler) override;
-  void Stop() override;
+ protected:
+  void SetTabWithExclusiveAccess(content::WebContents* tab);
+
+  ExclusiveAccessManager* exclusive_access_manager() const { return manager_; }
+
+  Browser* browser() const { return browser_; }
+
+  Profile* profile() const { return profile_; }
+
+  // Make the current tab exit exclusive access mode if it is in it.

[scheib, 2015/01/22 22:54:04]

(Descriptive vs Imperative)
// Exits exclusive access mode for the tab if currently exclusive.

[Sriram, 2015/01/23 07:13:43]

Done.

+  virtual void ExitExclusiveAccessIfNecessary() = 0;
+
+  // Notifies the tab that it has been forced out of exclusive access mode
+  // if necessary.
+  virtual void NotifyTabOfExclusiveAccessChange() = 0;

[scheib, 2015/01/22 22:54:04]

The implementations are for exiting exclusive access. Rename to e.g.
NotifyTabExclusiveAccessLost.

[Sriram, 2015/01/23 07:13:43]

Done.

 
  private:
-  // Called when the URL visited in |web_contents| was blocked by the
-  // SupervisedUserService. We forward this on to our handler_ via the
-  // normal OnLocalTabModified, but pass through here via a WeakPtr
-  // callback from SupervisedUserService and to extract the tab delegate
-  // from WebContents.
-  void OnNavigationBlocked(content::WebContents* web_contents);
+  void UpdateNotificationRegistrations();
 
-  // Called when the urls of favicon changed.
-  void OnFaviconChanged(const std::set<GURL>& changed_favicons);
+  ExclusiveAccessManager* const manager_;
+  Browser* const browser_;
+  Profile* const profile_;
 
-  LocalSessionEventHandler* handler_;
   content::NotificationRegistrar registrar_;
-  Profile* const profile_;
-  syncer::SyncableService::StartSyncFlare flare_;
 
-  scoped_ptr<base::CallbackList<void(const std::set<GURL>&)>::Subscription>
-      favicon_changed_subscription_;
+  content::WebContents* tab_with_exclusive_access_;
 
-  base::WeakPtrFactory<NotificationServiceSessionsRouter> weak_ptr_factory_;
-
-  DISALLOW_COPY_AND_ASSIGN(NotificationServiceSessionsRouter);
+  DISALLOW_COPY_AND_ASSIGN(ExclusiveAccessControllerBase);
 };
 
-}  // namespace browser_sync
-
-#endif  // CHROME_BROWSER_SYNC_SESSIONS_NOTIFICATION_SERVICE_SESSIONS_ROUTER_H_
+#endif  // CHROME_BROWSER_UI_EXCLUSIVE_ACCESS_EXCLUSIVE_ACCESS_CONTROLLER_BASE_H_