Closed
Bug 1233410
Opened 10 years ago
Closed 9 years ago
Enable OMX wemb decoding
Categories
(Core :: Audio/Video: Playback, defect, P2)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
mozilla47
| Tracking | Status | |
|---|---|---|
| firefox47 | --- | fixed |
People
(Reporter: ayang, Assigned: ayang)
References
Details
Attachments
(3 files)
Enable MSE test case on ICS and KK emulator
|
Assignee | |
Comment 1•10 years ago
|
||
Assignee: nobody → ayang
|
||
Updated•10 years ago
|
Priority: -- → P2
|
||
Comment 2•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/35609/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/35609/
Attachment #8721215 -
Flags: review?(ayang)
|
|
||
Comment 3•9 years ago
|
||
Comment on attachment 8721215 [details]
MozReview Request: Bug 1233410 - Refactor: let component decide which compression format to use. r?alfredo
Review request updated; see interdiff: https://reviewboard.mozilla.org/r/35609/diff/1-2/
|
|
||
Comment 4•9 years ago
|
||
(In reply to John Lin [:jolin][:jhlin] from comment #3)
> Comment on attachment 8721215 [details]
> MozReview Request: Bug 1233410 - Support VP8 video in OpenMAX PDM. r?alfredo
>
> Review request updated; see interdiff:
> https://reviewboard.mozilla.org/r/35609/diff/1-2/
Update for offline comments:
- translate standard VP8 coding type value to proprietary definition.
- translate VP8 video MIME type.
|
|
Assignee | |
Comment 5•9 years ago
|
||
Comment on attachment 8721215 [details]
MozReview Request: Bug 1233410 - Refactor: let component decide which compression format to use. r?alfredo
https://reviewboard.mozilla.org/r/35609/#review32609
LGTM. Thank you.
Attachment #8721215 -
Flags: review?(ayang) → review+
|
|
||
Comment 6•9 years ago
|
||
Comment on attachment 8721215 [details]
MozReview Request: Bug 1233410 - Refactor: let component decide which compression format to use. r?alfredo
Review request updated; see interdiff: https://reviewboard.mozilla.org/r/35609/diff/2-3/
|
|
||
Comment 7•9 years ago
|
||
Comment on attachment 8721215 [details]
MozReview Request: Bug 1233410 - Refactor: let component decide which compression format to use. r?alfredo
Review request updated; see interdiff: https://reviewboard.mozilla.org/r/35609/diff/3-4/
Attachment #8721215 -
Attachment description: MozReview Request: Bug 1233410 - Support VP8 video in OpenMAX PDM. r?alfredo → MozReview Request: Bug 1233410 - Refactor: let component decide which compression format to use. r?alfredo
|
|
||
Comment 8•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/35989/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/35989/
Attachment #8722354 -
Flags: review?(ayang)
|
|
||
Comment 9•9 years ago
|
||
Address offline comments by letting Gonk platform implementation decide which compression format definition to use:
- refactor: introduce a virtual function for subclasses to override format definition when necessary.
- Let Gonk implementation uses Android proprietary definition for VP8 video.
|
|
Assignee | |
Comment 10•9 years ago
|
||
|
|
Assignee | |
Updated•9 years ago
|
Attachment #8722354 -
Flags: review?(ayang) → review+
|
|
Assignee | |
Comment 11•9 years ago
|
||
Comment on attachment 8722354 [details]
MozReview Request: Bug 1233410 - Support VP8 video in OpenMAX PDM. r?alfredo
https://reviewboard.mozilla.org/r/35989/#review32625
Thank you.
|
||
Comment 12•9 years ago
|
||
|
||
Comment 13•9 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/06f762ea5c7f
https://hg.mozilla.org/mozilla-central/rev/f523d4705481
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox47:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
You need to log in
before you can comment on or make changes to this bug.
Description
•