102721 – [chromium] Make lazy image decoding thread-safe

Bug 102721 - [chromium] Make lazy image decoding thread-safe

Summary: [chromium] Make lazy image decoding thread-safe

Attachments
Patch (25.68 KB, patch) 2012-11-19 15:34 PST, Hin-Chung Lam	no flags	Details \| Formatted Diff \| Diff
Patch (9.11 KB, patch) 2012-11-19 15:35 PST, Hin-Chung Lam	no flags	Details \| Formatted Diff \| Diff
Patch (9.24 KB, patch) 2012-11-19 18:32 PST, Hin-Chung Lam	no flags	Details \| Formatted Diff \| Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Hin-Chung Lam 2012-11-19 15:18:52 PST

After image decoding operation is recorded make sure decode and playback and happen safely on impl thread.

Comment 1 Hin-Chung Lam 2012-11-19 15:34:47 PST

Created attachment 175054 [details]
Patch

Comment 2 Hin-Chung Lam 2012-11-19 15:35:25 PST

Created attachment 175055 [details]
Patch

Comment 3 Hin-Chung Lam 2012-11-19 18:32:17 PST

Created attachment 175108 [details]
Patch

Comment 4 Hin-Chung Lam 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.

Comment 5 Stephen White 2012-11-20 13:58:19 PST

Comment on attachment 175108 [details]
Patch

OK.  r=me

Comment 6 Nat Duca 2012-11-20 14:04:07 PST

Comment on attachment 175108 [details]
Patch

Omnomnom lgtm

Comment 7 WebKit Review Bot 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>

Comment 8 WebKit Review Bot 2012-11-20 14:27:51 PST

All reviewed patches have been landed.  Closing bug.