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

Unified Diff: third_party/WebKit/LayoutTests/fast/dom/Geolocation/notimer-after-unload.html

Issue 2667393002: Stop using script-tests in fast/dom/. (Closed)
Patch Set: . Created 3 years, 11 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/LayoutTests/fast/dom/Geolocation/notimer-after-unload.html
diff --git a/third_party/WebKit/LayoutTests/fast/dom/Geolocation/notimer-after-unload.html b/third_party/WebKit/LayoutTests/fast/dom/Geolocation/notimer-after-unload.html
index 5c18e449571bcd8acfbb70cf012045dbceb4fea9..fdd4491c105295f7ce3d66bf600c4d4451f9fbac 100644
--- a/third_party/WebKit/LayoutTests/fast/dom/Geolocation/notimer-after-unload.html
+++ b/third_party/WebKit/LayoutTests/fast/dom/Geolocation/notimer-after-unload.html
@@ -6,6 +6,28 @@
<script src="resources/geolocation-mock.js"></script>
</head>
<body>
-<script src="script-tests/notimer-after-unload.js"></script>
+<script>
+description("Tests that no timers will trigger for navigator.geolocation object after onunload.");
+
+if (!window.testRunner)
+ debug('This test can not run without testRunner');
+
+geolocationServiceMock.then(mock => {
+ mock.setGeolocationPermission(true);
+
+ location = "data:text/html,You should have seen one unload alert appear.<script>window.setTimeout('if (window.testRunner) testRunner.notifyDone();', 100);</" + "script>";
+});
+
+document.body.onunload = function() {
+ navigator.geolocation.getCurrentPosition(
+ function(p) {alert('FAIL: Unexpected Geolocation success callback.');},
+ function(e) {alert('FAIL: Unexpected Geolocation error callback.' + e.code + e.message);},
+ {timeout: 0, maximumAge:0}
+ );
+ alert("unload-called");
+}
+
+window.jsTestIsAsync = true;
+</script>
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698