Allow manifest handlers to declare keys they depend on that must be parsed before them.

chrome/common/extensions/manifest_tests/extension_manifests_override_unittest.cc

Index: chrome/common/extensions/manifest_tests/extension_manifests_override_unittest.cc
diff --git a/chrome/common/extensions/manifest_tests/extension_manifests_override_unittest.cc b/chrome/common/extensions/manifest_tests/extension_manifests_override_unittest.cc
index 9e4489d2e74d6519818e4010e7b418a6cf90b29a..16200c2eba7a447c50bbce3f189f029eb23aeccd 100644
--- a/chrome/common/extensions/manifest_tests/extension_manifests_override_unittest.cc
+++ b/chrome/common/extensions/manifest_tests/extension_manifests_override_unittest.cc
@@ -13,8 +13,9 @@ namespace errors = extension_manifest_errors;
 class URLOverridesManifestTest : public ExtensionManifestTest {
   virtual void SetUp() OVERRIDE {
     ExtensionManifestTest::SetUp();
-    extensions::ManifestHandler::Register(keys::kChromeURLOverrides,
-                                          new extensions::URLOverridesHandler);
+    extensions::ManifestHandler::Register(
+        keys::kChromeURLOverrides,  // TODO(yoz): ???

[Matt Perry, 2013/02/01 23:22:37]

What does this mean?

[Yoyo Zhou, 2013/02/02 00:32:22]

It means this only works if there is a namespace alias declared in one of the
included header files, which seems wrong, and I was going to find that and fix
it, which I have now done.

+        make_linked_ptr(new extensions::URLOverridesHandler));
   }
 };