Bug 1331899 - Fix lint errors, a=testonly draft
authorJames Graham <james@hoppipolla.co.uk>
Mon, 09 Jan 2017 11:25:58 +0000
changeset 462970 9630e7823a62abc6bfa2d29cc53dcc5ab0082771
parent 462969 743b837bfc24c502555f53de2f85d532dd0c0e42
child 462971 8421a79719237042e300873834f462d27f568bcd
push id41928
push userbmo:james@hoppipolla.co.uk
push dateWed, 18 Jan 2017 10:52:19 +0000
reviewerstestonly
bugs1331899
milestone53.0a1
Bug 1331899 - Fix lint errors, a=testonly MozReview-Commit-ID: 7AI7CAiEwop
testing/web-platform/tests/html/semantics/embedded-content/the-img-element/update-src-complete.html
testing/web-platform/tests/uievents/order-of-events/focus-events/focus-automated-blink-webkit.html
--- a/testing/web-platform/tests/html/semantics/embedded-content/the-img-element/update-src-complete.html
+++ b/testing/web-platform/tests/html/semantics/embedded-content/the-img-element/update-src-complete.html
@@ -1,23 +1,23 @@
-<!doctype html>
+<!doctype html>
 <meta charset="utf-8">
 <title>Changing the img src should retain the 'complete' property</title>
 <script src="/resources/testharness.js"></script>
 <script src="/resources/testharnessreport.js"></script>
-<p id="display"><img src="image.png"></p>
-<script>
-    function check() {
-        var img = document.querySelector("img");
-        assert_true(img.complete, "By onload, image should have loaded");
-        img.src = `image.png?${Math.random()}`;
-        assert_false(img.complete, "Now that we're loading we should no longer be complete");
-        img.onload = function () {
-            assert_true(img.complete, "The new thing should have loaded.");
-            done();
-        }
-    }
-
-    onload = function () {
-        check();
+<p id="display"><img src="image.png"></p>
+<script>
+    function check() {
+        var img = document.querySelector("img");
+        assert_true(img.complete, "By onload, image should have loaded");
+        img.src = `image.png?${Math.random()}`;
+        assert_false(img.complete, "Now that we're loading we should no longer be complete");
+        img.onload = function () {
+            assert_true(img.complete, "The new thing should have loaded.");
+            done();
+        }
+    }
+
+    onload = function () {
+        check();
     };
-
-</script>
+
+</script>
--- a/testing/web-platform/tests/uievents/order-of-events/focus-events/focus-automated-blink-webkit.html
+++ b/testing/web-platform/tests/uievents/order-of-events/focus-events/focus-automated-blink-webkit.html
@@ -1,19 +1,18 @@
 <!DOCTYPE html>
 <!-- Modified from Chris Rebert's manual version -->
 <!-- This documents the behavior according to blink's implementation -->
 <html>
   <head>
     <meta charset="utf-8">
     <title>Focus-related events should fire in the correct order</title>
     <link rel="help" href="https://w3c.github.io/uievents/#events-focusevent-event-order">
-    <meta name="flags" content="interact">
-  <script src="/resources/testharness.js"></script>
-  <script src="/resources/testharnessreport.js"></script>
+    <script src="/resources/testharness.js"></script>
+    <script src="/resources/testharnessreport.js"></script>
   </head>
   <body id="body">
     <input type="text" id="a" value="First">
     <input type="text" id="b" value="Second">
     <br>
     <input type="text" id="c" value="Third">
     <iframe id="iframe">
     </iframe>
@@ -80,21 +79,16 @@
         for (var i = 0; i < inputs.length; i++) {
           for (var k = 0; k < relevantEvents.length; k++) {
             inputs[i].addEventListener(relevantEvents[k], record, false);
           }
         }
 
         a.addEventListener('focusin', function() { b.focus(); }, false);
         b.addEventListener('focusin', function() {
-            console.log(events['normal']);
-            console.log(targets['normal']);
-            console.log(relatedTargets['normal']);
-            console.log(focusedElements['normal']);
-
             async_test_normal.step( function() {
                 assert_array_equals(
                   events['normal'],
                   ['focus', 'focusin', 'blur', 'focusout', 'focus', 'focusin'],
                   'Focus-related events should fire in this order: focusin, focus, focusout, focusin, blur, focus'
                 );
 
                 assert_array_equals(
@@ -120,21 +114,16 @@
 
             b.addEventListener('focusout', function() { finish(); c.focus(); });
             b.blur();
 
             }, false);
 
         c.addEventListener('focusin', function() {d.focus();});
         d.addEventListener('focusin', function() {
-            console.log(events['iframe']);
-            console.log(targets['iframe']);
-            console.log(relatedTargets['iframe']);
-            console.log(focusedElements['iframe']);
-
             async_test_iframe_static.step(function() {
                 assert_array_equals(
                   events['iframe'],
                   ['focus', 'focusin', 'blur', 'focusout', 'focus', 'focusin'],
                   'Focus-related events should fire in this order: focusin, focus, focusout, focusin, blur, focus'
                 );
 
                 assert_array_equals(