1207964 - Remove workaround from bug 1080461

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect, P2)

Description

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

https://hg.mozilla.org/mozilla-central/file/abe43c30d78d/dom/media/MediaDecoderStateMachine.cpp#l1848

Since bug 1114434 is now fixed, can we remove the if statement now?

Comment 1

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

Hi Jonathan,
Can you check if it is fine to remove the workaround now? Thanks.

Comment 2

[Jonathan Hao (inactive) [:jhao]]

(In reply to JW Wang [:jwwang] from comment #1)
> Hi Jonathan,
> Can you check if it is fine to remove the workaround now? Thanks.

I think so, but unfortunately, there's currently a regression (bug 1205209) that makes RTSP not working at all, so I can't test if it's actually fine.

Comment 3

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

Hi Jonathan,
Since bug 1205209 is fixed, can you take this bug now?

Comment 4

[Jonathan Hao (inactive) [:jhao]]

Attachment: Remove workaround from bug 1080461

Hi JW,
Could you review this patch? Thanks.

Comment 5

[Jonathan Hao (inactive) [:jhao]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=4b4ec6e6c78d

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a7da2ababd29

Comment 7

[Carsten Book [:Tomcat]]

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