Bug 102721

Summary: [chromium] Make lazy image decoding thread-safe
Product: WebKit Reporter: Hin-Chung Lam <hclam>
Component: ImagesAssignee: Hin-Chung Lam <hclam>
Status: RESOLVED FIXED    
Severity: Normal CC: cc-bugs, enne, jamesr, levin+threading, nduca, qinmin, reed, senorblanco, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 99790    
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Hin-Chung Lam
Reported 2012-11-19 15:18:52 PST
After image decoding operation is recorded make sure decode and playback and happen safely on impl thread.
Attachments
Patch (25.68 KB, patch)
2012-11-19 15:34 PST, Hin-Chung Lam 		no flags
DetailsFormatted DiffDiff
Patch (9.11 KB, patch)
2012-11-19 15:35 PST, Hin-Chung Lam 		no flags
DetailsFormatted DiffDiff
Patch (9.24 KB, patch)
2012-11-19 18:32 PST, Hin-Chung Lam 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Hin-Chung Lam
Comment 1 2012-11-19 15:34:47 PST
Created attachment 175054 [details] Patch
Hin-Chung Lam
Comment 2 2012-11-19 15:35:25 PST
Created attachment 175055 [details] Patch
Hin-Chung Lam
Comment 3 2012-11-19 18:32:17 PST
Created attachment 175108 [details] Patch
Hin-Chung Lam
Comment 4 2012-11-20 13:40:00 PST
nduca@, senorblanco@: any comments on this? I want to have this thread safety so I can work backwards to implement the thread safe caching backend.
Stephen White
Comment 5 2012-11-20 13:58:19 PST
Comment on attachment 175108 [details] Patch OK. r=me
Nat Duca
Comment 6 2012-11-20 14:04:07 PST
Comment on attachment 175108 [details] Patch Omnomnom lgtm
WebKit Review Bot
Comment 7 2012-11-20 14:27:48 PST
Comment on attachment 175108 [details] Patch Clearing flags on attachment: 175108 Committed r135313: <http://trac.webkit.org/changeset/135313>
WebKit Review Bot
Comment 8 2012-11-20 14:27:51 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.