Closed Bug 1113429 Opened 5 years ago Closed 5 years ago

Optimize idle thread annotations in workers

Categories

(Core :: DOM: Workers, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla37

People

(Reporter: bent.mozilla, Assigned: bent.mozilla)

References

Details

Attachments

(1 file)

Bug 970307 added some idle annotations in the an imperfect spot I think. Hopefully this works the same and will be better.
Comment on attachment 8538871 [details] [diff] [review]
Patch, v1

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

The previous spot didn't take nested event loops into account either...
Duplicate of this bug: 1113428
Attachment #8538871 - Flags: review?(nfroyd) → review+
https://hg.mozilla.org/mozilla-central/rev/3285d8bef13f
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
sorry had to back this out in https://treeherder.mozilla.org/#/jobs?repo=b2g-inbound&revision=614d52bca5ac since one of this changes broke ics debug tests with failures like :

https://treeherder.mozilla.org/logviewer.html#?job_id=1958358&repo=b2g-inbound
Flags: needinfo?(cyu)
Canceling NI flag. The bustage is fixed and the change is repushed.
Flags: needinfo?(cyu)
You need to log in before you can comment on or make changes to this bug.