2058556 - When performing a double-click while holding down the Ctrl key using the Actions API, the dblclick event does not fire.

Status: RESOLVED FIXED

(Remote Protocol :: Agent, defect, P3)

Description

[Alexandra Borovova [:Sasha]]

Reported via https://github.com/mozilla/geckodriver/issues/2253.

Performing a double-click while holding down the Ctrl key, e.g., with:

button = driver.find_element(By.CSS_SELECTOR, "button")
ActionChains(driver)\
    .key_down(Keys.CONTROL)\
    .double_click(button)\
    .key_up(Keys.CONTROL)\
    .perform()

produces only two click events but no dblclick event on all platforms.

From my understanding, that is correct behavior for Mac that results in the context menu being open in this case, but from my testing of Firefox on Linux we should synthesize the double click for other platforms. So I think the code here instead of:

    if (mouseEvent.ctrlKey) {
      if (lazy.AppInfo.isMac) {
        mouseEvent.button = 2;
        state.clickTracker.reset();
      }
    } else {
      mouseEvent.clickCount = state.clickTracker.count + 1;
    }

should be:

    if (mouseEvent.ctrlKey && lazy.AppInfo.isMac) {
        mouseEvent.button = 2;
        state.clickTracker.reset();
    } else {
      mouseEvent.clickCount = state.clickTracker.count + 1;
    }

Comment 1

[Nirmal Advani]

Attachment: Bug 2058556 - Synthesize dblclick events for Ctrl-click on non-Mac platforms r?Sasha

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/3bd4b704767b
https://hg.mozilla.org/integration/autoland/rev/a511fb644cf5
Synthesize dblclick events for Ctrl-click on non-Mac platforms r=Sasha

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/61807 for changes under testing/web-platform/tests

Comment 4

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/a511fb644cf5

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot