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

Unified Diff: LayoutTests/navigator_language/navigator_language.html

Issue 284793003: Implement languagechange event, fired when accept languages changes. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: fix tests Created 6 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: LayoutTests/navigator_language/navigator_language.html
diff --git a/LayoutTests/navigator_language/navigator_language.html b/LayoutTests/navigator_language/navigator_language.html
new file mode 100644
index 0000000000000000000000000000000000000000..a616050a2abec060f4c7d1e3df62fe27cecf279a
--- /dev/null
+++ b/LayoutTests/navigator_language/navigator_language.html
@@ -0,0 +1,18 @@
+<!DOCTYPE html>
jochen (gone - plz use gerrit) 2014/05/21 09:20:11 where are the expected results?
mlamouri (slow - plz ping) 2014/05/21 16:27:55 testharness tests don't need expected files. They
+<html>
+<body>
+<script src="../resources/testharness.js"></script>
+<script src="../resources/testharnessreport.js"></script>
+<script>
+
+test(function() {
+ assert_true('language' in window.navigator);
jochen (gone - plz use gerrit) 2014/05/21 09:20:11 4spaces indent
mlamouri (slow - plz ping) 2014/05/21 16:27:55 Could Blink and Chrome just find an agreement here
+ assert_false('languages' in window.navigator);
+ assert_true('onlanguagechange' in window);
+}, "Test that NavigatorLanguage API is present in window");
+
+// FIXME: test runner needs to be extended in order to test the feature.
+
+</script>
+</body>
+</html>

Powered by Google App Engine
This is Rietveld 408576698