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

Side by Side Diff: chrome/browser/history/history_backend.h

Issue 16517002: Track fraction of visits to top URLs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Added more detail to histogram description. Created 7 years, 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_HISTORY_HISTORY_BACKEND_H_ 5 #ifndef CHROME_BROWSER_HISTORY_HISTORY_BACKEND_H_
6 #define CHROME_BROWSER_HISTORY_HISTORY_BACKEND_H_ 6 #define CHROME_BROWSER_HISTORY_HISTORY_BACKEND_H_
7 7
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 #include <utility> 10 #include <utility>
(...skipping 886 matching lines...) Expand 10 before | Expand all | Expand 10 after
897 // loaded. 897 // loaded.
898 BookmarkService* bookmark_service_; 898 BookmarkService* bookmark_service_;
899 899
900 // Publishes the history to all indexers which are registered to receive 900 // Publishes the history to all indexers which are registered to receive
901 // history data from us. Can be NULL if there are no listeners. 901 // history data from us. Can be NULL if there are no listeners.
902 scoped_ptr<HistoryPublisher> history_publisher_; 902 scoped_ptr<HistoryPublisher> history_publisher_;
903 903
904 #if defined(OS_ANDROID) 904 #if defined(OS_ANDROID)
905 // Used to provide the Android ContentProvider APIs. 905 // Used to provide the Android ContentProvider APIs.
906 scoped_ptr<AndroidProviderBackend> android_provider_backend_; 906 scoped_ptr<AndroidProviderBackend> android_provider_backend_;
907
908 // Used to provide UMA on the number of page visits that are to the most
909 // visited URLs. This is here because the backend both has access to this
910 // information and is notified of page visits. The top sites service should
911 // be used instead whenever possible.
912 MostVisitedURLList most_visited_urls_;
jar (doing other things) 2013/06/17 22:58:14 nit: I went to see what type this was... and notic
bengr 2013/06/17 23:46:59 Done.
907 #endif 913 #endif
908 914
909 // Used to manage syncing of the typed urls datatype. This will be NULL 915 // Used to manage syncing of the typed urls datatype. This will be NULL
910 // before Init is called. 916 // before Init is called.
911 scoped_ptr<TypedUrlSyncableService> typed_url_syncable_service_; 917 scoped_ptr<TypedUrlSyncableService> typed_url_syncable_service_;
912 918
913 DISALLOW_COPY_AND_ASSIGN(HistoryBackend); 919 DISALLOW_COPY_AND_ASSIGN(HistoryBackend);
914 }; 920 };
915 921
916 } // namespace history 922 } // namespace history
917 923
918 #endif // CHROME_BROWSER_HISTORY_HISTORY_BACKEND_H_ 924 #endif // CHROME_BROWSER_HISTORY_HISTORY_BACKEND_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/history/history_backend.cc » ('j') | chrome/browser/history/history_backend.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698