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

Side by Side Diff: components/history/core/browser/history_service_observer.h

Issue 573553004: Eliminate NOTIFICATION_HISTORY_LOADED notification (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add ScopedObserver to InMemoryHistoryBackend,PrerenderLocalPredictor,ChromeTemplateURLServiceClient Created 6 years 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 COMPONENTS_HISTORY_CORE_BROWSER_HISTORY_SERVICE_OBSERVER_H_ 5 #ifndef COMPONENTS_HISTORY_CORE_BROWSER_HISTORY_SERVICE_OBSERVER_H_
6 #define COMPONENTS_HISTORY_CORE_BROWSER_HISTORY_SERVICE_OBSERVER_H_ 6 #define COMPONENTS_HISTORY_CORE_BROWSER_HISTORY_SERVICE_OBSERVER_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "components/history/core/browser/history_types.h" 9 #include "components/history/core/browser/history_types.h"
10 10
(...skipping 24 matching lines...) Expand all
35 // |changed_urls| lists the information for each of the URLs affected. The 35 // |changed_urls| lists the information for each of the URLs affected. The
36 // rows will have the IDs that are currently in effect in the main history 36 // rows will have the IDs that are currently in effect in the main history
37 // database. 37 // database.
38 virtual void OnURLsModified(HistoryService* history_service, 38 virtual void OnURLsModified(HistoryService* history_service,
39 const URLRows& changed_urls) {} 39 const URLRows& changed_urls) {}
40 40
41 // Called on changes to the VisitDatabase. 41 // Called on changes to the VisitDatabase.
42 virtual void OnAddVisit(HistoryService* history_service, 42 virtual void OnAddVisit(HistoryService* history_service,
43 const BriefVisitInfo& info) {} 43 const BriefVisitInfo& info) {}
44 44
45 // Is called to notify when |history_service| has finished loading.
46 virtual void OnHistoryServiceLoaded(HistoryService* history_service) {}
47
48 // Is called to notify when |history_service| is being deleted.
49 virtual void HistoryServiceBeingDeleted(HistoryService* history_service) {}
50
45 private: 51 private:
46 DISALLOW_COPY_AND_ASSIGN(HistoryServiceObserver); 52 DISALLOW_COPY_AND_ASSIGN(HistoryServiceObserver);
47 }; 53 };
48 54
49 } // namespace history 55 } // namespace history
50 56
51 #endif // COMPONENTS_HISTORY_CORE_BROWSER_HISTORY_SERVICE_OBSERVER_H_ 57 #endif // COMPONENTS_HISTORY_CORE_BROWSER_HISTORY_SERVICE_OBSERVER_H_
OLDNEW
« chrome/browser/chrome_notification_types.h ('K') | « chrome/test/base/ui_test_utils.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698