| Index: chrome/browser/extensions/activity_log/fullstream_ui_policy.h
|
| diff --git a/chrome/browser/extensions/activity_log/fullstream_ui_policy.h b/chrome/browser/extensions/activity_log/fullstream_ui_policy.h
|
| index e4a2f6484ff9fff5321b14cd1904871413b0c9ae..7c403d4f9cd4c6f10c473b731abc1c4d17121290 100644
|
| --- a/chrome/browser/extensions/activity_log/fullstream_ui_policy.h
|
| +++ b/chrome/browser/extensions/activity_log/fullstream_ui_policy.h
|
| @@ -28,23 +28,13 @@ class FullStreamUIPolicy : public ActivityLogDatabasePolicy {
|
|
|
| virtual void ProcessAction(scoped_refptr<Action> action) OVERRIDE;
|
|
|
| - // TODO(felt,dbabic) This is overly specific to FullStreamUIPolicy.
|
| - // It assumes that the callback can return a sorted vector of actions. Some
|
| - // policies might not do that. For instance, imagine a trivial policy that
|
| - // just counts the frequency of certain actions within some time period,
|
| - // this call would be meaningless, as it couldn't return anything useful.
|
| - virtual void ReadData(
|
| - const std::string& extension_id,
|
| - const int day,
|
| - const base::Callback
|
| - <void(scoped_ptr<Action::ActionVector>)>& callback) OVERRIDE;
|
| -
|
| virtual void ReadFilteredData(
|
| const std::string& extension_id,
|
| const Action::ActionType type,
|
| const std::string& api_name,
|
| const std::string& page_url,
|
| const std::string& arg_url,
|
| + const int days_ago,
|
| const base::Callback
|
| <void(scoped_ptr<Action::ActionVector>)>& callback) OVERRIDE;
|
|
|
| @@ -90,10 +80,6 @@ class FullStreamUIPolicy : public ActivityLogDatabasePolicy {
|
| // database thread.
|
| void QueueAction(scoped_refptr<Action> action);
|
|
|
| - // The implementation of ReadData; this must only run on the database thread.
|
| - scoped_ptr<Action::ActionVector> DoReadData(const std::string& extension_id,
|
| - const int days_ago);
|
| -
|
| // Internal method to read data from the database; called on the database
|
| // thread.
|
| scoped_ptr<Action::ActionVector> DoReadFilteredData(
|
| @@ -101,7 +87,8 @@ class FullStreamUIPolicy : public ActivityLogDatabasePolicy {
|
| const Action::ActionType type,
|
| const std::string& api_name,
|
| const std::string& page_url,
|
| - const std::string& arg_url);
|
| + const std::string& arg_url,
|
| + const int days_ago);
|
| };
|
|
|
| } // namespace extensions
|
|
|