1323631 - The 1st Frame after seeking is not rendered

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: Audio/Video, defect, P1)

Description

[John Lin [:jhlin][:jolin]]

Remote decoding relies on VideoData::Listener to update surface content, but VideoData::ShallowCopyUpdateTimestamp() called from [1] doesn't copy the registered listener.

[1] http://searchfox.org/mozilla-central/source/dom/media/AccurateSeekTask.cpp#395

Comment 1

[John Lin [:jhlin][:jolin]]

Attachment: Bug 1323631 - update original frame timestamp or duration rather than creating shallow copy.

Review commit: https://reviewboard.mozilla.org/r/98812/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/98812/

Comment 2

[John Lin [:jhlin][:jolin]]

Comment on attachment 8818849 [details]
Bug 1323631 - update original frame timestamp or duration rather than creating shallow copy.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/98812/diff/1-2/

Comment 3

[JW Wang [:jwwang] [:jw_wang]]

Comment on attachment 8818849 [details]
Bug 1323631 - update original frame timestamp or duration rather than creating shallow copy.

https://reviewboard.mozilla.org/r/98812/#review99238

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b4ac412412f1
update original frame timestamp or duration rather than creating shallow copy. r=jwwang

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b4ac412412f1