MSE addSourceBuffer stops due to PRELOAD_NONE is true on gonk pref.

RESOLVED FIXED in Firefox 38

Status

()

Core
Audio/Video
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: alfredo, Assigned: alfredo)

Tracking

(Blocks: 1 bug)

unspecified
mozilla38
ARM
Gonk (Firefox OS)
Points:
---

Firefox Tracking Flags

(firefox38 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Comment 1

3 years ago
Created attachment 8563868 [details] [diff] [review]
skip_pref_preload_on_gonk
Attachment #8563868 - Flags: review?(cpearce)
Comment on attachment 8563868 [details] [diff] [review]
skip_pref_preload_on_gonk

Review of attachment 8563868 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/html/HTMLMediaElement.cpp
@@ +1078,5 @@
>    } else {
>      // Find the appropriate preload action by looking at the attribute.
>      const nsAttrValue* val = mAttrsAndChildren.GetAttr(nsGkAtoms::preload,
>                                                         kNameSpaceID_None);
> +    // MSE doesn't work if repload is none, so it igonres the pref when src is

s/repload/preload/
s/igonres/ignores/
Attachment #8563868 - Flags: review?(cpearce) → review+
(Assignee)

Comment 3

3 years ago
Created attachment 8564834 [details] [diff] [review]
skip_pref_preload_on_gonk

Thanks for review.
Attachment #8563868 - Attachment is obsolete: true
Attachment #8564834 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/c598faec1736
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox38: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla38

Updated

3 years ago
Depends on: 1155494

Updated

3 years ago
Depends on: 1155489

Updated

3 years ago
No longer depends on: 1155489

Updated

3 years ago
No longer depends on: 1155494
You need to log in before you can comment on or make changes to this bug.