Closed Bug 1350964 Opened 5 years ago Closed 5 years ago

Add more assertions to TabGroup and DocGroup to ensure non-main-threads aren't accessing main thread only member variables

Categories

(Core :: DOM: Core & HTML, enhancement)

50 Branch
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: smaug, Assigned: smaug)

Details

Attachments

(1 file)

.
Assignee: nobody → bugs
Attached patch assertionsSplinter Review
Attachment #8851667 - Flags: review?(michael)
Comment on attachment 8851667 [details] [diff] [review]
assertions

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

LGTM but the commit message should be updated to match the bug name :)
Attachment #8851667 - Flags: review?(michael) → review+
Yeah, I don't ever have commit messages in patches ;)
Pushed by opettay@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4e2be6839abe
Add more assertions to TabGroup and DocGroup to ensure non-main-threads aren't accessing main thread only member variables, r=mystor
https://hg.mozilla.org/mozilla-central/rev/4e2be6839abe
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.