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

Unified Diff: chrome/browser/history/android/android_provider_backend_unittest.cc

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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/history/android/android_provider_backend_unittest.cc
diff --git a/chrome/browser/history/android/android_provider_backend_unittest.cc b/chrome/browser/history/android/android_provider_backend_unittest.cc
index f527a8a6e82140287aa47a2786ec2efd175ae3f5..043c7afe37188c9d32de6d701e740fd54eff6624 100644
--- a/chrome/browser/history/android/android_provider_backend_unittest.cc
+++ b/chrome/browser/history/android/android_provider_backend_unittest.cc
@@ -60,15 +60,16 @@ class AndroidProviderBackendDelegate : public HistoryBackend::Delegate {
sql::InitStatus init_status) OVERRIDE {}
virtual void SetInMemoryBackend(int backend_id,
InMemoryHistoryBackend* backend) OVERRIDE {}
+ virtual void SendFaviconChangedNotification(
+ chrome::FaviconChangedDetails* details) OVERRIDE {
+ favicon_details_.reset(details);
+ }
virtual void BroadcastNotifications(int type,
HistoryDetails* details) OVERRIDE {
switch (type) {
case chrome::NOTIFICATION_HISTORY_URLS_DELETED:
deleted_details_.reset(static_cast<URLsDeletedDetails*>(details));
break;
- case chrome::NOTIFICATION_FAVICON_CHANGED:
- favicon_details_.reset(static_cast<FaviconChangeDetails*>(details));
- break;
case chrome::NOTIFICATION_HISTORY_URLS_MODIFIED:
modified_details_.reset(static_cast<URLsModifiedDetails*>(details));
break;
@@ -87,7 +88,7 @@ class AndroidProviderBackendDelegate : public HistoryBackend::Delegate {
return modified_details_.get();
}
- FaviconChangeDetails* favicon_details() const {
+ chrome::FaviconChangedDetails* favicon_details() const {
return favicon_details_.get();
}
@@ -100,7 +101,7 @@ class AndroidProviderBackendDelegate : public HistoryBackend::Delegate {
private:
scoped_ptr<URLsDeletedDetails> deleted_details_;
scoped_ptr<URLsModifiedDetails> modified_details_;
- scoped_ptr<FaviconChangeDetails> favicon_details_;
+ scoped_ptr<chrome::FaviconChangedDetails> favicon_details_;
DISALLOW_COPY_AND_ASSIGN(AndroidProviderBackendDelegate);
};

Powered by Google App Engine
This is Rietveld 408576698