Closed
Bug 1471588
Opened 6 years ago
Closed 6 years ago
Expand on MSGTracing to trace more things on the audio thread
Categories
(Core :: Audio/Video: MediaStreamGraph, enhancement, P2)
Core
Audio/Video: MediaStreamGraph
Tracking
()
RESOLVED
FIXED
mozilla63
Tracking | Status | |
---|---|---|
firefox63 | --- | fixed |
People
(Reporter: pehrsons, Assigned: pehrsons)
Details
Attachments
(8 files)
59 bytes,
text/x-review-board-request
|
padenot
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
padenot
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
padenot
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
padenot
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
padenot
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
padenot
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
padenot
:
review+
|
Details |
59 bytes,
text/x-review-board-request
|
padenot
:
review+
|
Details |
No description provided.
Assignee | ||
Updated•6 years ago
|
Rank: 15
Priority: -- → P2
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 9•6 years ago
|
||
mozreview-review |
Comment on attachment 8988265 [details] Bug 1471588 - Don't pass aComment to PrintBudget, it builds its own comment. https://reviewboard.mozilla.org/r/253512/#review260918
Attachment #8988265 -
Flags: review?(padenot) → review+
Comment 10•6 years ago
|
||
mozreview-review |
Comment on attachment 8988266 [details] Bug 1471588 - Include sample rate in budget comment. https://reviewboard.mozilla.org/r/253514/#review260920
Attachment #8988266 -
Flags: review?(padenot) → review+
Comment 11•6 years ago
|
||
mozreview-review |
Comment on attachment 8988267 [details] Bug 1471588 - Use format strings for TRACE_COMMENT. https://reviewboard.mozilla.org/r/253516/#review260922
Attachment #8988267 -
Flags: review?(padenot) → review+
Comment 12•6 years ago
|
||
mozreview-review |
Comment on attachment 8988268 [details] Bug 1471588 - Add macro for tracing with comment on the audio thread. https://reviewboard.mozilla.org/r/253518/#review260924
Attachment #8988268 -
Flags: review?(padenot) → review+
Comment 13•6 years ago
|
||
mozreview-review |
Comment on attachment 8988269 [details] Bug 1471588 - Turn on MediaPipeline tracing. https://reviewboard.mozilla.org/r/253520/#review260926
Attachment #8988269 -
Flags: review?(padenot) → review+
Comment 14•6 years ago
|
||
mozreview-review |
Comment on attachment 8988270 [details] Bug 1471588 - Add details to SourceMediaStream and TrackUnionStream tracing. https://reviewboard.mozilla.org/r/253522/#review260928
Attachment #8988270 -
Flags: review?(padenot) → review+
Comment 15•6 years ago
|
||
mozreview-review |
Comment on attachment 8988271 [details] Bug 1471588 - Trace all NotifyPull calls. https://reviewboard.mozilla.org/r/253524/#review260932
Attachment #8988271 -
Flags: review?(padenot) → review+
Comment 16•6 years ago
|
||
mozreview-review |
Comment on attachment 8988272 [details] Bug 1471588 - Trace track and stream media consumers. https://reviewboard.mozilla.org/r/253526/#review260934
Attachment #8988272 -
Flags: review?(padenot) → review+
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 25•6 years ago
|
||
Pushed by pehrsons@gmail.com: https://hg.mozilla.org/integration/autoland/rev/75024c16fbbf Don't pass aComment to PrintBudget, it builds its own comment. r=padenot https://hg.mozilla.org/integration/autoland/rev/cb4e0b310d17 Include sample rate in budget comment. r=padenot https://hg.mozilla.org/integration/autoland/rev/d2317efa41cc Use format strings for TRACE_COMMENT. r=padenot https://hg.mozilla.org/integration/autoland/rev/56927ab6c40c Add macro for tracing with comment on the audio thread. r=padenot https://hg.mozilla.org/integration/autoland/rev/0fffc7902a91 Turn on MediaPipeline tracing. r=padenot https://hg.mozilla.org/integration/autoland/rev/3bbc83b7469f Add details to SourceMediaStream and TrackUnionStream tracing. r=padenot https://hg.mozilla.org/integration/autoland/rev/2c429714e908 Trace all NotifyPull calls. r=padenot https://hg.mozilla.org/integration/autoland/rev/de3e3149c345 Trace track and stream media consumers. r=padenot
Comment 26•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/75024c16fbbf https://hg.mozilla.org/mozilla-central/rev/cb4e0b310d17 https://hg.mozilla.org/mozilla-central/rev/d2317efa41cc https://hg.mozilla.org/mozilla-central/rev/56927ab6c40c https://hg.mozilla.org/mozilla-central/rev/0fffc7902a91 https://hg.mozilla.org/mozilla-central/rev/3bbc83b7469f https://hg.mozilla.org/mozilla-central/rev/2c429714e908 https://hg.mozilla.org/mozilla-central/rev/de3e3149c345
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in
before you can comment on or make changes to this bug.
Description
•