Bug 1331899 - Fix lint errors, a=testonly
authorJames Graham <james@hoppipolla.co.uk>
Mon, 09 Jan 2017 11:25:58 +0000
changeset 375127 474e53d47c3534a789cc19b49a029a6bcf3fbe9c
parent 375126 432c0dee9e826a15c3734b24df27cb8493484129
child 375128 53c4c315533af6baf8ca02aff812592364ae6cba
push id6996
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 20:48:21 +0000
treeherdermozilla-beta@d89512dab048 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstestonly
bugs1331899
milestone53.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
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(