Closed Bug 1673823 Opened 4 years ago Closed 4 years ago

Move navigation event distribution from framescript to the JSWindowActor implementation

Categories

(Remote Protocol :: Marionette, task, P1)

Default
task

Tracking

(Fission Milestone:M7, firefox84 fixed)

RESOLVED FIXED
84 Branch
Fission Milestone M7
Tracking Status
firefox84 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

References

Details

(Whiteboard: [marionette-fission-mvp][simple])

Attachments

(3 files)

To get bug 1669174 fixed we will have to duplicate the distribution of navigation related events to our JSWindowActor implementation. That should be faster than using webprogress listeners (bug 1664165), and would require less code changes by keeping the risk for possible regressions low.

Note that the events that we currently register for MarionetteFrame should be moved to another actor that we could call MarionettePageLoadFrame.

Assignee: nobody → hskupin
Status: NEW → ASSIGNED
Priority: P3 → P1

Tracking marionette-fission-mvp bugs for Fission Beta milestone (M7).

Fission Milestone: --- → M7
Whiteboard: [marionette-fission-mvp] → [marionette-fission-mvp][simple]
Pushed by hskupin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f52c01b27cbb [marionette] Rename actor MarionetteFrame to MarionetteCommands. r=marionette-reviewers,jdescottes https://hg.mozilla.org/integration/autoland/rev/267a6fdaf468 [marionette] Rename actor MarionetteReftestFrame to MarionetteReftest. r=marionette-reviewers,jdescottes https://hg.mozilla.org/integration/autoland/rev/43302f9a8da6 [marionette] Use navigation events from the actor and not the framescript when actors are enabled. r=marionette-reviewers,jdescottes
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: