Make the Iframe picker Fission compatible
Categories
(DevTools :: Framework, defect, P2)
Tracking
(Fission Milestone:Future)
Fission Milestone | Future |
People
(Reporter: Honza, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: dt-fission-future, dt-perf-stability-mvp )
The IFrame Picker button available in Toolbox Toolbar - used to select an iframe as the currently targeted document should continue working in Fission world.
Honza
Reporter | ||
Updated•5 years ago
|
Updated•5 years ago
|
Comment 1•5 years ago
|
||
This would only be doable once we fully support target switching, which itself would only be possible once we support early resources correctly.
So this will be actionnable only once M2 is completed.
Comment 2•5 years ago
|
||
dt-fission-m2-reserve bugs do not need to block Fission Nightly (M6). For now, let's track them for Fission riding the trains to Beta (M7) so we revisit these bugs before we ship Fission.
Comment 3•4 years ago
|
||
Bulk move of all dt-fission-m2-reserve bugs to Fission MVP milestone.
Reporter | ||
Updated•4 years ago
|
Reporter | ||
Updated•4 years ago
|
Comment 4•4 years ago
|
||
I'm wondering if this depends on bug 1602748, where we would spawn one target per WindowGlobal.
Here for the iframe picker, we need to spawn targets for in-process iframe. Today, they are being debugged via the top level target.
But it would be nice if all iframes were debugged the same way, via a dedicated target.
But doing this means that all features currently broken for remote iframe, will suddently also be broken for in-process iframes.
Otherwise, it means that the iframe picker has to handle both codepath, which may be very hairy:
- in-process iframes, with existing code, listening to
frameUpdate
event on top level target, and usingBrowsingContextTargetActor.switchToFrame
, in order to do a server side "document switching", - remote iframes, with new code to write, listening to target available, and doing a target switching on the client side, via
TargetList.switchToTarget
.
Comment 5•4 years ago
|
||
Depends on 1685500 in order to emit targets for in-process iframes.
Comment 6•4 years ago
|
||
Moving "dt-fission-m3-reserve" bugs to "dt-fission-future" because they don't block Fission MVP.
Updated•3 years ago
|
Reporter | ||
Updated•3 years ago
|
Comment 7•3 years ago
|
||
(this doesn't really block the evaluation context selector work)
Comment 9•3 years ago
|
||
With EFT enabled, the iframe picker is definitely Fission compatible
Description
•