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

Unified Diff: components/signin/core/browser/signin_error_controller.h

Issue 809313009: Make the SigninErrorController a PKS. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove unnecessary header 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: components/signin/core/browser/signin_error_controller.h
diff --git a/components/signin/core/browser/signin_error_controller.h b/components/signin/core/browser/signin_error_controller.h
index 77bd78d3751e8defd4b29beb26f7989669e668a1..368b916a84eae7162bb245b7adff0e97964e2f52 100644
--- a/components/signin/core/browser/signin_error_controller.h
+++ b/components/signin/core/browser/signin_error_controller.h
@@ -9,13 +9,14 @@
#include "base/basictypes.h"
#include "base/compiler_specific.h"
#include "base/observer_list.h"
+#include "components/keyed_service/core/keyed_service.h"
#include "google_apis/gaia/google_service_auth_error.h"
// Keep track of auth errors and expose them to observers in the UI. Services
// that wish to expose auth errors to the user should register an
// AuthStatusProvider to report their current authentication state, and should
// invoke AuthStatusChanged() when their authentication state may have changed.
-class SigninErrorController {
+class SigninErrorController : public KeyedService {
public:
class AuthStatusProvider {
public:
@@ -42,7 +43,7 @@ class SigninErrorController {
};
SigninErrorController();
- ~SigninErrorController();
+ ~SigninErrorController() override;
// Adds a provider which the SigninErrorController object will start querying
// for auth status.

Powered by Google App Engine
This is Rietveld 408576698