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

Unified Diff: chrome/browser/history/visit_database.h

Issue 6990074: Change GetVisitCountToHost() to GetVisibleVisitCountToHost(), and restrict it to returning "user-... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 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
« no previous file with comments | « chrome/browser/history/history_marshaling.h ('k') | chrome/browser/history/visit_database.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/history/visit_database.h
===================================================================
--- chrome/browser/history/visit_database.h (revision 86341)
+++ chrome/browser/history/visit_database.h (working copy)
@@ -134,13 +134,15 @@
VisitID* from_visit,
GURL* from_url);
- // Returns the number of visits to all urls on the scheme/host/post
- // identified by url. This is only valid for http and https urls (all other
- // schemes are ignored and false is returned).
- // count is set to the number of visits, first_visit is set to the first time
- // the host was visited. Returns true on success.
- bool GetVisitCountToHost(const GURL& url, int* count,
- base::Time* first_visit);
+ // Gets the number of user-visible visits to all URLs on the same
+ // scheme/host/port as |url|, as well as the time of the earliest visit.
+ // "User-visible" is defined as in GetVisibleVisitsInRange() above, i.e.
+ // excluding redirects and subframes.
+ // This function is only valid for HTTP and HTTPS URLs; all other schemes
+ // cause the function to return false.
+ bool GetVisibleVisitCountToHost(const GURL& url,
+ int* count,
+ base::Time* first_visit);
// Get the time of the first item in our database.
bool GetStartDate(base::Time* first_visit);
« no previous file with comments | « chrome/browser/history/history_marshaling.h ('k') | chrome/browser/history/visit_database.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698