Add logs to debug test_bug495145.html timeout

RESOLVED FIXED in Firefox 58

Status

()

Core
Audio/Video: Playback
P5
normal
RESOLVED FIXED
26 days ago
25 days ago

People

(Reporter: jwwang, Assigned: jwwang)

Tracking

(Blocks: 1 bug)

unspecified
mozilla58
Points:
---

Firefox Tracking Flags

(firefox58 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(5 attachments)

(Assignee)

Description

26 days ago
Per https://bugzilla.mozilla.org/show_bug.cgi?id=1404373#c10.
(Assignee)

Updated

26 days ago
Assignee: nobody → jwwang
Blocks: 1404373
Priority: -- → P5
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
(Assignee)

Updated

26 days ago
Attachment #8921772 - Flags: review?(gsquelart)
Attachment #8921773 - Flags: review?(gsquelart)
Attachment #8921774 - Flags: review?(gsquelart)
Attachment #8921775 - Flags: review?(gsquelart)
Attachment #8921776 - Flags: review?(gsquelart)

Comment 6

26 days ago
mozreview-review
Comment on attachment 8921772 [details]
Bug 1411476. P1 - add a space before mAudioCompleted and run clang-format.

https://reviewboard.mozilla.org/r/192798/#review198066
Attachment #8921772 - Flags: review?(gsquelart) → review+

Comment 7

26 days ago
mozreview-review
Comment on attachment 8921773 [details]
Bug 1411476. P2 - show duration in debug logs.

https://reviewboard.mozilla.org/r/192800/#review198072
Attachment #8921773 - Flags: review?(gsquelart) → review+

Comment 8

26 days ago
mozreview-review
Comment on attachment 8921774 [details]
Bug 1411476. P3 - show a more descriptive message in DecodeError().

https://reviewboard.mozilla.org/r/192802/#review198074
Attachment #8921774 - Flags: review?(gsquelart) → review+

Comment 9

26 days ago
mozreview-review
Comment on attachment 8921775 [details]
Bug 1411476. P4 - rewrite GetDebugInfo() functions to make string concatenation easier.

https://reviewboard.mozilla.org/r/192804/#review198076
Attachment #8921775 - Flags: review?(gsquelart) → review+

Comment 10

26 days ago
mozreview-review
Comment on attachment 8921776 [details]
Bug 1411476. P5 - dump debug info for ChannelMediaResource.

https://reviewboard.mozilla.org/r/192806/#review198078
Attachment #8921776 - Flags: review?(gsquelart) → review+
(Assignee)

Comment 11

26 days ago
Thanks for the review!

Comment 12

26 days ago
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1b9b85bd44aa
P1 - add a space before mAudioCompleted and run clang-format. r=gerald
https://hg.mozilla.org/integration/autoland/rev/b197f21686c7
P2 - show duration in debug logs. r=gerald
https://hg.mozilla.org/integration/autoland/rev/a95d5ff230bc
P3 - show a more descriptive message in DecodeError(). r=gerald
https://hg.mozilla.org/integration/autoland/rev/dfaab2423219
P4 - rewrite GetDebugInfo() functions to make string concatenation easier. r=gerald
https://hg.mozilla.org/integration/autoland/rev/3a04b939c55a
P5 - dump debug info for ChannelMediaResource. r=gerald

Comment 13

25 days ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/1b9b85bd44aa
https://hg.mozilla.org/mozilla-central/rev/b197f21686c7
https://hg.mozilla.org/mozilla-central/rev/a95d5ff230bc
https://hg.mozilla.org/mozilla-central/rev/dfaab2423219
https://hg.mozilla.org/mozilla-central/rev/3a04b939c55a
Status: NEW → RESOLVED
Last Resolved: 25 days ago
status-firefox58: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.