Modifications to performance monitor UI to address real webui.

chrome/browser/ui/webui/performance_monitor/web_ui.cc

Index: chrome/browser/ui/webui/performance_monitor/web_ui.cc
diff --git a/chrome/browser/ui/webui/performance_monitor/web_ui.cc b/chrome/browser/ui/webui/performance_monitor/web_ui.cc
new file mode 100644
index 0000000000000000000000000000000000000000..029795b378a6bcad3fac55d78f17a0ef57cdb75b
--- /dev/null
+++ b/chrome/browser/ui/webui/performance_monitor/web_ui.cc
@@ -0,0 +1,44 @@
+// 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/performance_monitor/web_ui.h"
+
+#include "base/values.h"
+#include "chrome/browser/performance_monitor/performance_monitor.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/webui/chrome_url_data_manager.h"
+#include "chrome/browser/ui/webui/chrome_web_ui_data_source.h"
+#include "chrome/browser/ui/webui/performance_monitor/web_ui_handler.h"
+#include "chrome/common/url_constants.h"
+#include "content/public/browser/web_ui.h"
+#include "grit/browser_resources.h"
+#include "grit/generated_resources.h"
+
+namespace {
+
+ChromeWebUIDataSource* CreateWebUIHTMLSource() {
+  ChromeWebUIDataSource* source =
+      new ChromeWebUIDataSource(chrome::kChromeUIPerformanceMonitorHost);
+
+  source->add_resource_path("chart.js", IDR_PERFORMANCE_MONITOR_CHART_JS);
+  source->add_resource_path("chart.css", IDR_PERFORMANCE_MONITOR_CHART_CSS);

[Evan Stade, 2012/07/28 01:01:16]

I don't think you need this these two (chart.*). Grit will inline them since you
have flattenhtml.

[clintstaley, 2012/07/31 02:34:32]

I don't actually have flattenhtml on both, tho, so I added that.  Any quick
advice on when to use this option and when not?  A "flattenhtml" search on the
dev site produces no result and a general grep on the source tree doesn't show
any inline docs on it (just lots of uses)

Also, I found I got errors w/o the chart.js add_resource_path, even with
flattenhtml.  Was able to eliminate the .css call, though.

[Evan Stade, 2012/07/31 22:57:02]

flattenhtml inlines references. You only need flattenhtml on the html file.
Otherwise what you have looks fine.

+  source->add_resource_path("jquery.js", IDR_PERFORMANCE_MONITOR_JQUERY_JS);
+  source->add_resource_path("flot.js", IDR_PERFORMANCE_MONITOR_JQUERY_FLOT_JS);
+  source->set_default_resource(IDR_PERFORMANCE_MONITOR_HTML);
+  return source;
+}
+
+}  // namespace
+
+namespace performance_monitor {
+
+WebUI::WebUI(content::WebUI* web_ui) : WebUIController(web_ui) {
+  web_ui->AddMessageHandler(new performance_monitor::WebUIHandler());
+
+  ChromeWebUIDataSource* html_source = CreateWebUIHTMLSource();
+  Profile* profile = Profile::FromWebUI(web_ui);
+  ChromeURLDataManager::AddDataSource(profile, html_source);
+}
+
+}  // namespace performance_monitor