Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3501)

Unified Diff: chrome/browser/background/background_contents_service.cc

Issue 7633029: Remove extension.h #include from profile.h. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix unit_tests. Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 9ec28ecc40c092d1afae60eedd6bdcca5e85c4d0..bc3e6062d18e3acdb8dd3919983c16d67f048ab2 100644
--- a/chrome/browser/background/background_contents_service.cc
+++ b/chrome/browser/background/background_contents_service.cc
@@ -24,6 +24,7 @@
#include "chrome/common/chrome_notification_types.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/common/extensions/extension.h"
+#include "chrome/common/extensions/extension_constants.h"
#include "chrome/common/pref_names.h"
#include "content/browser/renderer_host/render_view_host.h"
#include "content/browser/site_instance.h"
@@ -292,13 +293,14 @@ void BackgroundContentsService::Observe(int type,
}
case chrome::NOTIFICATION_EXTENSION_UNLOADED:
switch (Details<UnloadedExtensionInfo>(details)->reason) {
- case UnloadedExtensionInfo::DISABLE: // Intentionally fall through.
- case UnloadedExtensionInfo::UNINSTALL:
+ // Intentionally fall through.
+ case extension_misc::UNLOAD_REASON_DISABLE:
+ case extension_misc::UNLOAD_REASON_UNINSTALL:
ShutdownAssociatedBackgroundContents(
ASCIIToUTF16(
Details<UnloadedExtensionInfo>(details)->extension->id()));
break;
- case UnloadedExtensionInfo::UPDATE: {
+ case extension_misc::UNLOAD_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,
// then it will be loaded from LOADED callback. Otherwise, leave

Powered by Google App Engine
This is Rietveld 408576698