From f8bea85c88cb9be059f0fab65b46d4fb40316f86 Mon Sep 17 00:00:00 2001
From: Pavel Feldman <pavel.feldman@gmail.com>
Date: Fri, 1 May 2020 12:59:18 -0700
Subject: [PATCH] feat(webkit): disable pause on click (#2077)

---
 src/webkit/wkPage.ts |  1 -
 test/click.spec.js   | 30 ++++++++++++++++++++++++++----
 2 files changed, 26 insertions(+), 5 deletions(-)

diff --git a/src/webkit/wkPage.ts b/src/webkit/wkPage.ts
index 127860ba50..9471ddbbdc 100644
--- a/src/webkit/wkPage.ts
+++ b/src/webkit/wkPage.ts
@@ -762,7 +762,6 @@ export class WKPage implements PageDelegate {
   }
 
   async setActivityPaused(paused: boolean): Promise<void> {
-    await this._session.send('Page.setActivityPaused', { paused });
   }
 
   async getContentQuads(handle: dom.ElementHandle): Promise<types.Quad[] | null> {
diff --git a/test/click.spec.js b/test/click.spec.js
index afe8823c40..037c814cc1 100644
--- a/test/click.spec.js
+++ b/test/click.spec.js
@@ -584,7 +584,7 @@ describe('Page.click', function() {
     expect(await page.evaluate(() => window.clicked)).toBe(undefined);
     expect(error.message).toContain('timeout exceeded');
   });
-  it.fail(CHROMIUM || FFOX)('should pause animations', async({page}) => {
+  it.skip(true)('should pause animations', async({page}) => {
     // This test requires pausing the page.
     await page.setContent(`<style>
       @keyframes spinner {
@@ -605,7 +605,7 @@ describe('Page.click', function() {
     await page.click('#target', { __testHookBeforeHitTarget: () => new Promise(f => setTimeout(f, 1000)) });
     expect(await page.evaluate(() => window.clicked)).toBe(true);
   });
-  it.fail(CHROMIUM || FFOX)('should defer timers', async({page}) => {
+  it.skip(true)('should defer timers', async({page}) => {
     // This test requires pausing the page.
     await page.setContent(`<button id=button onclick="window.clicked=true">Click me</button>`);
     await page.click('button', { __testHookBeforeHitTarget: async () => {
@@ -616,7 +616,7 @@ describe('Page.click', function() {
     }});
     expect(await page.evaluate(() => window.clicked)).toBe(true);
   });
-  it.fail(CHROMIUM || FFOX)('should defer rafs', async({page}) => {
+  it.skip(true)('should defer rafs', async({page}) => {
     // This test requires pausing the page.
     await page.setContent(`<button id=button onclick="window.clicked=true">Click me</button>`);
     await page.click('button', { __testHookBeforeHitTarget: async () => {
@@ -627,7 +627,7 @@ describe('Page.click', function() {
     }});
     expect(await page.evaluate(() => window.clicked)).toBe(true);
   });
-  it.fail(CHROMIUM || FFOX)('should defer fetch', async({page, server}) => {
+  it.skip(true)('should defer fetch', async({page, server}) => {
     // This test requires pausing the page.
     await page.goto(server.EMPTY_PAGE);
     await page.setContent(`<button id=button onclick="window.clicked=true">Click me</button>`);
@@ -639,6 +639,28 @@ describe('Page.click', function() {
     }});
     expect(await page.evaluate(() => window.clicked)).toBe(true);
   });
+  it('should dispatch microtasks in order', async({page, server}) => {
+    await page.setContent(`
+      <button id=button>Click me</button>
+      <script>
+        let mutationCount = 0;
+        const observer = new MutationObserver((mutationsList, observer) => {
+          for(let mutation of mutationsList)
+            ++mutationCount;
+        });
+        observer.observe(document.body, { attributes: true, childList: true, subtree: true });
+        button.addEventListener('mousedown', () => {
+          mutationCount = 0;
+          document.body.appendChild(document.createElement('div'));
+        });
+        button.addEventListener('mouseup', () => {
+          window.result = mutationCount;
+        });
+      </script>
+    `);
+    await page.click('button');
+    expect(await page.evaluate(() => window.result)).toBe(1);
+  });
 });
 
 describe('Page.check', function() {