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

Unified Diff: third_party/WebKit/LayoutTests/fast/dom/Geolocation/reentrant-success.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/reentrant-success.html
diff --git a/third_party/WebKit/LayoutTests/fast/dom/Geolocation/reentrant-success.html b/third_party/WebKit/LayoutTests/fast/dom/Geolocation/reentrant-success.html
index 2147c80700643641259617b15e48a62e47ebfb6e..a9f105b8fdb0b42b8beaec9458f439524887acb5 100644
--- a/third_party/WebKit/LayoutTests/fast/dom/Geolocation/reentrant-success.html
+++ b/third_party/WebKit/LayoutTests/fast/dom/Geolocation/reentrant-success.html
@@ -6,6 +6,59 @@
<script src="resources/geolocation-mock.js"></script>
</head>
<body>
-<script src="script-tests/reentrant-success.js"></script>
+<script>
+description("Tests that reentrant calls to Geolocation methods from the success callback are OK.");
+
+var mockLatitude = 51.478;
+var mockLongitude = -0.166;
+var mockAccuracy = 100.0;
+
+var position;
+
+geolocationServiceMock.then(mock => {
+ mock.setGeolocationPermission(true);
+ mock.setGeolocationPosition(mockLatitude,
+ mockLongitude,
+ mockAccuracy);
+
+ var successCallbackInvoked = false;
+ navigator.geolocation.getCurrentPosition(function(p) {
+ if (successCallbackInvoked) {
+ testFailed('Success callback invoked unexpectedly');
+ finishJSTest();
+ }
+ successCallbackInvoked = true;
+
+ position = p;
+ shouldBe('position.coords.latitude', 'mockLatitude');
+ shouldBe('position.coords.longitude', 'mockLongitude');
+ shouldBe('position.coords.accuracy', 'mockAccuracy');
+ debug('');
+ continueTest();
+ }, function(e) {
+ testFailed('Error callback invoked unexpectedly');
+ finishJSTest();
+ });
+
+ function continueTest() {
+ mock.setGeolocationPosition(++mockLatitude,
+ ++mockLongitude,
+ ++mockAccuracy);
+
+ navigator.geolocation.getCurrentPosition(function(p) {
+ position = p;
+ shouldBe('position.coords.latitude', 'mockLatitude');
+ shouldBe('position.coords.longitude', 'mockLongitude');
+ shouldBe('position.coords.accuracy', 'mockAccuracy');
+ finishJSTest();
+ }, function(e) {
+ testFailed('Error callback invoked unexpectedly');
+ finishJSTest();
+ });
+ }
+});
+
+window.jsTestIsAsync = true;
+</script>
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698