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

Unified Diff: ui/message_center/message_center_impl_unittest.cc

Issue 149433005: Adds a small icon to notifications, and connects it to synced notifications. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add testing. Created 6 years, 11 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: ui/message_center/message_center_impl_unittest.cc
diff --git a/ui/message_center/message_center_impl_unittest.cc b/ui/message_center/message_center_impl_unittest.cc
index 1e8ab41f29cd395d8adaffbd6927cd60dd5e5d79..b223ad92d4a1be3678fde99c0b25454e478a28dd 100644
--- a/ui/message_center/message_center_impl_unittest.cc
+++ b/ui/message_center/message_center_impl_unittest.cc
@@ -706,6 +706,7 @@ TEST_F(MessageCenterImplTest, QueuedDirectUpdates) {
gfx::Image testImage(gfx::Image(gfx::ImageSkia(canvas.ExtractImageRep())));
message_center()->SetNotificationIcon(id, testImage);
message_center()->SetNotificationImage(id, testImage);
+ message_center()->SetNotificationSmallImage(id, testImage);
message_center()->SetNotificationButtonIcon(id, 0, testImage);
message_center()->SetNotificationButtonIcon(id, 1, testImage);
@@ -724,6 +725,7 @@ TEST_F(MessageCenterImplTest, QueuedDirectUpdates) {
EXPECT_EQ(new_size, mc_notification->icon().Size());
EXPECT_EQ(new_size, mc_notification->image().Size());
+ EXPECT_EQ(new_size, mc_notification->small_image().Size());
EXPECT_EQ(new_size, buttons[0].icon.Size());
EXPECT_EQ(new_size, buttons[1].icon.Size());
}

Powered by Google App Engine
This is Rietveld 408576698