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

Unified Diff: chrome/browser/notifications/sync_notifier/synced_notification_unittest.cc

Issue 23981004: Synced Notifications Layout Changes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Layout Changes - fix unit tests Created 7 years, 3 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
« no previous file with comments | « chrome/browser/notifications/sync_notifier/synced_notification.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/notifications/sync_notifier/synced_notification_unittest.cc
diff --git a/chrome/browser/notifications/sync_notifier/synced_notification_unittest.cc b/chrome/browser/notifications/sync_notifier/synced_notification_unittest.cc
index 49d3668e9a6bb6b8352758d97f05404c2e78f9d7..2c4986d7de22ec2635e138cf042d7c88beb89ffc 100644
--- a/chrome/browser/notifications/sync_notifier/synced_notification_unittest.cc
+++ b/chrome/browser/notifications/sync_notifier/synced_notification_unittest.cc
@@ -327,20 +327,12 @@ TEST_F(SyncedNotificationTest, ShowTest) {
// Check the base fields of the notification.
EXPECT_EQ(message_center::NOTIFICATION_TYPE_IMAGE, notification.type());
EXPECT_EQ(std::string(kTitle1), UTF16ToUTF8(notification.title()));
- EXPECT_EQ(std::string(kText1And1), UTF16ToUTF8(notification.message()));
+ EXPECT_EQ(std::string(kText1), UTF16ToUTF8(notification.message()));
EXPECT_EQ(std::string(kExpectedOriginUrl), notification.origin_url().spec());
EXPECT_EQ(std::string(kKey1), UTF16ToUTF8(notification.replace_id()));
EXPECT_EQ(kFakeCreationTime, notification.timestamp().ToDoubleT());
EXPECT_EQ(kNotificationPriority, notification.priority());
-
- EXPECT_EQ(UTF8ToUTF16(kContainedTitle1), notification.items()[0].title);
- EXPECT_EQ(UTF8ToUTF16(kContainedTitle2), notification.items()[1].title);
- EXPECT_EQ(UTF8ToUTF16(kContainedTitle3), notification.items()[2].title);
-
- EXPECT_EQ(UTF8ToUTF16(kContainedMessage1), notification.items()[0].message);
- EXPECT_EQ(UTF8ToUTF16(kContainedMessage2), notification.items()[1].message);
- EXPECT_EQ(UTF8ToUTF16(kContainedMessage3), notification.items()[2].message);
}
TEST_F(SyncedNotificationTest, DismissTest) {
@@ -406,7 +398,7 @@ TEST_F(SyncedNotificationTest, OnFetchCompleteTest) {
notification_manager.notification().type());
EXPECT_EQ(std::string(kTitle1),
UTF16ToUTF8(notification_manager.notification().title()));
- EXPECT_EQ(std::string(kText1And1),
+ EXPECT_EQ(std::string(kText1),
UTF16ToUTF8(notification_manager.notification().message()));
// TODO(petewil): Check that the bitmap in the notification is what we expect.
« no previous file with comments | « chrome/browser/notifications/sync_notifier/synced_notification.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698