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

Unified Diff: third_party/WebKit/Source/modules/notifications/NotificationDataTest.cpp

Issue 2392543004: Reflow comments in the Push and Notification modules. (Closed)
Patch Set: Created 4 years, 2 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: third_party/WebKit/Source/modules/notifications/NotificationDataTest.cpp
diff --git a/third_party/WebKit/Source/modules/notifications/NotificationDataTest.cpp b/third_party/WebKit/Source/modules/notifications/NotificationDataTest.cpp
index 17eaafe2b8fd8e9afd7701d4db64f8dc44a07147..149efe959c6ceae94119a57eb1bd709415a5cc4a 100644
--- a/third_party/WebKit/Source/modules/notifications/NotificationDataTest.cpp
+++ b/third_party/WebKit/Source/modules/notifications/NotificationDataTest.cpp
@@ -104,7 +104,8 @@ TEST_F(NotificationDataTest, ReflectProperties) {
EXPECT_EQ(kNotificationBody, notificationData.body);
EXPECT_EQ(kNotificationTag, notificationData.tag);
- // TODO(peter): Test the various icon properties when ExecutionContext::completeURL() works in this test.
+ // TODO(peter): Test the various icon properties when
+ // ExecutionContext::completeURL() works in this test.
ASSERT_EQ(vibrationPattern.size(), notificationData.vibrate.size());
for (size_t i = 0; i < vibrationPattern.size(); ++i)
@@ -243,8 +244,9 @@ TEST_F(NotificationDataTest, DefaultTimestampValue) {
getExecutionContext(), kNotificationTitle, options, exceptionState);
EXPECT_FALSE(exceptionState.hadException());
- // The timestamp should be set to the current time since the epoch if it wasn't supplied by the developer.
- // "32" has no significance, but an equal comparison of the value could lead to flaky failures.
+ // The timestamp should be set to the current time since the epoch if it
+ // wasn't supplied by the developer. "32" has no significance, but an equal
+ // comparison of the value could lead to flaky failures.
EXPECT_NEAR(notificationData.timestamp, WTF::currentTimeMS(), 32);
}

Powered by Google App Engine
This is Rietveld 408576698