Chromium Code Reviews| Index: chrome/browser/ui/webui/popular_sites_internals_message_handler.h |
| diff --git a/chrome/browser/ui/webui/popular_sites_internals_message_handler.h b/chrome/browser/ui/webui/popular_sites_internals_message_handler.h |
| index 700de05e6b5e2a5d568ca270b5e99ceeb7118dd9..7d82a5a045c096e2ffbe5509fa26b5f026334ac3 100644 |
| --- a/chrome/browser/ui/webui/popular_sites_internals_message_handler.h |
| +++ b/chrome/browser/ui/webui/popular_sites_internals_message_handler.h |
| @@ -9,20 +9,13 @@ |
| #include <string> |
| #include "base/macros.h" |
| -#include "base/memory/weak_ptr.h" |
| +#include "components/ntp_tiles/webui/popular_sites_internals_message_handler_impl.h" |
| #include "content/public/browser/web_ui_message_handler.h" |
| -namespace base { |
| -class ListValue; |
| -} // namespace base |
| - |
| -namespace ntp_tiles { |
| -class PopularSites; |
| -} // namespace ntp_tiles |
| - |
| // The implementation for the chrome://popular-sites-internals page. |
| class PopularSitesInternalsMessageHandler |
|
Bernhard Bauer
2016/10/28 16:56:38
Nit: At the risk of embiggening the class name eve
|
| - : public content::WebUIMessageHandler { |
| + : public content::WebUIMessageHandler, |
| + public ntp_tiles::PopularSitesInternalsMessageHandlerClient { |
| public: |
| PopularSitesInternalsMessageHandler(); |
| ~PopularSitesInternalsMessageHandler() override; |
| @@ -31,20 +24,18 @@ class PopularSitesInternalsMessageHandler |
| // content::WebUIMessageHandler: |
| void RegisterMessages() override; |
| - void HandleRegisterForEvents(const base::ListValue* args); |
| - void HandleUpdate(const base::ListValue* args); |
| - void HandleViewJson(const base::ListValue* args); |
| - |
| - void SendOverrides(); |
| - void SendDownloadResult(bool success); |
| - void SendSites(); |
| - void SendJson(const std::string& json); |
| - |
| - void OnPopularSitesAvailable(bool explicit_request, bool success); |
| - |
| - std::unique_ptr<ntp_tiles::PopularSites> popular_sites_; |
| - |
| - base::WeakPtrFactory<PopularSitesInternalsMessageHandler> weak_ptr_factory_; |
| + // PopularSitesInternalsMessageHandlerClient |
| + base::SequencedWorkerPool* GetBlockingPool() override; |
| + std::unique_ptr<ntp_tiles::PopularSites> MakePopularSites() override; |
| + PrefService* GetPrefs() override; |
| + void RegisterMessageCallback( |
| + const std::string& message, |
| + const base::Callback<void(const base::ListValue*)>& callback) override; |
| + void CallJavascriptFunctionVector( |
| + const std::string& name, |
| + const std::vector<const base::Value*>& values) override; |
| + |
| + ntp_tiles::PopularSitesInternalsMessageHandlerImpl impl_; |
| DISALLOW_COPY_AND_ASSIGN(PopularSitesInternalsMessageHandler); |
| }; |