1190495 - Hoist TaskQueue into XPCOM

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Comment 1

[Bobby Holley (:bholley)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=4e030c8cf8c3

Comment 2

[Bobby Holley (:bholley)]

Attachment: Part 1 - Separate FlushableTaskQueue into its own file. v1

This thing is garbage, and I don't want to hoist it into XPCOM.

Comment 3

[Bobby Holley (:bholley)]

Attachment: Part 2 - Remove TaskQueue::SyncDispatch. v1

Looks like the last consumers of this went away. \o/

Comment 4

[Bobby Holley (:bholley)]

Attachment: Part 3 - Switch TaskQueue entirely to nsRefPtr. v1

Comment 5

[Bobby Holley (:bholley)]

Attachment: Part 4 - Hoist TaskQueue into xpcom. v1

Comment 6

[Bobby Holley (:bholley)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=28fb44b6f892

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/390bedd83929
https://hg.mozilla.org/integration/mozilla-inbound/rev/446acb1f222c
https://hg.mozilla.org/integration/mozilla-inbound/rev/9f162f396c26
https://hg.mozilla.org/integration/mozilla-inbound/rev/030f75609702
https://hg.mozilla.org/integration/mozilla-inbound/rev/f78347b683ee

Comment 8

[Bobby Holley (:bholley)]

(In reply to Bobby Holley (Mostly Out This Week) from comment #6)
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=28fb44b6f892

Sorry for totally missing the B2G L bustage in that try push - I actually looked at it, and then my eyes glazed over on the pip/http stuff.

Comment 9

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/12dbfc219eae

Comment 10

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/390bedd83929
https://hg.mozilla.org/mozilla-central/rev/446acb1f222c
https://hg.mozilla.org/mozilla-central/rev/9f162f396c26
https://hg.mozilla.org/mozilla-central/rev/030f75609702
https://hg.mozilla.org/mozilla-central/rev/f78347b683ee
https://hg.mozilla.org/mozilla-central/rev/12dbfc219eae