Move Extension Location and Type enums to Manifest, and move InstallWarning to its own file.

chrome/common/extensions/manifest.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.h"
 
 #include "base/basictypes.h"
 #include "base/lazy_instance.h"
 #include "base/logging.h"
 #include "base/string_split.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/common/extensions/extension_manifest_constants.h"
 #include "chrome/common/extensions/features/base_feature_provider.h"
 #include "extensions/common/error_utils.h"
 
 namespace errors = extension_manifest_errors;
 namespace keys = extension_manifest_keys;
 
 namespace extensions {
 
-Manifest::Manifest(Extension::Location location,
+namespace {
+
+// Rank extension locations in a way that allows
+// Manifest::GetHigherPriorityLocation() to compare locations.
+// An extension installed from two locations will have the location
+// with the higher rank, as returned by this function. The actual
+// integer values may change, and should never be persisted.
+int GetLocationRank(Manifest::Location location) {
+  const int kInvalidRank = -1;
+  int rank = kInvalidRank;  // Will CHECK that rank is not kInvalidRank.
+
+  switch (location) {
+    // Component extensions can not be overriden by any other type.
+    case Manifest::COMPONENT:
+      rank = 6;
+      break;
+
+    // Policy controlled extensions may not be overridden by any type
+    // that is not part of chrome.
+    case Manifest::EXTERNAL_POLICY_DOWNLOAD:
+      rank = 5;
+      break;
+
+    // A developer-loaded extension should override any installed type
+    // that a user can disable.
+    case Manifest::LOAD:
+      rank = 4;
+      break;
+
+    // The relative priority of various external sources is not important,
+    // but having some order ensures deterministic behavior.
+    case Manifest::EXTERNAL_REGISTRY:
+      rank = 3;
+      break;
+
+    case Manifest::EXTERNAL_PREF:
+      rank = 2;
+      break;
+
+    case Manifest::EXTERNAL_PREF_DOWNLOAD:
+      rank = 1;
+      break;
+
+    // User installed extensions are overridden by any external type.
+    case Manifest::INTERNAL:
+      rank = 0;
+      break;
+
+    default:
+      NOTREACHED() << "Need to add new extension locaton " << location;
+  }
+
+  CHECK(rank != kInvalidRank);
+  return rank;
+}
+
+}  // namespace
+
+// static
+Manifest::Location Manifest::GetHigherPriorityLocation(
+    Manifest::Location loc1, Manifest::Location loc2) {
+  if (loc1 == loc2)
+    return loc1;
+
+  int loc1_rank = GetLocationRank(loc1);
+  int loc2_rank = GetLocationRank(loc2);
+
+  // If two different locations have the same rank, then we can not
+  // deterministicly choose a location.
+  CHECK(loc1_rank != loc2_rank);
+
+  // Highest rank has highest priority.
+  return (loc1_rank > loc2_rank ? loc1 : loc2 );
+}
+
+// TODO(yoz): remove extraneous Manifest::

[Jeffrey Yasskin, 2013/01/29 22:54:07]

Why can't you do it now?

[Yoyo Zhou, 2013/01/30 01:16:01]

I left this here so I would remember to actually do it.

+Manifest::Manifest(Manifest::Location location,
                    scoped_ptr<DictionaryValue> value)
     : location_(location),
       value_(value.Pass()),
-      type_(Extension::TYPE_UNKNOWN) {
+      type_(Manifest::TYPE_UNKNOWN) {
   if (value_->HasKey(keys::kTheme)) {
-    type_ = Extension::TYPE_THEME;
+    type_ = Manifest::TYPE_THEME;
   } else if (value_->HasKey(keys::kApp)) {
     if (value_->Get(keys::kWebURLs, NULL) ||
         value_->Get(keys::kLaunchWebURL, NULL)) {
-      type_ = Extension::TYPE_HOSTED_APP;
+      type_ = Manifest::TYPE_HOSTED_APP;
     } else if (value_->Get(keys::kPlatformAppBackground, NULL)) {
-      type_ = Extension::TYPE_PLATFORM_APP;
+      type_ = Manifest::TYPE_PLATFORM_APP;
     } else {
-      type_ = Extension::TYPE_LEGACY_PACKAGED_APP;
+      type_ = Manifest::TYPE_LEGACY_PACKAGED_APP;
     }
   } else {
-    type_ = Extension::TYPE_EXTENSION;
+    type_ = Manifest::TYPE_EXTENSION;
   }
-  CHECK_NE(type_, Extension::TYPE_UNKNOWN);
+  CHECK_NE(type_, Manifest::TYPE_UNKNOWN);
 }
 
 Manifest::~Manifest() {
 }
 
 void Manifest::ValidateManifest(
     std::string* error,
-    Extension::InstallWarningVector* warnings) const {
+    InstallWarning::Vector* warnings) const {
   *error = "";
-  if (type_ == Extension::TYPE_PLATFORM_APP && GetManifestVersion() < 2) {
+  if (type_ == Manifest::TYPE_PLATFORM_APP && GetManifestVersion() < 2) {
     *error = errors::kPlatformAppNeedsManifestVersion2;
     return;
   }
 
   // Check every feature to see if its in the manifest. Note that this means
   // we will ignore keys that are not features; we do this for forward
   // compatibility.
   // TODO(aa): Consider having an error here in the case of strict error
   // checking to let developers know when they screw up.
 
   std::set<std::string> feature_names =
       BaseFeatureProvider::GetManifestFeatures()->GetAllFeatureNames();
   for (std::set<std::string>::iterator feature_name = feature_names.begin();
        feature_name != feature_names.end(); ++feature_name) {
     // Use Get instead of HasKey because the former uses path expansion.
     if (!value_->Get(*feature_name, NULL))
       continue;
 
     Feature* feature =
         BaseFeatureProvider::GetManifestFeatures()->GetFeature(*feature_name);
     Feature::Availability result = feature->IsAvailableToManifest(
         extension_id_, type_, Feature::ConvertLocation(location_),
         GetManifestVersion());
     if (!result.is_available())
-      warnings->push_back(Extension::InstallWarning(
-          Extension::InstallWarning::FORMAT_TEXT, result.message()));
+      warnings->push_back(InstallWarning(
+          InstallWarning::FORMAT_TEXT, result.message()));
   }
 
   // Also generate warnings for keys that are not features.
   for (DictionaryValue::key_iterator key = value_->begin_keys();
       key != value_->end_keys(); ++key) {
     if (!BaseFeatureProvider::GetManifestFeatures()->GetFeature(*key)) {
-      warnings->push_back(Extension::InstallWarning(
-          Extension::InstallWarning::FORMAT_TEXT,
+      warnings->push_back(InstallWarning(
+          InstallWarning::FORMAT_TEXT,
           base::StringPrintf("Unrecognized manifest key '%s'.",
                              (*key).c_str())));
     }
   }
 }
 
 bool Manifest::HasKey(const std::string& key) const {
   return CanAccessKey(key) && value_->HasKey(key);
 }
 
@@ skipping 44 matching lines @@
   return manifest;
 }
 
 bool Manifest::Equals(const Manifest* other) const {
   return other && value_->Equals(other->value());
 }
 
 int Manifest::GetManifestVersion() const {
   // Platform apps were launched after manifest version 2 was the preferred
   // version, so they default to that.
-  int manifest_version = type_ == Extension::TYPE_PLATFORM_APP ? 2 : 1;
+  int manifest_version = type_ == Manifest::TYPE_PLATFORM_APP ? 2 : 1;
   value_->GetInteger(keys::kManifestVersion, &manifest_version);
   return manifest_version;
 }
 
 bool Manifest::CanAccessPath(const std::string& path) const {
   std::vector<std::string> components;
   base::SplitString(path, '.', &components);
   std::string key;
   for (size_t i = 0; i < components.size(); ++i) {
     key += components[i];
     if (!CanAccessKey(key))
       return false;
     key += '.';
   }
   return true;
 }
 
 bool Manifest::CanAccessKey(const std::string& key) const {
   Feature* feature =
       BaseFeatureProvider::GetManifestFeatures()->GetFeature(key);
   if (!feature)
     return true;
 
   return feature->IsAvailableToManifest(
       extension_id_, type_, Feature::ConvertLocation(location_),
       GetManifestVersion()).is_available();
 }
 
 }  // namespace extensions