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

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

Issue 10375021: Move Extension into extensions namespace (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Take 2 Created 8 years, 7 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 b98f264599df930df1f7e12794b97e573b1fc714..61dc1b43409fdaf76357cd72d0e2259264d574c1 100644
--- a/chrome/browser/background/background_contents_service.cc
+++ b/chrome/browser/background/background_contents_service.cc
@@ -37,6 +37,8 @@
using content::SiteInstance;
using content::WebContents;
+using extensions::Extension;
+using extensions::UnloadedExtensionInfo;
namespace {
@@ -55,7 +57,8 @@ void ScheduleCloseBalloon(const std::string& extension_id) {
class CrashNotificationDelegate : public NotificationDelegate {
public:
- CrashNotificationDelegate(Profile* profile, const Extension* extension)
+ CrashNotificationDelegate(Profile* profile,
+ const Extension* extension)
: profile_(profile),
is_hosted_app_(extension->is_hosted_app()),
extension_id_(extension->id()) {
@@ -366,8 +369,8 @@ void BackgroundContentsService::LoadBackgroundContentsFromPrefs(
for (DictionaryValue::key_iterator it = contents->begin_keys();
it != contents->end_keys(); ++it) {
// Check to make sure that the parent extension is still enabled.
- const Extension* extension = extensions_service->GetExtensionById(
- *it, false);
+ const Extension* extension = extensions_service->
+ GetExtensionById(*it, false);
if (!extension) {
// We should never reach here - it should not be possible for an app
// to become uninstalled without the associated BackgroundContents being

Powered by Google App Engine
This is Rietveld 408576698