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

Unified Diff: chrome/browser/chromeos/login/wizard_controller.h

Issue 166273014: Remove dependency on NotificationService for cros a11y settings. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge ToT. Created 6 years, 10 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/chromeos/login/wizard_controller.h
diff --git a/chrome/browser/chromeos/login/wizard_controller.h b/chrome/browser/chromeos/login/wizard_controller.h
index b4140df4713d23f785c42a56f6c418855ce653a2..6f59f07ea84fed0c1cfea10e7cafc17b2700d5e3 100644
--- a/chrome/browser/chromeos/login/wizard_controller.h
+++ b/chrome/browser/chromeos/login/wizard_controller.h
@@ -14,10 +14,9 @@
#include "base/observer_list.h"
#include "base/time/time.h"
#include "base/timer/timer.h"
+#include "chrome/browser/chromeos/accessibility/accessibility_manager.h"
#include "chrome/browser/chromeos/login/screens/screen_observer.h"
#include "chrome/browser/chromeos/login/screens/wizard_screen.h"
-#include "content/public/browser/notification_observer.h"
-#include "content/public/browser/notification_registrar.h"
#include "ui/gfx/rect.h"
#include "url/gurl.h"
@@ -49,8 +48,7 @@ class WrongHWIDScreen;
// Class that manages control flow between wizard screens. Wizard controller
// interacts with screen controllers to move the user between screens.
-class WizardController : public ScreenObserver,
- public content::NotificationObserver {
+class WizardController : public ScreenObserver {
public:
// Observes screen changes.
class Observer {
@@ -227,10 +225,9 @@ class WizardController : public ScreenObserver,
virtual void ShowErrorScreen() OVERRIDE;
virtual void HideErrorScreen(WizardScreen* parent_screen) OVERRIDE;
- // Overridden from content::NotificationObserver:
- virtual void Observe(int type,
- const content::NotificationSource& source,
- const content::NotificationDetails& details) OVERRIDE;
+ // Notification of a change in the state of an accessibility setting.
+ void OnAccessibilityStatusChanged(
+ const AccessibilityStatusEventDetails& details);
// Switches from one screen to another.
void SetCurrentScreen(WizardScreen* screen);
@@ -342,9 +339,9 @@ class WizardController : public ScreenObserver,
friend class WizardInProcessBrowserTest;
friend class WizardControllerBrokenLocalStateTest;
- base::WeakPtrFactory<WizardController> weak_factory_;
+ scoped_ptr<AccessibilityStatusSubscription> accessibility_subscription_;
- content::NotificationRegistrar registrar_;
+ base::WeakPtrFactory<WizardController> weak_factory_;
DISALLOW_COPY_AND_ASSIGN(WizardController);
};

Powered by Google App Engine
This is Rietveld 408576698