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

chrome/browser/ui/webui/supervised_user_internals_message_handler.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/webui/supervised_user_internals_message_handler.h"
 
 #include "base/bind.h"
 #include "base/memory/ref_counted.h"
 #include "base/values.h"
 #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"
 
 using content::BrowserThread;
 
 namespace {
 
 // Creates a 'section' for display on about:supervised-user-internals,
 // consisting of a title and a list of fields. Returns a pointer to the new
@@ skipping 216 matching lines @@
   AddSectionEntry(section_filter, "Online checks active",
                   filter->HasAsyncURLChecker());
   AddSectionEntry(section_filter, "Default behavior",
                   FilteringBehaviorToString(
                       filter->GetDefaultFilteringBehavior()));
 
   base::DictionaryValue result;
   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);
+  // TODO: Change interface to SupervisedUserSettingsService to allow

[Bernhard Bauer, 2015/08/18 15:39:23]

Nit: Write TODOs in the form TODO(ldap), so it's easier to grep for them. And
BTW, the ldap doesn't actually have to be yours; it's the person who can be
contacted about the TODO, so feel free to make it TODO(bauerb).

+  // unsubscription, otherwise the subscription remains after this
+  // object is destroyed resulting in a small memory leak.
+  settings_service->Subscribe(base::Bind(
+        &SupervisedUserInternalsMessageHandler::SendSupervisedUserSettings,
+        weak_factory_.GetWeakPtr()));
+}
+
+void SupervisedUserInternalsMessageHandler::SendSupervisedUserSettings(
+    const base::DictionaryValue* settings){
+  web_ui()->CallJavascriptFunction(
+      "chrome.supervised_user_internals.receiveUserSettings",
+      *settings);
 }
 
 void SupervisedUserInternalsMessageHandler::OnTryURLResult(
     SupervisedUserURLFilter::FilteringBehavior behavior,
     SupervisedUserURLFilter::FilteringBehaviorReason reason,
     bool uncertain) {
   web_ui()->CallJavascriptFunction(
       "chrome.supervised_user_internals.receiveTryURLResult",
       base::StringValue(FilteringBehaviorToString(behavior, uncertain)));
 }
 
 void SupervisedUserInternalsMessageHandler::OnURLChecked(
     const GURL& url,
     SupervisedUserURLFilter::FilteringBehavior behavior,
     SupervisedUserURLFilter::FilteringBehaviorReason reason,
     bool uncertain) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   base::DictionaryValue result;
   result.SetString("url", url.possibly_invalid_spec());
   result.SetString("result", FilteringBehaviorToString(behavior, uncertain));
   result.SetString("reason", FilteringBehaviorReasonToString(reason));
   web_ui()->CallJavascriptFunction(
       "chrome.supervised_user_internals.receiveFilteringResult", result);
 }