Terminated extensions should be uninstalled without error now.

chrome/browser/extensions/api/management/management_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/management/management_api.h"
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 581 matching lines @@
 }
 
 ManagementUninstallFunctionBase::~ManagementUninstallFunctionBase() {
 }
 
 bool ManagementUninstallFunctionBase::Uninstall(
     const std::string& target_extension_id,
     bool show_confirm_dialog) {
   extension_id_ = target_extension_id;
   const Extension* target_extension =
-      service()->GetExtensionById(extension_id_, true);
+      extensions::ExtensionRegistry::Get(GetProfile())->

[Ken Rockot(use gerrit already), 2014/09/09 05:55:29]

Please use browser_context() instead of GetProfile()

+          GetExtensionById(extension_id_, ExtensionRegistry::EVERYTHING);
   if (!target_extension ||
       ui_util::ShouldNotBeVisible(target_extension, browser_context())) {
     error_ = ErrorUtils::FormatErrorMessage(
         keys::kNoExtensionError, extension_id_);
     return false;
   }
 
   if (!ExtensionSystem::Get(GetProfile())
            ->management_policy()
            ->UserMayModifySettings(target_extension, NULL)) {
@@ skipping 400 matching lines @@
 ManagementAPI::GetFactoryInstance() {
   return g_factory.Pointer();
 }
 
 void ManagementAPI::OnListenerAdded(const EventListenerInfo& details) {
   management_event_router_.reset(new ManagementEventRouter(browser_context_));
   EventRouter::Get(browser_context_)->UnregisterObserver(this);
 }
 
 }  // namespace extensions