test(passive-events): Passive event listener violation doesn't report…

… on passive:false now
GoogleChrome · Oct 6, 2017 · d8015b7 · d8015b7
1 parent bedb9a1
commit d8015b7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lighthouse-cli/test/fixtures/dobetterweb/dbw_tester.html b/lighthouse-cli/test/fixtures/dobetterweb/dbw_tester.html
@@ -209,7 +209,7 @@ <h2>Do better web tester page</h2>
     console.log('touchstart');
   });
 
-  // FAIL
+  // PASS - passive:false doesn't get a warning now. crbug.com/770208
   window.addEventListener('mousewheel', function(e) {
     console.log('mousewheel');
   }, {passive: false});

diff --git a/lighthouse-cli/test/smokehouse/dobetterweb/dbw-expectations.js b/lighthouse-cli/test/smokehouse/dobetterweb/dbw-expectations.js
@@ -158,7 +158,8 @@ module.exports = [
           value: {
             // Note: This would normally be 7 but M56 defaults document-level
             // listeners to passive. See https://www.chromestatus.com/features/5093566007214080
-            length: 4,
+            // Note: Explicitly passive:false doesn't get a warning as of M63: crbug.com/770208
+            length: '>=3',
           },
         },
       },