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

Unified Diff: ui/message_center/cocoa/notification_controller_unittest.mm

Issue 14631005: Enable users of NotificationUIManager to specify binary images. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove fake image from synced notification, stop checking icon_url in unit test. 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: ui/message_center/cocoa/notification_controller_unittest.mm
diff --git a/ui/message_center/cocoa/notification_controller_unittest.mm b/ui/message_center/cocoa/notification_controller_unittest.mm
index 71cd328d8bb38ec603558c95ccf4e137f6fe74da..49630f67efe5e3953118a62fd857805e74298a4a 100644
--- a/ui/message_center/cocoa/notification_controller_unittest.mm
+++ b/ui/message_center/cocoa/notification_controller_unittest.mm
@@ -99,6 +99,7 @@ TEST_F(NotificationControllerTest, BasicLayout) {
"",
ASCIIToUTF16("Added to circles"),
ASCIIToUTF16("Jonathan and 5 others"),
+ gfx::Image(),
string16(),
std::string(),
NULL,
@@ -128,6 +129,7 @@ TEST_F(NotificationControllerTest, OverflowText) {
ASCIIToUTF16("And even the message is long. This sure is a wordy "
"notification. Are you really going to read this "
"entire thing?"),
+ gfx::Image(),
string16(),
std::string(),
NULL,
@@ -148,6 +150,7 @@ TEST_F(NotificationControllerTest, Close) {
"an_id",
string16(),
string16(),
+ gfx::Image(),
string16(),
std::string(),
NULL,
@@ -174,6 +177,7 @@ TEST_F(NotificationControllerTest, Update) {
ASCIIToUTF16("A simple title"),
ASCIIToUTF16("This message isn't too long and should fit in the"
"default bounds."),
+ gfx::Image(),
string16(),
std::string(),
NULL,
@@ -207,6 +211,7 @@ TEST_F(NotificationControllerTest, Buttons) {
"an_id",
string16(),
string16(),
+ gfx::Image(),
string16(),
std::string(),
&buttons,
@@ -231,6 +236,7 @@ TEST_F(NotificationControllerTest, Image) {
"an_id",
string16(),
string16(),
+ gfx::Image(),
string16(),
std::string(),
NULL,

Powered by Google App Engine
This is Rietveld 408576698