Bug 1825087 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

STR:
1. Load a video on Yahoo Finance ([example](https://finance.yahoo.com/video/david-pogue-reviews-waterpic-sonic-162926350.html))
2. Open PiP window
3. Make sure there is sound on the video
4. Press mute button on PiP window

Expected results:
Mute button should become unmute button on Yahoo Finance video player, matching PiP window unmute button.

Actual results:
Mute button on Yahoo Finance video player remains unchanged visually.

Note:
The mute functionality actually works, it just looks visually inconsistent. Also, this will work on any other site using the Yahoo wrapper. Currently, the wrapper is named [yahoofinance.js](https://searchfox.org/mozilla-central/rev/f9beb753a84aa297713d1565dcd0c5e3c66e4174/browser/extensions/pictureinpicture/video-wrappers/yahoofinance.js), but this will be renamed to `yahoo.js` after landing Bug 1812310.
STR:
1. Load a video on Yahoo Finance ([example](https://finance.yahoo.com/video/david-pogue-reviews-waterpic-sonic-162926350.html))
2. Open PiP window
3. Make sure there is sound on the video
4. Press mute button on PiP window

Expected results:
Mute button should become unmute button on Yahoo Finance video player, matching PiP window unmute button.

Actual results:
Mute button on Yahoo Finance video player remains unchanged visually.

Note:
The mute functionality actually works, it just looks visually inconsistent. Also, this will work on any other site using the [Yahoo wrapper](https://searchfox.org/mozilla-central/rev/6e4dc22cb30c9ffbfee917268c42b17ab8534b91/browser/extensions/pictureinpicture/video-wrappers/yahoo.js).

Back to Bug 1825087 Comment 0