new implementation of default apps

chrome/browser/extensions/extension_service.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/extension_service.h"
 
 #include <algorithm>
 #include <set>
 
 #include "base/basictypes.h"
@@ skipping 2259 matching lines @@
          const Version* version,
          const FilePath& path,
          Extension::Location location,
          int creation_flags,
          bool mark_acknowledged) {
   CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   CHECK(Extension::IdIsValid(id));
   if (extension_prefs_->IsExternalExtensionUninstalled(id))
     return false;
 
-  DCHECK(version);
-
   // Before even bothering to unpack, check and see if we already have this
   // version. This is important because these extensions are going to get
   // installed on every startup.
   const Extension* existing = GetExtensionById(id, true);
+
   if (existing) {
-    switch (existing->version()->CompareTo(*version)) {
-      case -1:  // existing version is older, we should upgrade
-        break;
-      case 0:  // existing version is same, do nothing
-        return false;
-      case 1:  // existing version is newer, uh-oh
-        LOG(WARNING) << "Found external version of extension " << id
-                     << "that is older than current version. Current version "
-                     << "is: " << existing->VersionString() << ". New version "
-                     << "is: " << version->GetString()
-                     << ". Keeping current version.";
-        return false;
+    // The default apps will have the location set as INTERNAL. Since older
+    // default apps are installed as EXTERNAL, we override them. However, if the
+    // app is already installed as internal, then do the version check.
+    // TODO (grv) : Remove after migration (~6 months).
+    bool is_default_apps_migration =
+        (location == Extension::INTERNAL &&
+         Extension::IsExternalLocation(existing->location()));
+
+    if (!is_default_apps_migration) {
+

[Mihai Parparita -not on Chrome, 2012/08/16 00:58:24]

Nit: extra newline.

+      DCHECK(version);
+
+      switch (existing->version()->CompareTo(*version)) {
+        case -1:  // existing version is older, we should upgrade
+          break;
+        case 0:  // existing version is same, do nothing
+          return false;
+        case 1:  // existing version is newer, uh-oh
+          LOG(WARNING) << "Found external version of extension " << id
+                       << "that is older than current version. Current version "
+                       << "is: " << existing->VersionString() << ". New "
+                       << "version is: " << version->GetString()
+                       << ". Keeping current version.";
+          return false;
+      }
     }
   }
 
   // If the extension is already pending, don't start an install.
   if (!pending_extension_manager()->AddFromExternalFile(
           id, location, *version)) {
     return false;
   }
 
   // no client (silent install)
@@ skipping 259 matching lines @@
 
 ExtensionService::NaClModuleInfoList::iterator
     ExtensionService::FindNaClModule(const GURL& url) {
   for (NaClModuleInfoList::iterator iter = nacl_module_list_.begin();
        iter != nacl_module_list_.end(); ++iter) {
     if (iter->url == url)
       return iter;
   }
   return nacl_module_list_.end();
 }