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

Unified Diff: chrome/browser/extensions/api/activity_log_private/activity_log_private_api.cc

Issue 2898383002: [Extensions] Make Event::restrict_to_browser_context const. (Closed)
Patch Set: sync @tott Created 3 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/extensions/api/activity_log_private/activity_log_private_api.cc
diff --git a/chrome/browser/extensions/api/activity_log_private/activity_log_private_api.cc b/chrome/browser/extensions/api/activity_log_private/activity_log_private_api.cc
index cfe4292d20c69b746adf2b395ec0e2a1a487f480..7a527c1c029f2251a5031e1876ffc5c5e4d7cf0e 100644
--- a/chrome/browser/extensions/api/activity_log_private/activity_log_private_api.cc
+++ b/chrome/browser/extensions/api/activity_log_private/activity_log_private_api.cc
@@ -93,10 +93,10 @@ void ActivityLogAPI::OnExtensionActivity(scoped_refptr<Action> activity) {
std::unique_ptr<base::ListValue> value(new base::ListValue());
ExtensionActivity activity_arg = activity->ConvertToExtensionActivity();
value->Append(activity_arg.ToValue());
- std::unique_ptr<Event> event(new Event(
+ auto event = base::MakeUnique<Event>(
events::ACTIVITY_LOG_PRIVATE_ON_EXTENSION_ACTIVITY,
- activity_log_private::OnExtensionActivity::kEventName, std::move(value)));
- event->restrict_to_browser_context = browser_context_;
+ activity_log_private::OnExtensionActivity::kEventName, std::move(value),
+ browser_context_);
EventRouter::Get(browser_context_)->BroadcastEvent(std::move(event));
}

Powered by Google App Engine
This is Rietveld 408576698