Closed Bug 1300050 Opened 3 years ago Closed 3 years ago

Simplify synchronous waiting in ImageBridgeChild

Categories

(Core :: Graphics: Layers, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla51
Tracking Status
firefox51 --- fixed

People

(Reporter: dvander, Assigned: dvander)

References

Details

Attachments

(1 file)

Attached patch patchSplinter Review
This isn't the biggest lines-of-code win, but it makes these methods a lot easier to follow and removes a lot of boilerplate.

When we switch IBC to an XPCOM thread (not on my todo-list, but just sayin') then we can just delete all this synchronous task gunk.
Attachment #8787522 - Flags: review?(matt.woodrow)
Attachment #8787522 - Flags: review?(matt.woodrow) → review+
Pushed by danderson@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4bc688969533
Remove monitor-related boilerplate in ImageBridgeChild.cpp. (bug 1300050, r=mattwoodrow)
Pushed by danderson@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/92ca8f21c671
Remove monitor-related boilerplate in ImageBridgeChild.cpp. (bug 1300050, r=mattwoodrow)
Flags: needinfo?(dvander)
https://hg.mozilla.org/mozilla-central/rev/92ca8f21c671
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.