Convert SupervisedUserResourceThrottle to a NavigationThrottle.

chrome/browser/ui/webui/supervised_user_internals_message_handler.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_SUPERVISED_USER_INTERNALS_MESSAGE_HANDLER_H_
 #define CHROME_BROWSER_UI_WEBUI_SUPERVISED_USER_INTERNALS_MESSAGE_HANDLER_H_
 
 #include "base/callback_list.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/supervised_user/supervised_user_service_observer.h"
 #include "chrome/browser/supervised_user/supervised_user_url_filter.h"
 #include "components/supervised_user_error_page/supervised_user_error_page.h"
 #include "content/public/browser/web_ui_message_handler.h"
 
 namespace base {
 class ListValue;
 }  // namespace base
 
 class SupervisedUserService;
 
 // The implementation for the chrome://supervised-user-internals page.
 class SupervisedUserInternalsMessageHandler
     : public content::WebUIMessageHandler,
-      public SupervisedUserServiceObserver {
+      public SupervisedUserServiceObserver,
+      public SupervisedUserURLFilter::Observer {
  public:
   SupervisedUserInternalsMessageHandler();
   ~SupervisedUserInternalsMessageHandler() override;
 
  private:
-  class IOThreadHelper;
-
   // content::WebUIMessageHandler:
   void RegisterMessages() override;
 
   // SupervisedUserServiceObserver:
   void OnURLFilterChanged() override;
 
   SupervisedUserService* GetSupervisedUserService();
 
   void HandleRegisterForEvents(const base::ListValue* args);
   void HandleGetBasicInfo(const base::ListValue* args);
   void HandleTryURL(const base::ListValue* args);
 
   void SendBasicInfo();
   void SendSupervisedUserSettings(const base::DictionaryValue* settings);
 
   void OnTryURLResult(
       const std::map<std::string, base::string16>& whitelists,
       SupervisedUserURLFilter::FilteringBehavior behavior,
       supervised_user_error_page::FilteringBehaviorReason reason,
       bool uncertain);
 
+  // SupervisedUserURLFilter::Observer:
+  void OnSiteListUpdated() override;
   void OnURLChecked(const GURL& url,
                     SupervisedUserURLFilter::FilteringBehavior behavior,
                     supervised_user_error_page::FilteringBehaviorReason reason,
-                    bool uncertain);
+                    bool uncertain) override;
 
   std::unique_ptr<
       base::CallbackList<void(const base::DictionaryValue*)>::Subscription>
       user_settings_subscription_;
 
-  scoped_refptr<IOThreadHelper> io_thread_helper_;
+  bool is_watching_filter_;

[Marc Treib, 2017/03/28 10:22:58]

Would a ScopedObserver work here?

[mmenke, 2017/03/28 17:55:05]

I'd never heard of that class before.  Seems a bit like over-engineering to me,
but done.

 
   base::WeakPtrFactory<SupervisedUserInternalsMessageHandler> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(SupervisedUserInternalsMessageHandler);
 };
 
 #endif  // CHROME_BROWSER_UI_WEBUI_SUPERVISED_USER_INTERNALS_MESSAGE_HANDLER_H_