Change manifest handler interface to always (implicitly) pass the entire manifest to handlers.

chrome/common/extensions/manifest.cc

Index: chrome/common/extensions/manifest.cc
diff --git a/chrome/common/extensions/manifest.cc b/chrome/common/extensions/manifest.cc
index cb4e1b0219173eb1ddb4b665e193a64355209150..7d93a9ebe24f0c2829bdaf8c3503621a074a547e 100644
--- a/chrome/common/extensions/manifest.cc
+++ b/chrome/common/extensions/manifest.cc
@@ -124,11 +124,21 @@ bool Manifest::GetString(
 }
 
 bool Manifest::GetDictionary(
+    const std::string& path, const DictionaryValue** out_value) const {
+  return CanAccessPath(path) && value_->GetDictionary(path, out_value);

[Matt Perry, 2013/01/29 00:58:47]

implement these in terms of the non-const versions (use temporaries)

[Yoyo Zhou, 2013/01/29 04:32:42]

Done - let me know if this casting looks reasonable.

+}
+
+bool Manifest::GetDictionary(
     const std::string& path, DictionaryValue** out_value) const {
   return CanAccessPath(path) && value_->GetDictionary(path, out_value);
 }
 
 bool Manifest::GetList(
+    const std::string& path, const ListValue** out_value) const {
+  return CanAccessPath(path) && value_->GetList(path, out_value);
+}
+
+bool Manifest::GetList(
     const std::string& path, ListValue** out_value) const {
   return CanAccessPath(path) && value_->GetList(path, out_value);
 }