Add content-security-policy directive to chrome://crashes page.

chrome/browser/ui/webui/crashes_ui.cc

 // Copyright (c) 2011 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/crashes_ui.h"
 
 #include "base/i18n/time_formatting.h"
 #include "base/memory/ref_counted_memory.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/crash_upload_list.h"
 #include "chrome/browser/prefs/pref_service.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/common/chrome_version_info.h"
 #include "chrome/common/jstemplate_builder.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/url_constants.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "grit/browser_resources.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 
 #if defined(OS_CHROMEOS)
 #include "chrome/browser/chromeos/metrics_cros_settings_provider.h"
 #endif
 
 namespace {
 
+const char kStringsJsPath[] = "strings.js";
+const char kCrashesJsPath[] = "crashes.js";
+
 ///////////////////////////////////////////////////////////////////////////////
 //
 // CrashesUIHTMLSource
 //
 ///////////////////////////////////////////////////////////////////////////////
 
-class CrashesUIHTMLSource : public ChromeURLDataManager::DataSource {
+class CrashesUIHTMLSource : public ChromeWebUIDataSource {
  public:
-  CrashesUIHTMLSource()
-      : DataSource(chrome::kChromeUICrashesHost, MessageLoop::current()) {}
-
-  // Called when the network layer has requested a resource underneath
-  // the path we registered.
+  CrashesUIHTMLSource();
   virtual void StartDataRequest(const std::string& path,
                                 bool is_incognito,
                                 int request_id);
-  virtual std::string GetMimeType(const std::string&) const {
-    return "text/html";
-  }
+  virtual std::string GetMimeType(const std::string& path) const;
 
  private:
   ~CrashesUIHTMLSource() {}
-
   DISALLOW_COPY_AND_ASSIGN(CrashesUIHTMLSource);
 };
 
+
+CrashesUIHTMLSource::CrashesUIHTMLSource()
+  : ChromeWebUIDataSource(chrome::kChromeUICrashesHost) {
+  AddLocalizedString("crashesTitle",IDS_CRASHES_TITLE);
+  AddLocalizedString("crashCountFormat",
+                     IDS_CRASHES_CRASH_COUNT_BANNER_FORMAT);
+  AddLocalizedString("crashHeaderFormat", IDS_CRASHES_CRASH_HEADER_FORMAT);
+  AddLocalizedString("crashTimeFormat", IDS_CRASHES_CRASH_TIME_FORMAT);
+  AddLocalizedString("bugLinkText", IDS_CRASHES_BUG_LINK_LABEL);
+  AddLocalizedString("noCrashesMessage", IDS_CRASHES_NO_CRASHES_MESSAGE);
+  AddLocalizedString("disabledHeader", IDS_CRASHES_DISABLED_HEADER);
+  AddLocalizedString("disabledMessage", IDS_CRASHES_DISABLED_MESSAGE);
+}
+
 void CrashesUIHTMLSource::StartDataRequest(const std::string& path,

[Evan Stade, 2011/06/30 23:25:23]

didn't you change this interface recently

                                            bool is_incognito,
                                            int request_id) {
-  DictionaryValue localized_strings;
-  localized_strings.SetString("crashesTitle",
-      l10n_util::GetStringUTF16(IDS_CRASHES_TITLE));
-  localized_strings.SetString("crashCountFormat",
-      l10n_util::GetStringUTF16(IDS_CRASHES_CRASH_COUNT_BANNER_FORMAT));
-  localized_strings.SetString("crashHeaderFormat",
-      l10n_util::GetStringUTF16(IDS_CRASHES_CRASH_HEADER_FORMAT));
-  localized_strings.SetString("crashTimeFormat",
-      l10n_util::GetStringUTF16(IDS_CRASHES_CRASH_TIME_FORMAT));
-  localized_strings.SetString("bugLinkText",
-      l10n_util::GetStringUTF16(IDS_CRASHES_BUG_LINK_LABEL));
-  localized_strings.SetString("noCrashesMessage",
-      l10n_util::GetStringUTF16(IDS_CRASHES_NO_CRASHES_MESSAGE));
-  localized_strings.SetString("disabledHeader",
-      l10n_util::GetStringUTF16(IDS_CRASHES_DISABLED_HEADER));
-  localized_strings.SetString("disabledMessage",
-      l10n_util::GetStringUTF16(IDS_CRASHES_DISABLED_MESSAGE));
+  if (path == kStringsJsPath) {
+    SendLocalizedStringsAsJSON(request_id);
+  } else {
+    int idr = (path == kCrashesJsPath) ? IDR_CRASHES_JS : IDR_CRASHES_HTML;

[Evan Stade, 2011/06/30 23:25:23]

no ()

+    SendFromResourceBundle(request_id, idr);
+  }
+}
 
-  ChromeURLDataManager::DataSource::SetFontAndTextDirection(&localized_strings);
+std::string CrashesUIHTMLSource::GetMimeType(const std::string& path) const {
+  if (path == kCrashesJsPath || path == kStringsJsPath)
+    return "application/javascript";
 
-  static const base::StringPiece crashes_html(
-      ResourceBundle::GetSharedInstance().GetRawDataResource(IDR_CRASHES_HTML));
-  std::string full_html =
-      jstemplate_builder::GetI18nTemplateHtml(crashes_html, &localized_strings);
-  jstemplate_builder::AppendJsTemplateSourceHtml(&full_html);
-
-  scoped_refptr<RefCountedBytes> html_bytes(new RefCountedBytes);
-  html_bytes->data.resize(full_html.size());
-  std::copy(full_html.begin(), full_html.end(), html_bytes->data.begin());
-
-  SendResponse(request_id, html_bytes);
+  return "text/html";
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 //
 // CrashesDOMHandler
 //
 ////////////////////////////////////////////////////////////////////////////////
 
 // The handler for Javascript messages for the chrome://crashes/ page.
 class CrashesDOMHandler : public WebUIMessageHandler,
@@ skipping 109 matching lines @@
 bool CrashesUI::CrashReportingEnabled() {
 #if defined(GOOGLE_CHROME_BUILD) && !defined(OS_CHROMEOS)
   PrefService* prefs = g_browser_process->local_state();
   return prefs->GetBoolean(prefs::kMetricsReportingEnabled);
 #elif defined(GOOGLE_CHROME_BUILD) && defined(OS_CHROMEOS)
   return chromeos::MetricsCrosSettingsProvider::GetMetricsStatus();
 #else
   return false;
 #endif
 }