1607624 - [Fission] Port UnselectedTabHover.jsm to JSWindowActors

Status: RESOLVED FIXED

(Toolkit :: Video/Audio Controls, enhancement, P5)

Comment 1

[Chris Peterson [:cpeterson]]

Hi Abdoulaye, do you plan to work on this JSWindowActors bug? The bug priority is P5, suggesting no one plans to work on it soon.

Do we need to fix this bug before we ship Fission? Or is porting UnselectedTabHover.jsm to JSWindowActors just a performance improvement for video controls in remote iframes?

Comment 2

[Bernard Igiri]

Attachment: Bug 1607624 - Migrated UnselectedTabHover to use JSWindowActor

Comment 3

[Bernard Igiri]

Attachment: Bug 1607624

Fixed test.

Depends on D62057

Comment 4

[Abdoulaye O. LY]

(In reply to Chris Peterson [:cpeterson] from comment #1)

Hi Abdoulaye, do you plan to work on this JSWindowActors bug? The bug priority is P5, suggesting no one plans to work on it soon.

Do we need to fix this bug before we ship Fission? Or is porting UnselectedTabHover.jsm to JSWindowActors just a performance improvement for video controls in remote iframes?

Oops, this was lost in my notifications. Anyway, Bernard has a patch and I will just remove myself as the assignee.

Comment 5

[Bernard Igiri]

Attachment: Bug 1607624 - Correcting reference to contentWindow, correcting scope parameter passed to sendMessageActor, and removing unnecessary event declaration in ActorManagerParent.jsm.

Depends on D62098

Comment 6

[Bernard Igiri]

Attachment: Bug 1607624

Applying Mike Conley's suggestion.

Depends on D63313

Comment 7

[Bernard Igiri]

Attachment: Bug 1607624 - Migrating UnselectedTabHover to use JSWindowActor r=NeilDeakin

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8a2e9f0301bd
Migrating UnselectedTabHover to use JSWindowActor r=NeilDeakin

Comment 9

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/8a2e9f0301bd

Comment 10

[BugBot [:suhaib / :marco/ :calixte]]

Since the status are different for nightly and release, what's the status for beta?
For more information, please visit auto_nag documentation.