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

Unified Diff: chrome/browser/extensions/activity_log/stream_noargs_ui_policy.cc

Issue 19234003: Extension activity log database refactoring (step 2) (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Address some reviewer comments Created 7 years, 5 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/activity_log/stream_noargs_ui_policy.cc
diff --git a/chrome/browser/extensions/activity_log/stream_noargs_ui_policy.cc b/chrome/browser/extensions/activity_log/stream_noargs_ui_policy.cc
index c30d33dd414c6f8d3604fed41507ab6094c24c18..180b8023d599070a4af63961efd457c458e0137b 100644
--- a/chrome/browser/extensions/activity_log/stream_noargs_ui_policy.cc
+++ b/chrome/browser/extensions/activity_log/stream_noargs_ui_policy.cc
@@ -16,15 +16,15 @@ StreamWithoutArgsUIPolicy::StreamWithoutArgsUIPolicy(Profile* profile)
StreamWithoutArgsUIPolicy::~StreamWithoutArgsUIPolicy() {}
-std::string StreamWithoutArgsUIPolicy::ProcessArguments(
+scoped_ptr<base::ListValue> StreamWithoutArgsUIPolicy::ProcessArguments(
ActionType action_type,
const std::string& name,
const base::ListValue* args) const {
if (action_type == ACTION_DOM ||
arg_whitelist_api_.find(name) != arg_whitelist_api_.end()) {
- return FullStreamUIPolicy::ProcessArguments(action_type, name, args);
+ return FullStreamUIPolicy::ProcessArguments(action_type, name, args).Pass();
} else {
- return std::string();
+ return make_scoped_ptr(new ListValue());
}
}

Powered by Google App Engine
This is Rietveld 408576698