Activity Log: allow searching by day

chrome/browser/extensions/activity_log/activity_log_policy.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_ACTIVITY_LOG_ACTIVITY_LOG_POLICY_H_
 #define CHROME_BROWSER_EXTENSIONS_ACTIVITY_LOG_ACTIVITY_LOG_POLICY_H_
 
 #include <set>
 #include <string>
 
@@ skipping 56 matching lines @@
   // Instead of a public destructor, ActivityLogPolicy objects have a Close()
   // method which will cause the object to be deleted (but may do so on another
   // thread or in a deferred fashion).
   virtual void Close() = 0;
 
   // Updates the internal state of the model summarizing actions and possibly
   // writes to the database.  Implements the default policy storing internal
   // state to memory every 5 min.
   virtual void ProcessAction(scoped_refptr<Action> action) = 0;
 
-  // Gets all actions for a given extension for the specified day. 0 = today,
-  // 1 = yesterday, etc. Only returns 1 day at a time. Actions are sorted from
-  // newest to oldest. Results as passed to the specified callback when
-  // available.
-  //
-  // TODO(felt,dbabic) This is overly specific to the current implementation of
-  // the FullStreamUIPolicy.  We should refactor it to use a more generic read
-  // function, for example one that takes a dictionary of query parameters
-  // (extension_id, time range, etc.).
-  virtual void ReadData(
-      const std::string& extension_id,
-      const int day,
-      const base::Callback
-          <void(scoped_ptr<Action::ActionVector>)>& callback) = 0;
-
   // Gets all actions that match the specified fields. URLs are treated like
   // prefixes; other fields are exact matches. Empty strings are not matched to
-  // anything.
+  // anything. For the date: 0 = today, 1 = yesterday, etc.; if the data is
+  // negative, it will be treated as missing.
   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) = 0;
 
   // Clean the relevant URL data. The cleaning may need to be different for
   // different policies. If restrict_urls is empty then all URLs are removed.
   virtual void RemoveURLs(const std::vector<GURL>& restrict_urls) = 0;
 
   // For unit testing only.
   void SetClockForTesting(scoped_ptr<base::Clock> clock);
 
@@ skipping 111 matching lines @@
 
  private:
   // See the comments for the ActivityDatabase class for a discussion of how
   // database cleanup runs.
   ActivityDatabase* db_;
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_ACTIVITY_LOG_ACTIVITY_LOG_POLICY_H_