1396512 - Min Vid can no longer be always on top

Status: VERIFIED FIXED

(Core :: XPConnect, defect)

Description

[Ciprian Muresan [:cmuresan], Ecosystem QA]

Attachment: min vid not on top.gif

[Affected versions]:
- Nightly 2017-08-30
- Min-Vid 0.4.1 

[Affected Platforms]:
- All Windows
- All Mac
- All Linux

[Prerequisites]:
- Latest Min-Vid add-on (0.4.1) was previously installed on a clean profile from Test Pilot (https://testpilot.firefox.com/experiments/min-vid).

[Steps to reproduce]:
1. Open the browser with the profile from prerequisites and navigate to one of the supported platforms (e.g www.youtube.com).
2. Hover a video thumbnail and click the "Play Now" option
3. Click a window that is behind the Min Vid player.
4. Observe the behavior.

[Expected result]:
- The player remains on top.

[Actual result]:
- The player is moved into background.

[Regression]:
 9:24.80 INFO: Last good revision: a972e0fb330b7229f86b26dfc75279b44a41c25b
 9:24.81 INFO: First bad revision: 0ab0630eec0c15b591b379454fe59344bac62290
 9:24.81 INFO: Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=a972e0fb330b7229f86b26dfc75279b44a41c25b&tochange=0ab0630eec0c15b591b379454fe59344bac62290

[Additional notes]:
- This is also logged in the experiment's Github repository: https://github.com/meandavejustice/min-vid/issues/1023
- Attached a screen cast of the issue

Kris, can you take a look at this?

Comment 1

[Hsin-Yi Tsai (she/her) [:hsinyi]]

NI per comment 0

Comment 2

[Kris Maglione [:kmag]]

Looks like Jared already has a solution.

Comment 3

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

Yup, Kris beat me to it. Fixed by https://github.com/meandavejustice/min-vid/pull/1029. Closing

Comment 4

[Carmen Tataru [:ctataru], Ecosystem QA]

Marking as Verified as the addon is no longer available.