Added user settings to chrome://supervised-user-internals.

chrome/browser/ui/webui/supervised_user_internals_message_handler.cc

Index: chrome/browser/ui/webui/supervised_user_internals_message_handler.cc
diff --git a/chrome/browser/ui/webui/supervised_user_internals_message_handler.cc b/chrome/browser/ui/webui/supervised_user_internals_message_handler.cc
index 234c79964382074cf0532bbd7c42005fd1b2bdd5..2dbf55b90cd4ac16704b5ccde6c7d560c181e38a 100644
--- a/chrome/browser/ui/webui/supervised_user_internals_message_handler.cc
+++ b/chrome/browser/ui/webui/supervised_user_internals_message_handler.cc
@@ -10,6 +10,8 @@
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/supervised_user/supervised_user_service.h"
 #include "chrome/browser/supervised_user/supervised_user_service_factory.h"
+#include "chrome/browser/supervised_user/supervised_user_settings_service.h"
+#include "chrome/browser/supervised_user/supervised_user_settings_service_factory.h"
 #include "components/url_formatter/url_fixer.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/web_ui.h"
@@ -246,6 +248,20 @@ void SupervisedUserInternalsMessageHandler::SendBasicInfo() {
   result.Set("sections", section_list.Pass());
   web_ui()->CallJavascriptFunction(
       "chrome.supervised_user_internals.receiveBasicInfo", result);
+
+  // Trigger retrieval of the user settings
+  SupervisedUserSettingsService* settings_service =
+    SupervisedUserSettingsServiceFactory::GetForProfile(profile);

[Bernhard Bauer, 2015/08/18 13:09:09]

Nit: Broken lines are indentend four spaces.

[PEConn2, 2015/08/18 14:02:58]

Done.

+  settings_service->Subscribe(base::Bind(

[Bernhard Bauer, 2015/08/18 13:09:09]

Hm... any time we open this page, this will subscribe a new callback, which is
then no-op'ed out when this object is destroyed, but is still subscribed,
essentially making this a (small) leak. Maybe we should change the interface to
SupervisedUserSettingsService so that it allows unsubscription, e.g. by
returning a base::CallbackList::Subscription. But we can do that in a separate
CL and rebase this one on top, or do it in a followup CL (but in that case we
should add a TODO here).

[PEConn2, 2015/08/18 14:02:58]

Done.

+        &SupervisedUserInternalsMessageHandler::SendSupervisedUserSettings,
+        weak_factory_.GetWeakPtr()));
+}
+
+void SupervisedUserInternalsMessageHandler::SendSupervisedUserSettings(
+    const base::DictionaryValue* settings){
+  web_ui()->CallJavascriptFunction(
+      "chrome.supervised_user_internals.receiveUserSettings",
+      *settings);
 }
 
 void SupervisedUserInternalsMessageHandler::OnTryURLResult(