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

chrome/common/extensions/manifest_tests/extension_manifests_devtools_unittest.cc

 // 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/common/extensions/manifest_tests/extension_manifest_test.h"
-
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_manifest_constants.h"
+#include "chrome/common/extensions/manifest_handler.h"
+#include "chrome/common/extensions/manifest_tests/extension_manifest_test.h"
+#include "chrome/common/extensions/manifest_url_handler.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
+namespace keys = extension_manifest_keys;
 namespace errors = extension_manifest_errors;
 
-TEST_F(ExtensionManifestTest, DevToolsExtensions) {
+namespace {

[Yoyo Zhou, 2013/01/03 23:11:44]

By the way, I didn't notice this earlier - I don't think this is common
practice, so I wouldn't do it. (But feel free to point out counterexamples.)

[Joe Thomas, 2013/01/04 00:34:59]

I remove the namespace. I added that because I remember seeing such a usage
somewhere, but not able to recall. 

+
+class DevToolsPageManifestTest : public ExtensionManifestTest {
+  virtual void SetUp() OVERRIDE {
+    ExtensionManifestTest::SetUp();
+    extensions::ManifestHandler::Register(keys::kDevToolsPage,
+                                          new extensions::DevToolsPageHandler);
+  }
+};
+
+TEST_F(DevToolsPageManifestTest, DevToolsExtensions) {
   LoadAndExpectError("devtools_extension_url_invalid_type.json",
                      errors::kInvalidDevToolsPage);
 
   scoped_refptr<extensions::Extension> extension;
   extension = LoadAndExpectSuccess("devtools_extension.json");
   EXPECT_EQ(extension->url().spec() + "devtools.html",
-            extension->devtools_url().spec());
+            extensions::ManifestURL::GetDevToolsPage(extension).spec());
   EXPECT_TRUE(extension->HasEffectiveAccessToAllHosts());
 }
+
+}  // namespace