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

Unified Diff: LayoutTests/http/tests/notifications/notification-properties.html

Issue 1110223002: Use and add layout tests for BCP 47 language tag syntax validation. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 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: LayoutTests/http/tests/notifications/notification-properties.html
diff --git a/LayoutTests/http/tests/notifications/notification-properties.html b/LayoutTests/http/tests/notifications/notification-properties.html
index 38e05362905fb71bb7a4053658d9e66c1f03317f..e9603bcc2c63bfe55116466fcb3a9723df9efacb 100644
--- a/LayoutTests/http/tests/notifications/notification-properties.html
+++ b/LayoutTests/http/tests/notifications/notification-properties.html
@@ -58,6 +58,13 @@
// Icon URLs should be returned in serialized form.
assert_equals(serializedUrlNotification.icon, "http://example.com/");
+ var invalidLanguageNotification = new Notification("My Notification", {
+ lang: "invalid language"
+ });
+
+ // Invalid languages should be reset to an empty string.
+ assert_equals(invalidLanguageNotification.lang, "");
+
var noTagNotification = new Notification("My Notification"),
emptyTagNotification = new Notification("My Notification", { tag: "" });

Powered by Google App Engine
This is Rietveld 408576698