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

Unified Diff: chrome/browser/ui/webui/signin_internals_ui.cc

Issue 421583002: Show GAIA/cookie accounts on the signin-internals page. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: include for CrOS Created 6 years, 4 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/signin_internals_ui.cc
diff --git a/chrome/browser/ui/webui/signin_internals_ui.cc b/chrome/browser/ui/webui/signin_internals_ui.cc
index f61542785a38418c5ff61cd5e4c985bcee7ea6ee..c0c22d9ff3d67155c3f6946a936c131954ef459b 100644
--- a/chrome/browser/ui/webui/signin_internals_ui.cc
+++ b/chrome/browser/ui/webui/signin_internals_ui.cc
@@ -68,10 +68,10 @@ bool SignInInternalsUI::OverrideHandleWebUIMessage(
// empty in incognito mode. Alternatively, we could force about:signin to
// open in non-incognito mode always (like about:settings for ex.).
if (about_signin_internals) {
- const std::string& reply_handler =
- "chrome.signin.getSigninInfo.handleReply";
web_ui()->CallJavascriptFunction(
- reply_handler, *about_signin_internals->GetSigninStatus());
+ "chrome.signin.getSigninInfo.handleReply",
+ *about_signin_internals->GetSigninStatus());
+ about_signin_internals->GetCookieAccountsAsync();
return true;
}
@@ -81,6 +81,12 @@ bool SignInInternalsUI::OverrideHandleWebUIMessage(
void SignInInternalsUI::OnSigninStateChanged(
scoped_ptr<base::DictionaryValue> info) {
- const std::string& event_handler = "chrome.signin.onSigninInfoChanged.fire";
- web_ui()->CallJavascriptFunction(event_handler, *info);
+ web_ui()->CallJavascriptFunction(
+ "chrome.signin.onSigninInfoChanged.fire", *info);
+}
+
+void SignInInternalsUI::OnCookieAccountsFetched(
+ scoped_ptr<base::DictionaryValue> info) {
+ web_ui()->CallJavascriptFunction(
+ "chrome.signin.onCookieAccountsFetched.fire", *info);
}
« no previous file with comments | « chrome/browser/ui/webui/signin_internals_ui.h ('k') | components/signin/core/browser/about_signin_internals.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698