[Offline pages] Create offline internals page for Android

chrome/browser/ui/webui/offline_internals_ui.cc

+// Copyright 2016 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/offline_internals_ui.h"
+
+#include "base/bind.h"
+#include "base/bind_helpers.h"
+#include "base/macros.h"
+#include "base/memory/weak_ptr.h"
+#include "base/values.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/common/url_constants.h"
+#include "content/public/browser/web_ui.h"
+#include "content/public/browser/web_ui_controller.h"
+#include "content/public/browser/web_ui_data_source.h"
+#include "content/public/browser/web_ui_message_handler.h"
+#include "grit/components_resources.h"
+
+namespace {
+
+// Class acting as a controller of the chrome://offline-internals WebUI.
+class OfflineInternalsUIMessageHandler : public content::WebUIMessageHandler {
+ public:
+  OfflineInternalsUIMessageHandler();
+  ~OfflineInternalsUIMessageHandler() override;
+
+  // WebUIMessageHandler implementation.
+  void RegisterMessages() override;
+
+ private:
+  // Deletes all the pages in the store.
+  void DeleteAllPages(const base::ListValue* args);
+
+  // Delete selected list of page ids from the store.
+  void DeleteSelectPages(const base::ListValue* args);

[jianli, 2016/05/25 00:17:17]

nit: DeleteSelectedPages

[chili, 2016/05/25 02:14:48]

Done.

+
+  // Load all information.
+  void GetOfflineInternalsInfo(const base::ListValue* args);
+
+  // Factory for creating references in callbacks.
+  base::WeakPtrFactory<OfflineInternalsUIMessageHandler> weak_ptr_factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(OfflineInternalsUIMessageHandler);
+};
+
+OfflineInternalsUIMessageHandler::OfflineInternalsUIMessageHandler()
+    : weak_ptr_factory_(this) {}
+
+OfflineInternalsUIMessageHandler::~OfflineInternalsUIMessageHandler() {}
+
+void OfflineInternalsUIMessageHandler::DeleteAllPages(
+    const base::ListValue* args) {
+  web_ui()->CallJavascriptFunction("offlineInternals.pagesDeleted");
+}
+
+void OfflineInternalsUIMessageHandler::DeleteSelectPages(
+    const base::ListValue* args) {
+  web_ui()->CallJavascriptFunction("offlineInternals.pagesDeleted");
+}
+
+void OfflineInternalsUIMessageHandler::
+
+    void

[jianli, 2016/05/25 00:17:17]

Will this get compiled?

[chili, 2016/05/25 02:14:47]

I just did a git cl format... seemed to have messed this part up :(

+    OfflineInternalsUIMessageHandler::GetOfflineInternalsInfo(
+        const base::ListValue* args) {
+  base::DictionaryValue results;
+  base::ListValue* pages = new base::ListValue();
+  base::ListValue* queueItems = new base::ListValue();
+  results.Set("AllPages", pages);
+  results.Set("Queue", queueItems);
+
+  // Fake page
+  base::DictionaryValue* samplePage = new base::DictionaryValue();

[jianli, 2016/05/25 00:17:17]

Try not to land the CL with dummy code. You could simply keep them for your
private testing.

[chili, 2016/05/25 02:14:48]

Acknowledged.

+  pages->Append(samplePage);
+  samplePage->SetString("onlineUrl", "http://www.test.com");
+  samplePage->SetString("internalUrl", "128438sdjklre");
+  samplePage->SetString("namespace", "bookmark");
+  samplePage->SetInteger("size", 300);
+
+  // Fake queue
+  base::DictionaryValue* sampleQueueItem = new base::DictionaryValue();
+  queueItems->Append(sampleQueueItem);
+  sampleQueueItem->SetString("onlineUrl", "http://www.google.com");
+  sampleQueueItem->SetString("status", "pending");
+  sampleQueueItem->SetString("creation time", "5:00:32AM");
+
+  web_ui()->CallJavascriptFunction("offlineInternals.setOfflineInternalsInfo",
+                                   results);
+}
+
+void OfflineInternalsUIMessageHandler::RegisterMessages() {
+  web_ui()->RegisterMessageCallback(
+      "deleteAllPages",
+      base::Bind(&OfflineInternalsUIMessageHandler::DeleteAllPages,
+                 weak_ptr_factory_.GetWeakPtr()));
+  web_ui()->RegisterMessageCallback(
+      "deleteSelectPages",
+      base::Bind(&OfflineInternalsUIMessageHandler::DeleteSelectPages,
+                 weak_ptr_factory_.GetWeakPtr()));
+  web_ui()->RegisterMessageCallback(
+      "getOfflineInternalsInfo",
+      base::Bind(&OfflineInternalsUIMessageHandler::GetOfflineInternalsInfo,
+                 weak_ptr_factory_.GetWeakPtr()));
+}
+
+}  // namespace
+
+OfflineInternalsUI::OfflineInternalsUI(content::WebUI* web_ui)
+    : content::WebUIController(web_ui) {
+  // chrome://offline-internals source.
+  content::WebUIDataSource* html_source =
+      content::WebUIDataSource::Create(chrome::kChromeUIOfflineInternalsHost);
+
+  // Required resources

[jianli, 2016/05/25 00:17:17]

nit: end with period

[chili, 2016/05/25 02:14:48]

Done.

+  html_source->SetJsonPath("strings.js");
+  html_source->AddResourcePath("offline_internals.css",
+                               IDR_OFFLINE_INTERNALS_CSS);
+  html_source->AddResourcePath("offline_internals.js",
+                               IDR_OFFLINE_INTERNALS_JS);
+  html_source->SetDefaultResource(IDR_OFFLINE_INTERNALS_HTML);
+
+  content::WebUIDataSource::Add(Profile::FromWebUI(web_ui), html_source);
+
+  web_ui->AddMessageHandler(new OfflineInternalsUIMessageHandler());
+}
+
+OfflineInternalsUI::~OfflineInternalsUI() {}