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

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

Issue 15275004: bookmarks: Get rid of the dependency on history_notifications.h (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: more android fixes Created 7 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 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_SERVICE_H_ 5 #ifndef CHROME_BROWSER_HISTORY_HISTORY_SERVICE_H_
6 #define CHROME_BROWSER_HISTORY_HISTORY_SERVICE_H_ 6 #define CHROME_BROWSER_HISTORY_HISTORY_SERVICE_H_
7 7
8 #include <set> 8 #include <set>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 class PageUsageData; 47 class PageUsageData;
48 class PageUsageRequest; 48 class PageUsageRequest;
49 class Profile; 49 class Profile;
50 struct HistoryURLProviderParams; 50 struct HistoryURLProviderParams;
51 51
52 namespace base { 52 namespace base {
53 class FilePath; 53 class FilePath;
54 class Thread; 54 class Thread;
55 } 55 }
56 56
57 namespace chrome {
58 struct FaviconChangedDetails;
59 }
60
57 namespace visitedlink { 61 namespace visitedlink {
58 class VisitedLinkMaster; 62 class VisitedLinkMaster;
59 } 63 }
60 64
61 namespace history { 65 namespace history {
62 66
63 class HistoryBackend; 67 class HistoryBackend;
64 class HistoryDatabase; 68 class HistoryDatabase;
65 class HistoryDBTask; 69 class HistoryDBTask;
66 class HistoryQueryTest; 70 class HistoryQueryTest;
(...skipping 598 matching lines...) Expand 10 before | Expand all | Expand 10 after
665 bool Init(const base::FilePath& history_dir, 669 bool Init(const base::FilePath& history_dir,
666 BookmarkService* bookmark_service, 670 BookmarkService* bookmark_service,
667 bool no_db); 671 bool no_db);
668 672
669 // Called by the HistoryURLProvider class to schedule an autocomplete, it 673 // Called by the HistoryURLProvider class to schedule an autocomplete, it
670 // will be called back on the internal history thread with the history 674 // will be called back on the internal history thread with the history
671 // database so it can query. See history_autocomplete.cc for a diagram. 675 // database so it can query. See history_autocomplete.cc for a diagram.
672 void ScheduleAutocomplete(HistoryURLProvider* provider, 676 void ScheduleAutocomplete(HistoryURLProvider* provider,
673 HistoryURLProviderParams* params); 677 HistoryURLProviderParams* params);
674 678
679 // Sends chrome::NOTIFICATION_FAVICON_CHANGED notification.
680 // This function does not take ownership of |details|.
681 void SendFaviconChangedNotificationHelper(
682 chrome::FaviconChangedDetails* details);
683
675 // Broadcasts the given notification. This is called by the backend so that 684 // Broadcasts the given notification. This is called by the backend so that
676 // the notification will be broadcast on the main thread. 685 // the notification will be broadcast on the main thread.
677 // 686 //
678 // Compared to BroadcastNotifications(), this function does not take 687 // Compared to BroadcastNotifications(), this function does not take
679 // ownership of |details|. 688 // ownership of |details|.
680 void BroadcastNotificationsHelper(int type, 689 void BroadcastNotificationsHelper(int type,
681 history::HistoryDetails* details); 690 history::HistoryDetails* details);
682 691
683 // Initializes the backend. 692 // Initializes the backend.
684 void LoadBackendIfNecessary(); 693 void LoadBackendIfNecessary();
(...skipping 406 matching lines...) Expand 10 before | Expand all | Expand 10 after
1091 scoped_ptr<history::InMemoryURLIndex> in_memory_url_index_; 1100 scoped_ptr<history::InMemoryURLIndex> in_memory_url_index_;
1092 1101
1093 ObserverList<history::VisitDatabaseObserver> visit_database_observers_; 1102 ObserverList<history::VisitDatabaseObserver> visit_database_observers_;
1094 1103
1095 history::DeleteDirectiveHandler delete_directive_handler_; 1104 history::DeleteDirectiveHandler delete_directive_handler_;
1096 1105
1097 DISALLOW_COPY_AND_ASSIGN(HistoryService); 1106 DISALLOW_COPY_AND_ASSIGN(HistoryService);
1098 }; 1107 };
1099 1108
1100 #endif // CHROME_BROWSER_HISTORY_HISTORY_SERVICE_H_ 1109 #endif // CHROME_BROWSER_HISTORY_HISTORY_SERVICE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698