| 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 ceb31a27922650dcbb4d57d6771ee1f84fc956b3..b7b137c7c45dafd87a15138901a3e92751d35069 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;
|
|
|
| @@ -97,10 +87,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(
|
| @@ -108,7 +94,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
|
|
|