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

Unified Diff: extensions/browser/guest_view/app_view/app_view_guest.cc

Issue 2294653002: Some linked_ptr -> unique_ptr conversion in extensions/browser. (Closed)
Patch Set: address comments Created 4 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: extensions/browser/guest_view/app_view/app_view_guest.cc
diff --git a/extensions/browser/guest_view/app_view/app_view_guest.cc b/extensions/browser/guest_view/app_view/app_view_guest.cc
index dd5243c70445ab0783b84e1a47a8072b12c59f84..be13b200a7dd3bc3744073baafcaa2d00a4bb356 100644
--- a/extensions/browser/guest_view/app_view/app_view_guest.cc
+++ b/extensions/browser/guest_view/app_view/app_view_guest.cc
@@ -53,7 +53,7 @@ struct ResponseInfo {
~ResponseInfo() {}
};
-typedef std::map<int, linked_ptr<ResponseInfo> > PendingResponseMap;
+using PendingResponseMap = std::map<int, std::unique_ptr<ResponseInfo>>;
static base::LazyInstance<PendingResponseMap> pending_response_map =
LAZY_INSTANCE_INITIALIZER;
@@ -79,7 +79,7 @@ bool AppViewGuest::CompletePendingRequest(
return false;
}
- linked_ptr<ResponseInfo> response_info = it->second;
+ ResponseInfo* response_info = it->second.get();
if (!response_info->app_view_guest ||
(response_info->guest_extension->id() != guest_extension_id)) {
// The app is trying to communicate with an <appview> not assigned to it, or
@@ -191,12 +191,10 @@ void AppViewGuest::CreateWebContents(
return;
}
- pending_response_map.Get().insert(
- std::make_pair(guest_instance_id(),
- make_linked_ptr(new ResponseInfo(
- guest_extension,
- weak_ptr_factory_.GetWeakPtr(),
- callback))));
+ pending_response_map.Get().insert(std::make_pair(
+ guest_instance_id(),
+ base::MakeUnique<ResponseInfo>(
+ guest_extension, weak_ptr_factory_.GetWeakPtr(), callback)));
LazyBackgroundTaskQueue* queue =
LazyBackgroundTaskQueue::Get(browser_context());

Powered by Google App Engine
This is Rietveld 408576698