Pull CrxInstaller out of ExtensionsService.

chrome/browser/extensions/extension_updater.cc

 // Copyright (c) 2009 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_updater.h"
 
 #include <algorithm>
 #include <set>
 
 #include "base/logging.h"
@@ skipping 191 matching lines @@
   // If there are any pending downloads left, start one.
   if (extensions_pending_.size() > 0) {
     ExtensionFetch next = extensions_pending_.front();
     extensions_pending_.pop_front();
     FetchUpdatedExtension(next.id, next.url);
   }
 }
 
 void ExtensionUpdater::OnCRXFileWritten(const std::string& id,
                                         const FilePath& path) {
-  // TODO(asargent) - Instead of calling InstallExtension here, we should have
-  // an UpdateExtension method in ExtensionsService and rely on it to check
-  // that the extension is still installed, and still an older version than
-  // what we're handing it.
-  // (http://crbug.com/12764).
-  ExtensionInstallCallback* callback =
-      NewCallback(this, &ExtensionUpdater::OnExtensionInstallFinished);
-  service_->UpdateExtension(id, path, false, callback);
+  service_->UpdateExtension(id, path);
 }
 
 void ExtensionUpdater::OnExtensionInstallFinished(const FilePath& path,
                                                   Extension* extension) {
   // Have the file_handler_ delete the temp file on the file I/O thread.
   file_io_loop_->PostTask(FROM_HERE, NewRunnableMethod(
     file_handler_.get(), &ExtensionUpdaterFileHandler::DeleteFile, path));
 }
 
 
@@ skipping 348 matching lines @@
     }
   } else {
     extension_fetcher_.reset(
         URLFetcher::Create(kExtensionFetcherId, url, URLFetcher::GET, this));
     extension_fetcher_->set_request_context(
         Profile::GetDefaultRequestContext());
     extension_fetcher_->Start();
     current_extension_fetch_ = ExtensionFetch(id, url);
   }
 }