Move ExtensionAction from common/ to browser/.

chrome/browser/extensions/api/extension_action/extension_actions_api.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/browser/extensions/api/extension_action/extension_actions_api.h"
 
 #include <string>
 
 #include "base/base64.h"
 #include "base/string_number_conversions.h"
 #include "base/string_piece.h"
 #include "base/values.h"
 #include "chrome/browser/extensions/api/extension_action/extension_page_actions_api_constants.h"
+#include "chrome/browser/extensions/extension_action.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_system.h"
 #include "chrome/browser/extensions/extension_tab_util.h"
 #include "chrome/browser/extensions/location_bar_controller.h"
 #include "chrome/browser/extensions/state_store.h"
 #include "chrome/browser/extensions/tab_helper.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/tab_contents/tab_contents.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/extensions/extension.h"
-#include "chrome/common/extensions/extension_action.h"
 #include "chrome/common/extensions/extension_error_utils.h"
 #include "chrome/common/render_messages.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/notification_service.h"
 
 namespace {
 
 const char kBrowserActionStorageKey[] = "browser_action";
 const char kPopupUrlStorageKey[] = "poupup_url";
 const char kTitleStorageKey[] = "title";
@@ skipping 198 matching lines @@
 
   scoped_ptr<base::DictionaryValue> defaults =
       DefaultsToValue(extension_action);
   storage->SetExtensionValue(extension_action->extension_id(),
                              kBrowserActionStorageKey,
                              defaults.PassAs<base::Value>());
 }
 
 void ExtensionActionStorageManager::ReadFromStorage(
     const std::string& extension_id, scoped_ptr<base::Value> value) {
-  const Extension* extension =
-      ExtensionSystem::Get(profile_)->extension_service()->
-          GetExtensionById(extension_id, true);
+  const ExtensionService* service =
+      ExtensionSystem::Get(profile_)->extension_service();
+  const Extension* extension = service->GetExtensionById(extension_id, true);
   if (!extension)
     return;
 
-  CHECK(extension->browser_action());
+  ExtensionAction* browser_action = service->GetBrowserAction(*extension);
+  CHECK(browser_action);
 
   // Don't load values from storage if the extension has updated a value
   // already. The extension may have only updated some of the values, but
   // this is a good first approximation. If the extension is doing stuff
   // to the browser action, we can assume it is ready to take over.
-  if (extension->browser_action()->has_changed())
+  if (browser_action->has_changed())
     return;
 
   const base::DictionaryValue* dict = NULL;
   if (!value.get() || !value->GetAsDictionary(&dict))
     return;
 
-  SetDefaultsFromValue(dict, extension->browser_action());
+  SetDefaultsFromValue(dict, browser_action);
 }
 
 }  // namespace extensions
 
 
 //
 // ExtensionActionFunction
 //
 
 ExtensionActionFunction::ExtensionActionFunction()
     : details_(NULL),
       tab_id_(ExtensionAction::kDefaultTabId),
       contents_(NULL),
       extension_action_(NULL) {
 }
 
 ExtensionActionFunction::~ExtensionActionFunction() {
 }
 
 bool ExtensionActionFunction::RunImpl() {
   if (base::StringPiece(name()).starts_with("scriptBadge.")) {
-    extension_action_ = GetExtension()->script_badge();
+    extension_action_ = GetScriptBadge(profile(), *GetExtension());
   } else {
-    extension_action_ = GetExtension()->browser_action();
+    extension_action_ = GetBrowserAction(profile(), *GetExtension());
     if (!extension_action_)
-      extension_action_ = GetExtension()->page_action();
+      extension_action_ = GetPageAction(profile(), *GetExtension());
   }
   if (!extension_action_) {
     // TODO(kalman): ideally the browserAction/pageAction APIs wouldn't event
     // exist for extensions that don't have one declared. This should come as
     // part of the Feature system.
     error_ = kNoExtensionActionError;
     return false;
   }
 
   // There may or may not be details (depends on the function).
@@ skipping 46 matching lines @@
       error_ = ExtensionErrorUtils::FormatErrorMessage(
           kNoTabError, base::IntToString(tab_id_));
       return false;
     }
   }
 
   return RunExtensionAction();
 }
 
 void ExtensionActionFunction::NotifyChange() {
+  typedef extensions::Extension::ActionInfo ActionInfo;
   switch (extension_action_->action_type()) {
-    case ExtensionAction::TYPE_BROWSER:
-    case ExtensionAction::TYPE_PAGE:
+    case ActionInfo::TYPE_BROWSER:
+    case ActionInfo::TYPE_PAGE:
       if (extension_->browser_action()) {
         NotifyBrowserActionChange();
       } else if (extension_->page_action()) {
         NotifyLocationBarChange();
       }
       return;
-    case ExtensionAction::TYPE_SCRIPT_BADGE:
+    case ActionInfo::TYPE_SCRIPT_BADGE:
       NotifyLocationBarChange();
       return;
   }
   NOTREACHED();
 }
 
 void ExtensionActionFunction::NotifyBrowserActionChange() {
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_EXTENSION_BROWSER_ACTION_UPDATED,
       content::Source<ExtensionAction>(extension_action_),
@@ skipping 178 matching lines @@
 bool ExtensionActionGetBadgeBackgroundColorFunction::RunExtensionAction() {
   ListValue* list = new ListValue();
   SkColor color = extension_action_->GetBadgeBackgroundColor(tab_id_);
   list->Append(Value::CreateIntegerValue(SkColorGetR(color)));
   list->Append(Value::CreateIntegerValue(SkColorGetG(color)));
   list->Append(Value::CreateIntegerValue(SkColorGetB(color)));
   list->Append(Value::CreateIntegerValue(SkColorGetA(color)));
   SetResult(list);
   return true;
 }