Inspect Element broken for same-origin iframes with fission + contenttoolbox.fission
Categories
(DevTools :: Inspector, defect, P3)
Tracking
(Fission Milestone:M6c, firefox80 fixed)
| Tracking | Status | |
|---|---|---|
| firefox80 | --- | fixed |
People
(Reporter: jdescottes, Assigned: jdescottes)
References
(Blocks 1 open bug)
Details
(Whiteboard: dt-fission-m2-mvp)
Attachments
(5 files, 1 obsolete file)
Consider the following tree:
(served on domain-a.com)
<html>
<body>
<iframe src="domain-b.com/page1">
#document
body
<iframe src="domain-b.com/page2">
#document
body
<div>inspectme</div>
If fission.autostart and devtools.contenttoolbox.fission are both true, right clicking + "Inspect Element" on inspectme will not select the element in the markup view.
|
Assignee | |
Comment 1•4 years ago
|
||
|
|
Assignee | |
Comment 2•4 years ago
|
||
|
||
Comment 4•4 years ago
|
||
This bug can be really confusing and not so rare.
So I wish we could get to it within M2 timeframe.
It would be also important to keep DAMP test running (it has been disabled in bug 1646636)
|
|
Assignee | |
Comment 5•4 years ago
|
||
Agreed.
It would be also important to keep DAMP test running (it has been disabled in bug 1646636)
To clarify, we only skip the inspector "custom" test with Fission in bug 1646636, the rest of the DAMP tests still runs as usual.
Still a pretty big impact because a lot of inspector DAMP tests are part of the custom test.
|
|
Assignee | |
Comment 6•4 years ago
|
||
|
||
Updated•4 years ago
|
|
||
Comment 9•4 years ago
|
||
Tracking dt-fission-m2-mvp bugs for Fission Nightly milestone (M6c).
|
||
Comment 10•4 years ago
|
||
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7524460d2ad8 Check parent browsing context ids to find a valid BrowsingContextTarget r=ochameau,rcaliman https://hg.mozilla.org/integration/autoland/rev/8775afb03de6 Re-enable context menu tests that were skipped for fission r=ochameau,perftest-reviewers,Bebe https://hg.mozilla.org/integration/autoland/rev/8803eb351bb1 Add a test for Inspect Element with nested remote frames r=rcaliman,ochameau
|
||
Comment 11•4 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/7524460d2ad8
https://hg.mozilla.org/mozilla-central/rev/8775afb03de6
https://hg.mozilla.org/mozilla-central/rev/8803eb351bb1
Description
•