Bug 75573

Summary: [chromium] Make it safe to delete WorkerFileSystemContextObserver on any thread.
Product: WebKit Reporter: David Levin <levin>
Component: WebKit Misc.Assignee: David Levin <levin>
Status: RESOLVED FIXED    
Severity: Normal CC: dimich, dslomov, levin+threading, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 74666    
Attachments:
Description Flags
Patch
none
Patch
none
Patch dimich: review+

David Levin
Reported 2012-01-04 14:05:29 PST
See summary.
Attachments
Patch (8.88 KB, patch)
2012-01-04 14:23 PST, David Levin 		no flags
DetailsFormatted DiffDiff
Patch (9.61 KB, patch)
2012-01-04 14:27 PST, David Levin 		no flags
DetailsFormatted DiffDiff
Patch (9.70 KB, patch)
2012-01-04 14:28 PST, David Levin 		dimich: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
David Levin
Comment 1 2012-01-04 14:23:12 PST
Created attachment 121154 [details] Patch
David Levin
Comment 2 2012-01-04 14:27:16 PST
Created attachment 121155 [details] Patch
David Levin
Comment 3 2012-01-04 14:28:27 PST
Created attachment 121156 [details] Patch
Dmitry Titov
Comment 4 2012-01-04 14:40:59 PST
Comment on attachment 121156 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=121156&action=review Nice, one comment: > Source/WebKit/chromium/src/WorkerFileSystemCallbacksBridge.cpp:170 > + // so WorkerFileSystemCallbacksBridge will be valid while this class is around. Need to fix comment as discussed offline.
David Levin
Comment 5 2012-01-04 19:11:18 PST
Committed as http://trac.webkit.org/changeset/104111
Note You need to log in before you can comment on or make changes to this bug.