RESOLVED FIXED Bug 155322
Tracking bug for CG asynchronous image decoding
https://bugs.webkit.org/show_bug.cgi?id=155322
Summary Tracking bug for CG asynchronous image decoding
Said Abou-Hallawa
Reported 2016-03-10 14:04:13 PST
Tracking bug for forking the image decoding into two separate paths: 1. Synchronous: This is the current implementation where the image decoding usually happens in the whippet thread when an image drawing command is pushed into the CG context. 2. Asynchronous: This is a new mode where an image decoding will start immediately after receiving the first chunk of the encoded image date. And to not block the main thread, this image decoding has to run on a separate thread.
Attachments
Patch (140.30 KB, patch)
2016-03-11 19:24 PST, Said Abou-Hallawa
no flags
Patch (140.30 KB, patch)
2016-03-11 19:28 PST, Said Abou-Hallawa
no flags
Patch (139.87 KB, patch)
2016-03-11 19:35 PST, Said Abou-Hallawa
no flags
Patch (142.79 KB, patch)
2016-03-11 20:12 PST, Said Abou-Hallawa
no flags
Patch (143.35 KB, patch)
2016-03-11 20:38 PST, Said Abou-Hallawa
no flags
Patch (143.35 KB, patch)
2016-03-11 20:42 PST, Said Abou-Hallawa
no flags
Patch (146.90 KB, patch)
2016-03-11 21:35 PST, Said Abou-Hallawa
no flags
Patch (147.95 KB, patch)
2016-03-11 22:05 PST, Said Abou-Hallawa
no flags
Patch (147.94 KB, patch)
2016-03-11 22:42 PST, Said Abou-Hallawa
no flags
Archive of layout-test-results from ews102 for mac-yosemite (1.01 MB, application/zip)
2016-03-11 23:25 PST, Build Bot
no flags
Archive of layout-test-results from ews116 for mac-yosemite (948.76 KB, application/zip)
2016-03-12 00:30 PST, Build Bot
no flags
Patch (150.08 KB, patch)
2016-03-12 02:18 PST, Said Abou-Hallawa
no flags
Patch (150.04 KB, patch)
2016-03-12 02:26 PST, Said Abou-Hallawa
no flags
Archive of layout-test-results from ews100 for mac-yosemite (1.29 MB, application/zip)
2016-03-12 03:20 PST, Build Bot
no flags
Archive of layout-test-results from ews117 for mac-yosemite (879.38 KB, application/zip)
2016-03-12 05:37 PST, Build Bot
no flags
Patch (150.67 KB, patch)
2016-03-12 09:19 PST, Said Abou-Hallawa
no flags
Archive of layout-test-results from ews102 for mac-yosemite (1.06 MB, application/zip)
2016-03-12 10:11 PST, Build Bot
no flags
Archive of layout-test-results from ews114 for mac-yosemite (911.66 KB, application/zip)
2016-03-12 10:24 PST, Build Bot
no flags
Patch (151.13 KB, patch)
2016-03-12 13:34 PST, Said Abou-Hallawa
no flags
Archive of layout-test-results from ews100 for mac-yosemite (879.45 KB, application/zip)
2016-03-12 14:29 PST, Build Bot
no flags
Archive of layout-test-results from ews116 for mac-yosemite (986.71 KB, application/zip)
2016-03-12 14:50 PST, Build Bot
no flags
Patch (151.39 KB, patch)
2016-03-12 20:22 PST, Said Abou-Hallawa
no flags
Patch (153.15 KB, patch)
2016-03-12 20:52 PST, Said Abou-Hallawa
no flags
Archive of layout-test-results from ews101 for mac-yosemite (845.19 KB, application/zip)
2016-03-12 21:46 PST, Build Bot
no flags
Archive of layout-test-results from ews115 for mac-yosemite (923.13 KB, application/zip)
2016-03-12 22:11 PST, Build Bot
no flags
Patch (153.79 KB, patch)
2016-03-12 22:12 PST, Said Abou-Hallawa
no flags
Archive of layout-test-results from ews100 for mac-yosemite (1.04 MB, application/zip)
2016-03-12 23:06 PST, Build Bot
no flags
Patch (153.88 KB, patch)
2016-03-12 23:09 PST, Said Abou-Hallawa
no flags
Archive of layout-test-results from ews101 for mac-yosemite (1000.24 KB, application/zip)
2016-03-13 00:02 PST, Build Bot
no flags
Archive of layout-test-results from ews116 for mac-yosemite (1.02 MB, application/zip)
2016-03-13 00:23 PST, Build Bot
no flags
Patch (154.23 KB, patch)
2016-03-13 00:59 PST, Said Abou-Hallawa
no flags
Archive of layout-test-results from ews115 for mac-yosemite (1.18 MB, application/zip)
2016-03-13 01:55 PST, Build Bot
no flags
Patch (154.26 KB, patch)
2016-03-13 03:51 PDT, Said Abou-Hallawa
no flags
Said Abou-Hallawa
Comment 1 2016-03-11 19:24:23 PST
Said Abou-Hallawa
Comment 2 2016-03-11 19:26:43 PST
This patch is not for review. I am planning to split it into sub patches for review. The purpose of this patch is to fix the build breaks and the test failures.
Said Abou-Hallawa
Comment 3 2016-03-11 19:28:59 PST
Said Abou-Hallawa
Comment 4 2016-03-11 19:35:35 PST
Said Abou-Hallawa
Comment 5 2016-03-11 20:12:32 PST
Said Abou-Hallawa
Comment 6 2016-03-11 20:38:20 PST
Said Abou-Hallawa
Comment 7 2016-03-11 20:42:19 PST
Said Abou-Hallawa
Comment 8 2016-03-11 21:35:01 PST
Said Abou-Hallawa
Comment 9 2016-03-11 22:05:16 PST
Said Abou-Hallawa
Comment 10 2016-03-11 22:42:41 PST
Build Bot
Comment 11 2016-03-11 23:24:56 PST
Comment on attachment 273810 [details] Patch Attachment 273810 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/964479 New failing tests: transitions/extra-transition.html svg/carto.net/scrollbar.svg
Build Bot
Comment 12 2016-03-11 23:25:01 PST
Created attachment 273816 [details] Archive of layout-test-results from ews102 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews102 Port: mac-yosemite Platform: Mac OS X 10.10.5
Build Bot
Comment 13 2016-03-12 00:30:27 PST
Comment on attachment 273810 [details] Patch Attachment 273810 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/964779 New failing tests: svg/carto.net/scrollbar.svg
Build Bot
Comment 14 2016-03-12 00:30:33 PST
Created attachment 273820 [details] Archive of layout-test-results from ews116 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews116 Port: mac-yosemite Platform: Mac OS X 10.10.5
Said Abou-Hallawa
Comment 15 2016-03-12 02:18:47 PST
Said Abou-Hallawa
Comment 16 2016-03-12 02:26:35 PST
Build Bot
Comment 17 2016-03-12 03:20:44 PST
Comment on attachment 273828 [details] Patch Attachment 273828 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/965531 New failing tests: transitions/extra-transition.html
Build Bot
Comment 18 2016-03-12 03:20:50 PST
Created attachment 273831 [details] Archive of layout-test-results from ews100 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews100 Port: mac-yosemite Platform: Mac OS X 10.10.5
Build Bot
Comment 19 2016-03-12 05:37:06 PST
Comment on attachment 273828 [details] Patch Attachment 273828 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/965991 New failing tests: transitions/extra-transition.html svg/carto.net/scrollbar.svg
Build Bot
Comment 20 2016-03-12 05:37:10 PST
Created attachment 273834 [details] Archive of layout-test-results from ews117 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews117 Port: mac-yosemite Platform: Mac OS X 10.10.5
Said Abou-Hallawa
Comment 21 2016-03-12 09:19:50 PST
Build Bot
Comment 22 2016-03-12 10:11:41 PST
Comment on attachment 273838 [details] Patch Attachment 273838 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/967075 New failing tests: transitions/extra-transition.html svg/carto.net/scrollbar.svg
Build Bot
Comment 23 2016-03-12 10:11:46 PST
Created attachment 273841 [details] Archive of layout-test-results from ews102 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews102 Port: mac-yosemite Platform: Mac OS X 10.10.5
Build Bot
Comment 24 2016-03-12 10:24:00 PST
Comment on attachment 273838 [details] Patch Attachment 273838 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/967089 New failing tests: transitions/extra-transition.html svg/carto.net/scrollbar.svg
Build Bot
Comment 25 2016-03-12 10:24:04 PST
Created attachment 273842 [details] Archive of layout-test-results from ews114 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews114 Port: mac-yosemite Platform: Mac OS X 10.10.5
Said Abou-Hallawa
Comment 26 2016-03-12 13:34:46 PST
Build Bot
Comment 27 2016-03-12 14:29:09 PST
Comment on attachment 273847 [details] Patch Attachment 273847 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/967979 New failing tests: transitions/extra-transition.html svg/carto.net/scrollbar.svg
Build Bot
Comment 28 2016-03-12 14:29:13 PST
Created attachment 273851 [details] Archive of layout-test-results from ews100 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews100 Port: mac-yosemite Platform: Mac OS X 10.10.5
Build Bot
Comment 29 2016-03-12 14:50:47 PST
Comment on attachment 273847 [details] Patch Attachment 273847 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/968016 New failing tests: transitions/extra-transition.html svg/carto.net/scrollbar.svg
Build Bot
Comment 30 2016-03-12 14:50:53 PST
Created attachment 273855 [details] Archive of layout-test-results from ews116 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews116 Port: mac-yosemite Platform: Mac OS X 10.10.5
Said Abou-Hallawa
Comment 31 2016-03-12 20:22:23 PST
Said Abou-Hallawa
Comment 32 2016-03-12 20:52:27 PST
Build Bot
Comment 33 2016-03-12 21:46:11 PST
Comment on attachment 273867 [details] Patch Attachment 273867 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/969595 New failing tests: fast/images/gif-large-checkerboard.html transitions/extra-transition.html svg/carto.net/scrollbar.svg
Build Bot
Comment 34 2016-03-12 21:46:16 PST
Created attachment 273869 [details] Archive of layout-test-results from ews101 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews101 Port: mac-yosemite Platform: Mac OS X 10.10.5
Build Bot
Comment 35 2016-03-12 22:11:18 PST
Comment on attachment 273867 [details] Patch Attachment 273867 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/969610 New failing tests: transitions/extra-transition.html
Build Bot
Comment 36 2016-03-12 22:11:22 PST
Created attachment 273870 [details] Archive of layout-test-results from ews115 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews115 Port: mac-yosemite Platform: Mac OS X 10.10.5
Said Abou-Hallawa
Comment 37 2016-03-12 22:12:39 PST
Build Bot
Comment 38 2016-03-12 23:06:34 PST
Comment on attachment 273871 [details] Patch Attachment 273871 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/969872 New failing tests: transitions/extra-transition.html
Build Bot
Comment 39 2016-03-12 23:06:41 PST
Created attachment 273872 [details] Archive of layout-test-results from ews100 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews100 Port: mac-yosemite Platform: Mac OS X 10.10.5
Said Abou-Hallawa
Comment 40 2016-03-12 23:09:29 PST
Build Bot
Comment 41 2016-03-13 00:02:06 PST
Comment on attachment 273873 [details] Patch Attachment 273873 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/970068 New failing tests: svg/carto.net/scrollbar.svg
Build Bot
Comment 42 2016-03-13 00:02:12 PST
Created attachment 273877 [details] Archive of layout-test-results from ews101 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews101 Port: mac-yosemite Platform: Mac OS X 10.10.5
Build Bot
Comment 43 2016-03-13 00:23:33 PST
Comment on attachment 273873 [details] Patch Attachment 273873 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/970101 New failing tests: transitions/extra-transition.html
Build Bot
Comment 44 2016-03-13 00:23:40 PST
Created attachment 273878 [details] Archive of layout-test-results from ews116 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews116 Port: mac-yosemite Platform: Mac OS X 10.10.5
Said Abou-Hallawa
Comment 45 2016-03-13 00:59:20 PST
Build Bot
Comment 46 2016-03-13 01:55:12 PST
Comment on attachment 273880 [details] Patch Attachment 273880 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/970432 New failing tests: transitions/extra-transition.html fast/images/gif-short-app-extension-string.html
Build Bot
Comment 47 2016-03-13 01:55:17 PST
Created attachment 273883 [details] Archive of layout-test-results from ews115 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews115 Port: mac-yosemite Platform: Mac OS X 10.10.5
Said Abou-Hallawa
Comment 48 2016-03-13 03:51:59 PDT
Said Abou-Hallawa
Comment 49 2017-07-24 11:03:46 PDT
The asynchronous image decoding feature was implemented and has been enabled in WebKit. What is left here is fixing regular bugs. So resolve this bug as Fixed.
Note You need to log in before you can comment on or make changes to this bug.