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

Unified Diff: components/offline_items_collection/core/throttled_offline_content_provider_unittest.cc

Issue 2811803006: Add support for pulling icons for OfflineItems (Closed)
Patch Set: More findbugs Created 3 years, 8 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: components/offline_items_collection/core/throttled_offline_content_provider_unittest.cc
diff --git a/components/offline_items_collection/core/throttled_offline_content_provider_unittest.cc b/components/offline_items_collection/core/throttled_offline_content_provider_unittest.cc
index 414247b35c54eb35f9ad82c45b3aa3f7cf85d52d..912e930cc2c3b63d49858d0bf3f912d9de073cd3 100644
--- a/components/offline_items_collection/core/throttled_offline_content_provider_unittest.cc
+++ b/components/offline_items_collection/core/throttled_offline_content_provider_unittest.cc
@@ -105,15 +105,16 @@ TEST_F(ThrottledOfflineContentProviderTest, TestBasicPassthrough) {
EXPECT_CALL(wrapped_provider_, CancelDownload(id));
EXPECT_CALL(wrapped_provider_, PauseDownload(id));
EXPECT_CALL(wrapped_provider_, ResumeDownload(id));
+ EXPECT_CALL(wrapped_provider_, GetVisualsForItem(id, _));
EXPECT_CALL(wrapped_provider_, GetItemById(id)).WillRepeatedly(Return(&item));
EXPECT_CALL(wrapped_provider_, GetAllItems()).WillRepeatedly(Return(items));
-
wrapped_provider_.NotifyOnItemsAvailable();
provider_.OpenItem(id);
provider_.RemoveItem(id);
provider_.CancelDownload(id);
provider_.PauseDownload(id);
provider_.ResumeDownload(id);
+ provider_.GetVisualsForItem(id, OfflineContentProvider::VisualsCallback());
EXPECT_EQ(&item, provider_.GetItemById(id));
EXPECT_EQ(items, provider_.GetAllItems());
}
« no previous file with comments | « components/offline_items_collection/core/throttled_offline_content_provider.cc ('k') | tools/android/eclipse/.classpath » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698