Move the parsing of homepage_url" and "devtools_page" out of Extension.

chrome/renderer/chrome_content_renderer_client.cc

Index: chrome/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index eb60f9523fd2d533c6b2cb0182ce362273535f6c..c180d61e20042aadbcbfa011f434ade1e6aa57f1 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -19,10 +19,13 @@
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/content_settings_pattern.h"
+#include "chrome/common/extensions/api/url_parser/devtools_page_handler.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_constants.h"
+#include "chrome/common/extensions/extension_manifest_constants.h"
 #include "chrome/common/extensions/extension_process_policy.h"
 #include "chrome/common/extensions/extension_set.h"
+#include "chrome/common/extensions/manifest_handler.h"
 #include "chrome/common/external_ipc_fuzzer.h"
 #include "chrome/common/jstemplate_builder.h"
 #include "chrome/common/localized_error.h"
@@ -117,6 +120,14 @@ using WebKit::WebVector;
 
 namespace {
 
+// Explicitly register all ManifestHandlers, since the subprocess won't have the
+// same ManifestHandlerRegistry.
+void RegisterManifestHandlers() {

[Yoyo Zhou, 2012/12/27 17:16:17]

This should probably be called RegisterExtensionManifestHandlers. The comment
should say "all extension ManifestHandlers needed to parse fields used in the
renderer."

[Joe Thomas, 2012/12/30 04:26:00]

Done.

+  extensions::ManifestHandler::Register(
+      extension_manifest_keys::kDevToolsPage,
+      new extensions::DevToolsPageHandler);
+}
+
 static void AppendParams(const std::vector<string16>& additional_names,
                          const std::vector<string16>& additional_values,
                          WebVector<WebString>* existing_names,
@@ -258,6 +269,8 @@ void ChromeContentRendererClient::RenderThreadStarted() {
       extension_scheme);
   WebSecurityPolicy::registerURLSchemeAsBypassingContentSecurityPolicy(
       extension_resource_scheme);
+
+  RegisterManifestHandlers();
 }
 
 void ChromeContentRendererClient::RenderViewCreated(