Removal of geolocation APIs on insecure origins

third_party/WebKit/LayoutTests/http/tests/security/powerfulFeatureRestrictions/old-powerful-features-on-insecure-origin.html

Index: third_party/WebKit/LayoutTests/http/tests/security/powerfulFeatureRestrictions/old-powerful-features-on-insecure-origin.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/security/powerfulFeatureRestrictions/old-powerful-features-on-insecure-origin.html b/third_party/WebKit/LayoutTests/http/tests/security/powerfulFeatureRestrictions/old-powerful-features-on-insecure-origin.html
index d6f795161da9a8fd90717ec5b38ffc469ef792c1..a5c0f38c659a273b3a7a9e4bb56cb70d7cae7a05 100644
--- a/third_party/WebKit/LayoutTests/http/tests/security/powerfulFeatureRestrictions/old-powerful-features-on-insecure-origin.html
+++ b/third_party/WebKit/LayoutTests/http/tests/security/powerfulFeatureRestrictions/old-powerful-features-on-insecure-origin.html
@@ -30,47 +30,16 @@ if (window.location.origin != get_host_info().UNAUTHENTICATED_ORIGIN) {
     // Tests for APIs that are deprecated, but still allowed, on
     // insecure origins
     async_test(function() {
-        navigator.geolocation.getCurrentPosition(
-            this.step_func(function() {
-                this.done();
-            }),
-            this.step_func(function(error) {
-                assert_unreached('getCurrentPosition should succeed, but failed.');
-                this.done();
-            }));
-    }, 'getCurrentPosition');
-
-    // Note that the deprecation message for watchPosition() will be supressed
-    // because it is an exact duplicate of the getCurrentPosition() message.
-    // Thus, this test is really to confirm that it still executes.
-    async_test(function() {
-        navigator.geolocation.watchPosition(
-            this.step_func(function() {
-                this.done();
-            }),
-            this.step_func(function(error) {
-                assert_unreached('watchPosition should succeed, but failed.');
-                this.done();
-            }));
-    }, 'watchPosition');
-
-    async_test(function() {
         testRunner.setMockDeviceMotion(true, 0, true, 0, true, 0,
                                        true, 0, true, 0, true, 0,
                                        true, 0, true, 0, true, 0,
                                        0);
-
-        window.addEventListener('devicemotion', this.step_func(function() {
-            this.done();
-        }));
+        window.addEventListener('devicemotion', this.step_func_done());
     }, 'device motion');
 
     async_test(function() {
         testRunner.setMockDeviceOrientation(11.1, 22.2, 33.3, true);
-
-        window.addEventListener('deviceorientation', this.step_func(function() {
-            this.done();
-        }));
+        window.addEventListener('deviceorientation', this.step_func_done());
     }, 'device orientation');
 
     promise_test(function(test) {
@@ -79,6 +48,21 @@ if (window.location.origin != get_host_info().UNAUTHENTICATED_ORIGIN) {
 
     // Tests for APIs that have been turned off on insecure origins
     async_test(function() {
+        navigator.geolocation.getCurrentPosition(
+            this.unreached_func('getCurrentPosition should fail, but succeeded.'),
+            this.step_func_done());
+    }, 'getCurrentPosition');
+
+    // Note that the deprecation message for watchPosition() will be supressed

[dcheng, 2016/01/06 23:42:37]

Nit: suppressed

[jww, 2016/01/07 02:25:40]

Done.

+    // because it is an exact duplicate of the getCurrentPosition() message.
+    // Thus, this test is really to confirm that it still executes (and fails).
+    async_test(function() {
+        navigator.geolocation.watchPosition(
+            this.unreached_func('watchPosition should fail, but succeeded.'),
+            this.step_func_done());
+    }, 'watchPosition');
+
+    async_test(function() {
         navigator.webkitGetUserMedia({ audio: true, video: true },
             this.unreached_func('navigator.webkitGetUserMedia should call the error callback, but called the success callback instead.'),
             this.step_func_done());