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

Unified Diff: chrome/browser/history/history.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/download/download_history.cc ('k') | chrome/browser/history/history.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/history/history.h
===================================================================
--- chrome/browser/history/history.h (revision 86341)
+++ chrome/browser/history/history.h (working copy)
@@ -330,15 +330,17 @@
typedef Callback4<Handle,
bool, // Were we able to determine the # of visits?
int, // Number of visits.
- base::Time>::Type // Time of first visit. Only first bool
+ base::Time>::Type // Time of first visit. Only set if bool
// is true and int is > 0.
- GetVisitCountToHostCallback;
+ GetVisibleVisitCountToHostCallback;
- // Requests the number of visits to all urls on the scheme/host/post
- // identified by url. This is only valid for http and https urls.
- Handle GetVisitCountToHost(const GURL& url,
- CancelableRequestConsumerBase* consumer,
- GetVisitCountToHostCallback* callback);
+ // Requests the number of user-visible visits (i.e. no redirects or subframes)
+ // to all urls on the same scheme/host/port as |url|. This is only valid for
+ // HTTP and HTTPS URLs.
+ Handle GetVisibleVisitCountToHost(
+ const GURL& url,
+ CancelableRequestConsumerBase* consumer,
+ GetVisibleVisitCountToHostCallback* callback);
// Called when QueryTopURLsAndRedirects completes. The vector contains a list
// of the top |result_count| URLs. For each of these URLs, there is an entry
« no previous file with comments | « chrome/browser/download/download_history.cc ('k') | chrome/browser/history/history.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698