Summary: | ThreadRestrictionVerifier needs a mode where an object is tied to a particular dispatch queue | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Mark Rowe (bdash) <mrowe> | ||||
Component: | Web Template Framework | Assignee: | Mark Rowe (bdash) <mrowe> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | levin | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Mark Rowe (bdash)
2011-08-05 12:58:40 PDT
I’ve typed up a patch and am waiting on a build to confirm that it makes the asserts go away. It should be ready for review in an hour or so. Created attachment 103112 [details]
Patch
Comment on attachment 103112 [details]
Patch
I wonder if we could add a test to TestWebKitAPI for this?
Comment on attachment 103112 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=103112&action=review > Source/JavaScriptCore/wtf/ThreadRestrictionVerifier.h:57 > + , m_owningQueue(0) This should be inside an #if HAVE(DISPATCH_H) (In reply to comment #3) > (From update of attachment 103112 [details]) > I wonder if we could add a test to TestWebKitAPI for this? I have no idea. The original patch also didn’t have tests. Thanks for the fix (and for getting someone to review it quickly). I'm out and without much internet access right now. I'll add a bug to add tests for the original patch and this one. |