Index: chrome/browser/background/background_contents_service.cc |
diff --git a/chrome/browser/background/background_contents_service.cc b/chrome/browser/background/background_contents_service.cc |
index 3b87006e1d76365b6d5286d62ce719ca6ab42ba2..a13fe2f2309da73cddc48b0b580be5d218873760 100644 |
--- a/chrome/browser/background/background_contents_service.cc |
+++ b/chrome/browser/background/background_contents_service.cc |
@@ -458,10 +458,11 @@ void BackgroundContentsService::OnExtensionUnloaded( |
case UnloadedExtensionInfo::REASON_TERMINATE: // Fall through. |
case UnloadedExtensionInfo::REASON_UNINSTALL: // Fall through. |
case UnloadedExtensionInfo::REASON_BLACKLIST: // Fall through. |
+ case UnloadedExtensionInfo::REASON_LOCK_ALL: // Fall through. |
case UnloadedExtensionInfo::REASON_PROFILE_SHUTDOWN: |
ShutdownAssociatedBackgroundContents(base::ASCIIToUTF16(extension->id())); |
SendChangeNotification(Profile::FromBrowserContext(browser_context)); |
- break; |
+ return; |
case UnloadedExtensionInfo::REASON_UPDATE: { |
// If there is a manifest specified background page, then shut it down |
// here, since if the updated extension still has the background page, |
@@ -473,13 +474,15 @@ void BackgroundContentsService::OnExtensionUnloaded( |
ShutdownAssociatedBackgroundContents( |
base::ASCIIToUTF16(extension->id())); |
} |
+ return; |
+ case UnloadedExtensionInfo::REASON_UNDEFINED: |
+ // Fall through to undefined case. |
break; |
} |
- default: |
- NOTREACHED(); |
- ShutdownAssociatedBackgroundContents(base::ASCIIToUTF16(extension->id())); |
- break; |
} |
+ NOTREACHED() << "Undefined case " << reason; |
+ return ShutdownAssociatedBackgroundContents( |
+ base::ASCIIToUTF16(extension->id())); |
} |
void BackgroundContentsService::OnExtensionUninstalled( |