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 5f8a3e8fc7d68cf7563cafa47b992e162012817a..338a0768d9488af065e6f15dfb0e417a06a48ea9 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( |