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

Unified Diff: chrome/browser/ui/webui/ntp/new_tab_page_sync_handler.h

Issue 12088040: Add a SigninAllowed policy. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address nits and rebase to ToT. Created 7 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/ui/webui/ntp/new_tab_page_sync_handler.h
diff --git a/chrome/browser/ui/webui/ntp/new_tab_page_sync_handler.h b/chrome/browser/ui/webui/ntp/new_tab_page_sync_handler.h
index 9ac909283d3cbce671210e3b71cd6af9b6520d8c..4c10eb64ca616ce49552354ff5bec0beafb84492 100644
--- a/chrome/browser/ui/webui/ntp/new_tab_page_sync_handler.h
+++ b/chrome/browser/ui/webui/ntp/new_tab_page_sync_handler.h
@@ -7,6 +7,7 @@
#include <string>
+#include "base/prefs/public/pref_change_registrar.h"
#include "chrome/browser/sync/profile_sync_service.h"
#include "chrome/browser/sync/sync_ui_util.h"
#include "content/public/browser/web_ui_message_handler.h"
@@ -34,15 +35,21 @@ class NewTabPageSyncHandler : public content::WebUIMessageHandler,
// ProfileSyncServiceObserver
virtual void OnStateChanged() OVERRIDE;
+ void OnSigninAllowedPrefChange();
+
private:
+ friend class MockNewTabPageSyncHandler;
+ FRIEND_TEST_ALL_PREFIXES(NewTabPageSyncHandlerBrowserTest,
+ ChangeSigninAllowedToFalse);
+
enum MessageType {
HIDE,
SYNC_ERROR,
SYNC_PROMO
};
// Helper to invoke the |syncMessageChanged| JS function on the new tab page.
- void SendSyncMessageToPage(MessageType type,
- std::string msg, std::string linktext);
+ virtual void SendSyncMessageToPage(MessageType type,
+ std::string msg, std::string linktext);
// Helper to query the sync service and figure out what to send to
// the page, and send it via SendSyncMessageToPage.
@@ -59,6 +66,8 @@ class NewTabPageSyncHandler : public content::WebUIMessageHandler,
// Cached pointer to ProfileSyncService.
ProfileSyncService* sync_service_;
+ PrefChangeRegistrar profile_pref_registrar_;
+
// Used to make sure we don't register ourselves twice if the user refreshes
// the new tab page.
bool waiting_for_initial_page_load_;
« no previous file with comments | « chrome/browser/ui/toolbar/wrench_menu_model.cc ('k') | chrome/browser/ui/webui/ntp/new_tab_page_sync_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698