1647434 - [MediaControl-Windows] Rework image cache mechanism for SMTC

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, enhancement, P1)

Description

[C.M.Chang[:chunmin]]

There are two problem in the current cache mechanism:

1. The image url would be cached if it's valid. If the image fails to be set a the thumbnail and the next song uses the same image, the thumbnail would not be updated.
2. If the Artwork for track 1, 2 are <image A, image B> and <image B, image C> respectively, when the track of the media session is switched from 2 to 1, the track image (Thumbnail on SMTC) would be updated from image B to image A. However, we could skip the update since image B is acceptable for track 1.

Comment 1

[C.M.Chang[:chunmin]]

Attachment: Bug 1647434 - P1: Rework image cache mechanism for SMTC

Comment 2

[C.M.Chang[:chunmin]]

Attachment: Bug 1647434 - P2: Remove RefreshDisplay()

Depends on D80524

Comment 3

[C.M.Chang[:chunmin]]

Attachment: Bug 1647434 - P3: Set mThumbnailUrl in {Clear, Set}Thumbnail

Depends on D81571

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f224e05e4ec3
P1: Rework image cache mechanism for SMTC r=alwu
https://hg.mozilla.org/integration/autoland/rev/c7728ae84546
P2: Remove RefreshDisplay() r=alwu
https://hg.mozilla.org/integration/autoland/rev/15ade6b9452b
P3: Set mThumbnailUrl in {Clear, Set}Thumbnail r=alwu

Comment 5

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/f224e05e4ec3
https://hg.mozilla.org/mozilla-central/rev/c7728ae84546
https://hg.mozilla.org/mozilla-central/rev/15ade6b9452b