Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6664)

Unified Diff: chrome/browser/ui/exclusive_access/exclusive_access_controller_base.h

Issue 836933005: Refactor fullscreen_controller. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Sync to TOT Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/exclusive_access/exclusive_access_controller_base.h
diff --git a/chrome/browser/sync/sessions/notification_service_sessions_router.h b/chrome/browser/ui/exclusive_access/exclusive_access_controller_base.h
similarity index 15%
copy from chrome/browser/sync/sessions/notification_service_sessions_router.h
copy to chrome/browser/ui/exclusive_access/exclusive_access_controller_base.h
index f1249f19c2b5af626dc32a9e457d7ecb40f55aec..99c561426df9d497439b0f73108a37364bf1147d 100644
--- a/chrome/browser/sync/sessions/notification_service_sessions_router.h
+++ b/chrome/browser/ui/exclusive_access/exclusive_access_controller_base.h
@@ -1,73 +1,101 @@
-// 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 {
-
-// A SessionsSyncManager::LocalEventRouter that drives session sync via
-// the NotificationService.
-class NotificationServiceSessionsRouter
- : public LocalSessionEventRouter,
- public content::NotificationObserver {
+// The base class for the different exclusive access controllers like the
+// FullscreenController, and MouseLockController which controls lifetime for
+// which the resource (screen/mouse) is held exclusively.
+class ExclusiveAccessControllerBase : public content::NotificationObserver {
public:
- NotificationServiceSessionsRouter(
- Profile* profile,
- const syncer::SyncableService::StartSyncFlare& flare);
- ~NotificationServiceSessionsRouter() override;
+ explicit ExclusiveAccessControllerBase(ExclusiveAccessManager* manager,
+ Browser* browser);
+ ~ExclusiveAccessControllerBase() override;
+
+ GURL GetExclusiveAccessBubbleURL() const;
+ virtual GURL GetURLForExclusiveAccessBubble() const;
+
+ 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;
+
+ // Called to indicate that the permission for this exclusive access has been
+ // granted if requested. Returns true if the exit bubble content should be
+ // updated.
+ virtual bool OnAcceptExclusiveAccessPermission() = 0;
+
+ // Called to indicate that the permission for the exclusive access has been
+ // denied if requested. Returns true if the exit 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);
- // content::NotificationObserver implementation.
- // BrowserSessionProvider -> sync API model change application.
+ // 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);
- 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);
+ ExclusiveAccessManager* exclusive_access_manager() const { return manager_; }
- // Called when the urls of favicon changed.
- void OnFaviconChanged(const std::set<GURL>& changed_favicons);
+ Browser* browser() const { return browser_; }
- LocalSessionEventHandler* handler_;
- content::NotificationRegistrar registrar_;
+ Profile* profile() const { return profile_; }
+
+ // Exits exclusive access mode for the tab if currently exclusive.
+ virtual void ExitExclusiveAccessIfNecessary() = 0;
+
+ // Notifies the tab that it has been forced out of exclusive access mode
+ // if necessary.
+ virtual void NotifyTabExclusiveAccessLost() = 0;
+
+ private:
+ void UpdateNotificationRegistrations();
+
+ ExclusiveAccessManager* const manager_;
+ Browser* const browser_;
Profile* const profile_;
- syncer::SyncableService::StartSyncFlare flare_;
- scoped_ptr<base::CallbackList<void(const std::set<GURL>&)>::Subscription>
- favicon_changed_subscription_;
+ content::NotificationRegistrar registrar_;
- base::WeakPtrFactory<NotificationServiceSessionsRouter> weak_ptr_factory_;
+ content::WebContents* tab_with_exclusive_access_;
- 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_

Powered by Google App Engine
This is Rietveld 408576698