Closed
Bug 1341221
Opened 8 years ago
Closed 7 years ago
Enable mp4 rust parser
Categories
(Core :: Audio/Video: Playback, defect, P3)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
mozilla56
Tracking | Status | |
---|---|---|
firefox56 | --- | fixed |
People
(Reporter: ayang, Assigned: ayang)
References
Details
Attachments
(1 file)
After bug 1331330, 1340446, 1340447 and 1340980. We can pass all tests on try Linux platform [1]. I think it's about time to enable it officially.
Following is my plan:
1. enable it on linux debug mode, so we can see the result on try.
2. enable it on linux release; at this stage, we may need QA get involved finding bugs quickly.
3. enable it on other platforms, like Mac.
4. enable it on remains platforms.
Any idea?
[1] https://treeherder.mozilla.org/#/jobs?repo=try&revision=6f6279e991097eb3fb4d52db0e431d12cd5cd95b
Assignee | ||
Updated•8 years ago
|
Assignee | ||
Comment 1•8 years ago
|
||
As we talked last time, it'd be better to get QA resource at stage 2.
Flags: needinfo?(bwu)
Flags: needinfo?(ajones)
Comment 2•8 years ago
|
||
Rachelle should be able to help us find out QA resource. I have cc-ed her in this bug.
Before that, do we have some ideas about what we expect QA to test and how they should test.
Flags: needinfo?(bwu)
Depends on: 1341483
Updated•8 years ago
|
Flags: needinfo?(ajones)
Assignee | ||
Comment 3•8 years ago
|
||
try result in testmode after applied patch in bug 1343793. I will keep trying.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=53ebf03f2fc79263280ef82eb2b1e22aaf831b07
Assignee | ||
Comment 4•8 years ago
|
||
Pass all tests on linux after applied patches in bug 1330243. Taget to enable linux debug this week if everything goes smoothly.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=68cfc91823fb59221c8ae8b4b20884e5437c07ce
Comment hidden (mozreview-request) |
Assignee | ||
Updated•8 years ago
|
Keywords: leave-open
Comment 6•8 years ago
|
||
mozreview-review |
Comment on attachment 8851860 [details]
Bug 1341221 - enable rust mp4 parser on linux debug build.
https://reviewboard.mozilla.org/r/124080/#review126582
Attachment #8851860 -
Flags: review?(kinetik) → review+
Assignee | ||
Comment 7•8 years ago
|
||
Try looks good on linux debug build.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=7a5014adfec7bc94d9f96d1e735b7ef468bb6881
Pushed by ayang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fb09758c43f3
enable rust mp4 parser on linux debug build. r=kinetik
Comment 9•8 years ago
|
||
bugherder |
Assignee | ||
Comment 10•8 years ago
|
||
To fix corner cases efficiently, it'd be very nice if bug 1341483 can be solved before enabling rust parser on release build.
You'll also want bug 1343442, which is where the error is picked up and displayed as a notification.
(There is some higher priority work for me at the moment, but the code is almost complete, so I'll try to push it for review soon...)
Depends on: 1343442
Updated•7 years ago
|
Priority: -- → P3
Assignee | ||
Updated•7 years ago
|
Assignee | ||
Comment 12•7 years ago
|
||
Due to bug 1370175, I plan to have a patch which is as https://bugzilla.mozilla.org/show_bug.cgi?id=1370175#c9. That actually enable the rust mp4 parser on all platforms.
Assignee | ||
Comment 13•7 years ago
|
||
Close this one since the mp4 rust parser is enable in bug 1370175.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•