"W/MPEG4Extractor(71435): Error -1004 parsing chunck at offset 852 looking for first track" spams logs, causes OS X debug build to fail with "Output exceeded 52428800 bytes"

RESOLVED FIXED in Firefox 45

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: botond, Assigned: gerald)

Tracking

Trunk
mozilla45
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox45 fixed)

Details

Attachments

(1 attachment)

Here's a Try push where an OS X debug build fails with "Output exceeded 52428800 bytes" due to this warning spamming the logs:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=931fc5e2d320
Probably my fault, all the new gtests in libstagefright produce a lot of these.

An easy&quick solution would be to reduce the way subsets of files are tested.

A more difficult solution (but better I think) would be to be able to suppress these warnings. They are currently hardcoded to always be printed.
Ideally, they should trickle back to the caller, to be displayed in the console (see bug 1197438).


Because we're having build issues, I will start with the quick&easy solution here.
QA Contact: gsquelart
Assignee: nobody → gsquelart
QA Contact: gsquelart
Disable Stagefright subset gtests.

These tests are exploratory, they don't particularly match whatever the
original issues were discovered, so it is safe to disable them for now.
A future bug will look at re-enabling them, but without all the output
that they generate.
Attachment #8686325 - Flags: review?(jyavenard)
Comment on attachment 8686325 [details] [diff] [review]
1224019-disable-stagefright-subset-gtests.patch

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

Rubber-stamping after :gerald indicated on IRC that :jya gave a verbal ok.
Attachment #8686325 - Flags: review?(jyavenard) → review+
Blocks: 1224052
https://hg.mozilla.org/mozilla-central/rev/cdb6a1c38b89
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
You need to log in before you can comment on or make changes to this bug.