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

Unified Diff: apps/app_restore_service.cc

Issue 15984016: Call scoped_refptr<T>::get() rather than relying on implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 7 years, 6 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
« no previous file with comments | « no previous file | apps/shell_window_geometry_cache_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: apps/app_restore_service.cc
diff --git a/apps/app_restore_service.cc b/apps/app_restore_service.cc
index 37a463be4586ef988ede153aecd814618c8c1e59..f5cf21c419289322359f4a763f8d6fefd2e3e995 100644
--- a/apps/app_restore_service.cc
+++ b/apps/app_restore_service.cc
@@ -69,14 +69,14 @@ void AppRestoreService::HandleStartup(bool should_restore_apps) {
for (ExtensionSet::const_iterator it = extensions->begin();
it != extensions->end(); ++it) {
- const Extension* extension = *it;
+ const Extension* extension = it->get();
if (extension_prefs->IsExtensionRunning(extension->id())) {
RecordAppStop(extension->id());
// If we are not restoring apps (e.g., because it is a clean restart), and
// the app does not have retain permission, explicitly clear the retained
// entries queue.
if (should_restore_apps) {
- RestoreApp(*it);
+ RestoreApp(it->get());
} else {
SavedFilesService::Get(profile_)->ClearQueueIfNoRetainPermission(
extension);
« no previous file with comments | « no previous file | apps/shell_window_geometry_cache_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698