Don't close crashed tabs from extensions with background pages.

chrome/browser/extensions/extension_info_map.cc

 // Copyright (c) 2011 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_info_map.h"
 
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_set.h"
 #include "chrome/common/url_constants.h"
 #include "content/public/browser/browser_thread.h"
@@ skipping 47 matching lines @@
   extra_data_[extension->id()].install_time = install_time;
   extra_data_[extension->id()].incognito_enabled = incognito_enabled;
 }
 
 void ExtensionInfoMap::RemoveExtension(const std::string& extension_id,
     const extension_misc::UnloadedExtensionReason reason) {
   CheckOnValidThread();
   const Extension* extension = extensions_.GetByID(extension_id);
   extra_data_.erase(extension_id);  // we don't care about disabled extra data
   if (extension) {
-    if (reason == extension_misc::UNLOAD_REASON_DISABLE)
+    if (reason == extension_misc::UNLOAD_REASON_DISABLE ||
+        reason == extension_misc::UNLOAD_REASON_TERMINATE)

[Matt Perry, 2011/11/10 22:07:59]

nit: pull this condition out into a shared bool like 'was_disabled'

[Yoyo Zhou, 2011/11/10 23:26:30]

Done.

       disabled_extensions_.Insert(extension);
     extensions_.Remove(extension_id);
-  } else if (reason != extension_misc::UNLOAD_REASON_DISABLE) {
+  } else if (reason != extension_misc::UNLOAD_REASON_DISABLE &&
+             reason != extension_misc::UNLOAD_REASON_TERMINATE) {
     // If the extension was uninstalled, make sure it's removed from the map of
     // disabled extensions.
     disabled_extensions_.Remove(extension_id);
   } else {
     // NOTE: This can currently happen if we receive multiple unload
     // notifications, e.g. setting incognito-enabled state for a
     // disabled extension (e.g., via sync).  See
     // http://code.google.com/p/chromium/issues/detail?id=50582 .
     NOTREACHED() << extension_id;
   }
@@ skipping 55 matching lines @@
   ExtensionSet::ExtensionMap::const_iterator i = extensions_.begin();
   for (; i != extensions_.end(); ++i) {
     if (i->second->web_extent().MatchesSecurityOrigin(origin) &&
         process_map_.Contains(i->first, process_id) &&
         i->second->HasAPIPermission(permission)) {
       return true;
     }
   }
   return false;
 }