Add ErrorConsole UI for Extension Install Warnings

chrome/browser/ui/webui/extensions/extension_error_handler.h

Index: chrome/browser/ui/webui/extensions/extension_error_handler.h
diff --git a/chrome/browser/ui/webui/extensions/extension_error_handler.h b/chrome/browser/ui/webui/extensions/extension_error_handler.h
new file mode 100644
index 0000000000000000000000000000000000000000..2bda914087714ab70ff9c33789fdfb23bfe801bc
--- /dev/null
+++ b/chrome/browser/ui/webui/extensions/extension_error_handler.h
@@ -0,0 +1,63 @@
+// Copyright 2013 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.
+
+#ifndef CHROME_BROWSER_UI_WEBUI_EXTENSIONS_EXTENSION_ERROR_HANDLER_H_
+#define CHROME_BROWSER_UI_WEBUI_EXTENSIONS_EXTENSION_ERROR_HANDLER_H_
+
+#include <string>
+
+#include "base/basictypes.h"
+#include "base/compiler_specific.h"
+#include "content/public/browser/web_ui_message_handler.h"
+
+namespace base {
+class DictionaryValue;
+class ListValue;
+}
+
+namespace content {
+class WebUIDataSource;
+}
+
+class Profile;
+
+namespace extensions {
+
+class Extension;
+
+// The handler page for the Extension Commands UI overlay.
+class ExtensionErrorHandler : public content::WebUIMessageHandler {
+ public:
+  explicit ExtensionErrorHandler(Profile* profile);
+  virtual ~ExtensionErrorHandler();
+
+  // Fetches the localized values for the page and deposits them into |source|.
+  void GetLocalizedValues(content::WebUIDataSource* source);
+
+  // WebUIMessageHandler implementation.
+  virtual void RegisterMessages() OVERRIDE;
+
+ private:
+  friend class ManifestHighlightUnitTest;
+
+  // Handle the "requestFileSource" call.
+  void HandleRequestFileSource(const base::ListValue* args);
+
+  // Populate the results for a manifest file's content in response to the
+  // "requestFileSource" call. Highlight the part of the manifest which
+  // corresponds to the given |key| and |specific| locations.

[Dan Beam, 2013/08/29 00:24:30]

nit: mention ownership model of |dict| and |contents| (it's not apparent from
looking at the header if they're in-params [which should be const or taken
ownership of] or out-params [which should be overwritten])

[Devlin, 2013/08/29 21:02:04]

Done.

+  void GetManifestFileCallback(base::DictionaryValue* dict,
+                               const std::string& key,
+                               const std::string& specific,
+                               std::string* contents);
+
+  // The profile with which this Handler is associated.
+  Profile* profile_;
+
+  DISALLOW_COPY_AND_ASSIGN(ExtensionErrorHandler);
+};
+
+}  // namespace extensions
+
+#endif  // CHROME_BROWSER_UI_WEBUI_EXTENSIONS_EXTENSION_ERROR_HANDLER_H_