Closed
Bug 1480130
Opened 7 years ago
Closed 7 years ago
Shadow DOM interferes with Cmd+click on wpt staging site
Categories
(Core :: DOM: Events, defect, P2)
Core
DOM: Events
Tracking
()
RESOLVED
FIXED
mozilla63
| Tracking | Status | |
|---|---|---|
| firefox63 | --- | fixed |
People
(Reporter: bzbarsky, Assigned: smaug)
References
Details
Attachments
(1 file)
|
7.01 KB,
patch
|
Felipe
:
review+
|
Details | Diff | Splinter Review |
STEPS TO REPRODUCE (on Mac, but using Ctrl+click on Windows/Linux has the same behavior, I am told):
1) Use nightly, make sure the "dom.webcomponents.shadowdom.enabled" pref is true.
2) Load https://staging.wpt.fyi/results/
3) Cmd+click on the "2dcontext/" link.
EXPECTED RESULTS: New tab opens with https://staging.wpt.fyi/results/2dcontext
ACTUAL RESULTS: Nothing happens.
If I flip "dom.webcomponents.shadowdom.enabled" to false, things work. So presumably shadow DOM messes with the event getting to the browser chrome somehow?
| Reporter | ||
Comment 1•7 years ago
|
||
Per https://github.com/web-platform-tests/wpt.fyi/issues/193 middle-click has the same issue, for probably the same reason.
| Assignee | ||
Comment 2•7 years ago
|
||
e10s and non-e10s use almost exactly the same code
https://searchfox.org/mozilla-central/rev/196560b95f191b48ff7cba7c2ba9237bba6b5b6a/browser/base/content/browser.js#5910
https://searchfox.org/mozilla-central/rev/196560b95f191b48ff7cba7c2ba9237bba6b5b6a/browser/modules/ClickEventHandler.jsm#125
Assignee: nobody → bugs
| Assignee | ||
Comment 3•7 years ago
|
||
We're lacking tests in general here, but this modified existing test which works in both e10s and non-e10s mode.
remote: View your changes here:
remote: https://hg.mozilla.org/try/rev/72e1899f3617e368ce8dd4be16236aeb23055024
remote: https://hg.mozilla.org/try/rev/a626bbb30a71238ba0eb3ca42cb9ec2322c2c85b
remote:
remote: Follow the progress of your build on Treeherder:
remote: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a626bbb30a71238ba0eb3ca42cb9ec2322c2c85b
remote: recorded changegroup in replication log in 0.006s
Attachment #8996842 -
Flags: review?(felipc)
Updated•7 years ago
|
Attachment #8996842 -
Flags: review?(felipc) → review+
Pushed by opettay@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/fbc754cfd3ce
Shadow DOM interferes with Cmd+click handling, r=felipe
Updated•7 years ago
|
Priority: -- → P2
Comment 6•7 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in
before you can comment on or make changes to this bug.
Description
•