Closed
Bug 1359717
Opened 7 years ago
Closed 7 years ago
Remove unnecessary SetIsPaused() calls
Categories
(Core :: Gecko Profiler, enhancement)
Core
Gecko Profiler
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: n.nethercote, Assigned: n.nethercote)
Details
Attachments
(1 file)
2.55 KB,
patch
|
mstange
:
review+
|
Details | Diff | Splinter Review |
We pause/unpause the profiler before/after some streaming operations. But these pause/unpause pairs occur with gPSMutex locked, and ActivePS::IsPaused() also requires that gPSMutex be locked. Therefore these pause/unpause pairs cannot be observed, and so this patch removes them.
Assignee | ||
Comment 1•7 years ago
|
||
Attachment #8861783 -
Flags: review?(mstange)
Updated•7 years ago
|
Attachment #8861783 -
Flags: review?(mstange) → review+
Assignee | ||
Comment 2•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/4b70d46e776013af3ee53f055679b92286649bd0 Bug 1359717 - Remove unnecessary SetIsPaused() calls. r=mstange.
Comment 3•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/4b70d46e7760
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in
before you can comment on or make changes to this bug.
Description
•