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

chrome/common/extensions/api/extension_action/browser_action_handler.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/api/extension_action/browser_action_handler.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "base/utf_string_conversions.h"
 #include "base/values.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/feature_switch.h"
+#include "chrome/common/extensions/manifest.h"
 #include "chrome/common/extensions/manifest_handler_helpers.h"
 
 namespace extensions {
 
 BrowserActionHandler::BrowserActionHandler() {
 }
 
 BrowserActionHandler::~BrowserActionHandler() {
 }
 
-bool BrowserActionHandler::Parse(const base::Value* value,
-                                 Extension* extension,
+bool BrowserActionHandler::Parse(Extension* extension,
                                  string16* error) {
   const DictionaryValue* dict = NULL;
-  if (!value->GetAsDictionary(&dict)) {
+  if (!extension->manifest()->GetDictionary(
+          extension_manifest_keys::kBrowserAction, &dict)) {
     *error = ASCIIToUTF16(extension_manifest_errors::kInvalidBrowserAction);
     return false;
   }
 
   scoped_ptr<ActionInfo> action_info =
       manifest_handler_helpers::LoadActionInfo(extension, dict, error);
   if (!action_info.get())
     return false;  // Failed to parse browser action definition.
 
   ActionInfo::SetBrowserActionInfo(extension, action_info.release());
 
   return true;
 }
 
 }  // namespace extensions