Chrome diagnostics: First cut at implementing chrome://diagnostics.

chrome/browser/ui/webui/chromeos/diagnostics/diagnostics_ui.cc

Index: chrome/browser/ui/webui/chromeos/diagnostics/diagnostics_ui.cc
diff --git a/chrome/browser/ui/webui/chromeos/diagnostics/diagnostics_ui.cc b/chrome/browser/ui/webui/chromeos/diagnostics/diagnostics_ui.cc
new file mode 100644
index 0000000000000000000000000000000000000000..bd52413f2c28d8e077d69dd7d5db86e52cbd2aa3
--- /dev/null
+++ b/chrome/browser/ui/webui/chromeos/diagnostics/diagnostics_ui.cc
@@ -0,0 +1,74 @@
+// Copyright (c) 2012 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/chromeos/diagnostics/diagnostics_ui.h"
+
+#include "base/bind.h"
+#include "base/memory/weak_ptr.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/webui/chrome_web_ui_data_source.h"
+#include "chrome/common/url_constants.h"
+#include "content/public/browser/browser_thread.h"
+#include "content/public/browser/web_ui.h"
+#include "content/public/browser/web_ui_message_handler.h"
+#include "grit/browser_resources.h"
+
+namespace chromeos {
+
+namespace {
+
+////////////////////////////////////////////////////////////////////////////////
+// DiagnosticsHandler
+
+// Class to handle messages from chrome://diagnostics.
+class DiagnosticsWebUIHandler : public content::WebUIMessageHandler {
+ public:
+  DiagnosticsWebUIHandler()
+      : weak_ptr_factory_(this) {
+  }
+  virtual ~DiagnosticsWebUIHandler() { }
+
+ private:
+  // WebUIMessageHandler override.

[James Hawkins, 2012/08/01 00:47:42]

s/override/implementation/

[hshi1, 2012/08/01 00:57:10]

Done.

+  virtual void RegisterMessages() OVERRIDE;
+
+  // Called when the page is first loaded.
+  void OnPageLoaded(const base::ListValue* args);
+
+  base::WeakPtrFactory<DiagnosticsWebUIHandler> weak_ptr_factory_;
+  DISALLOW_COPY_AND_ASSIGN(DiagnosticsWebUIHandler);
+};
+
+void DiagnosticsWebUIHandler::RegisterMessages() {
+  web_ui()->RegisterMessageCallback(
+      "pageLoaded",
+      base::Bind(&DiagnosticsWebUIHandler::OnPageLoaded,
+                 weak_ptr_factory_.GetWeakPtr()));
+}
+
+void DiagnosticsWebUIHandler::OnPageLoaded(const base::ListValue* args) {
+  // TODO: invoke debugd methods to retrieve diagnostics information, and

[James Hawkins, 2012/08/01 00:47:42]

s/debugd/debug/

...right?

[hshi1, 2012/08/01 00:57:10]

Actually no the 'debugd' here refers to the debug daemon (often abbreviated as
debugd). What needs to happen here is that upon page loaded, the
DiagnosticsWebUIHandler should kick off a debugd method to get network
interfaces info asynchronously, and upon completion call a javascript function
to display the updated information. The call would look somewhat like this:

chromeos::DBusThreadManager::Get()->GetDebugDaemonClient()->
GetNetworkInterfaces(base::Bind(&DiagnosticsWebUIHandler::OnGetNetworkInterfaces,
weak_ptr_factory_.GetWeakPtr());

For the sake of clarify I plan to get the "first-cut" landed first, then
implement the functionality in a later CL.

On 2012/08/01 00:47:42, James Hawkins wrote:

+  // upon completion call javascript function to update status.
+}
+
+}  // namespace
+
+////////////////////////////////////////////////////////////////////////////////
+// DiagnosticsUI
+
+DiagnosticsUI::DiagnosticsUI(content::WebUI* web_ui)
+    : WebUIController(web_ui) {
+  web_ui->AddMessageHandler(new DiagnosticsWebUIHandler());
+
+  ChromeWebUIDataSource* source =
+      new ChromeWebUIDataSource(chrome::kChromeUIDiagnosticsHost);
+  source->add_resource_path("main.css", IDR_DIAGNOSTICS_MAIN_CSS);
+  source->add_resource_path("main.js", IDR_DIAGNOSTICS_MAIN_JS);
+  source->set_default_resource(IDR_DIAGNOSTICS_MAIN_HTML);
+
+  Profile* profile = Profile::FromWebUI(web_ui);
+  ChromeURLDataManager::AddDataSource(profile, source);
+}
+
+} // namespace chromeos