NEW 189873
DisplayList should record and replay back the painting of the form controls
https://bugs.webkit.org/show_bug.cgi?id=189873
Summary DisplayList should record and replay back the painting of the form controls
Said Abou-Hallawa
Reported 2018-09-21 18:59:31 PDT
The form control drawing will recorded as a paintRect and a callback. If the GraphicsContext is recording, the paintRect and the callback will be stored. Otherwise the callback will be called by passing the GraphicsContext and the paintRect. The callback will be a lambda expression which captures whatever it needs to draw the form control. Since the renderer outlives recording and replaying back the display list, it can be passed by reference. I am going to enable display list drawing temporarily till the layout tests pass on all bots.
Attachments
Patch (336.87 KB, patch)
2018-09-21 20:01 PDT, Said Abou-Hallawa
no flags
Archive of layout-test-results from ews102 for mac-sierra (5.69 MB, application/zip)
2018-09-21 20:59 PDT, EWS Watchlist
no flags
Archive of layout-test-results from ews113 for mac-sierra (4.35 MB, application/zip)
2018-09-21 21:10 PDT, EWS Watchlist
no flags
Archive of layout-test-results from ews106 for mac-sierra-wk2 (9.58 MB, application/zip)
2018-09-21 21:39 PDT, EWS Watchlist
no flags
Archive of layout-test-results from ews126 for ios-simulator-wk2 (2.04 MB, application/zip)
2018-09-22 00:20 PDT, EWS Watchlist
no flags
Patch (340.11 KB, patch)
2018-09-22 18:43 PDT, Said Abou-Hallawa
no flags
Patch (340.22 KB, patch)
2018-09-22 19:24 PDT, Said Abou-Hallawa
no flags
Patch (342.59 KB, patch)
2018-09-22 20:24 PDT, Said Abou-Hallawa
no flags
Patch (342.65 KB, patch)
2018-09-22 20:35 PDT, Said Abou-Hallawa
no flags
Patch (342.67 KB, patch)
2018-09-22 20:41 PDT, Said Abou-Hallawa
no flags
Patch (342.56 KB, patch)
2018-09-22 20:56 PDT, Said Abou-Hallawa
no flags
Patch (342.56 KB, patch)
2018-09-22 21:04 PDT, Said Abou-Hallawa
no flags
Archive of layout-test-results from ews103 for mac-sierra (8.10 MB, application/zip)
2018-09-22 22:01 PDT, EWS Watchlist
no flags
Archive of layout-test-results from ews104 for mac-sierra-wk2 (9.16 MB, application/zip)
2018-09-22 22:29 PDT, EWS Watchlist
no flags
Archive of layout-test-results from ews116 for mac-sierra (8.92 MB, application/zip)
2018-09-22 23:09 PDT, EWS Watchlist
no flags
Archive of layout-test-results from ews122 for ios-simulator-wk2 (1.65 MB, application/zip)
2018-09-23 00:45 PDT, EWS Watchlist
no flags
Archive of layout-test-results from ews113 for mac-sierra (8.93 MB, application/zip)
2018-09-23 01:06 PDT, EWS Watchlist
no flags
Patch (343.72 KB, patch)
2018-09-23 23:33 PDT, Said Abou-Hallawa
no flags
Archive of layout-test-results from ews102 for mac-sierra (8.10 MB, application/zip)
2018-09-24 00:44 PDT, EWS Watchlist
no flags
Archive of layout-test-results from ews116 for mac-sierra (8.93 MB, application/zip)
2018-09-24 01:40 PDT, EWS Watchlist
no flags
Archive of layout-test-results from ews126 for ios-simulator-wk2 (1.70 MB, application/zip)
2018-09-24 03:10 PDT, EWS Watchlist
no flags
Archive of layout-test-results from ews117 for mac-sierra (8.95 MB, application/zip)
2018-09-24 03:37 PDT, EWS Watchlist
no flags
Archive of layout-test-results from ews104 for mac-sierra-wk2 (9.16 MB, application/zip)
2018-09-24 04:59 PDT, EWS Watchlist
no flags
Patch (343.82 KB, patch)
2018-09-24 07:59 PDT, Said Abou-Hallawa
no flags
Archive of layout-test-results from ews100 for mac-sierra (2.36 MB, application/zip)
2018-09-24 09:10 PDT, EWS Watchlist
no flags
Archive of layout-test-results from ews107 for mac-sierra-wk2 (2.92 MB, application/zip)
2018-09-24 09:24 PDT, EWS Watchlist
no flags
Archive of layout-test-results from ews115 for mac-sierra (3.15 MB, application/zip)
2018-09-24 09:55 PDT, EWS Watchlist
no flags
Patch (344.38 KB, patch)
2018-09-24 11:20 PDT, Said Abou-Hallawa
ews-watchlist: commit-queue-
Archive of layout-test-results from ews102 for mac-sierra (2.33 MB, application/zip)
2018-09-24 12:32 PDT, EWS Watchlist
no flags
Archive of layout-test-results from ews116 for mac-sierra (3.39 MB, application/zip)
2018-09-24 13:13 PDT, EWS Watchlist
no flags
Patch for review (76.73 KB, patch)
2018-09-24 15:54 PDT, Said Abou-Hallawa
no flags
Said Abou-Hallawa
Comment 1 2018-09-21 20:01:58 PDT
EWS Watchlist
Comment 2 2018-09-21 20:59:46 PDT
Comment on attachment 350481 [details] Patch Attachment 350481 [details] did not pass mac-ews (mac): Output: https://webkit-queues.webkit.org/results/9307324 Number of test failures exceeded the failure limit.
EWS Watchlist
Comment 3 2018-09-21 20:59:48 PDT
Created attachment 350487 [details] Archive of layout-test-results from ews102 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews102 Port: mac-sierra Platform: Mac OS X 10.12.6
EWS Watchlist
Comment 4 2018-09-21 21:10:18 PDT
Comment on attachment 350481 [details] Patch Attachment 350481 [details] did not pass mac-debug-ews (mac): Output: https://webkit-queues.webkit.org/results/9307246 Number of test failures exceeded the failure limit.
EWS Watchlist
Comment 5 2018-09-21 21:10:19 PDT
Created attachment 350489 [details] Archive of layout-test-results from ews113 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews113 Port: mac-sierra Platform: Mac OS X 10.12.6
EWS Watchlist
Comment 6 2018-09-21 21:39:39 PDT
Comment on attachment 350481 [details] Patch Attachment 350481 [details] did not pass mac-wk2-ews (mac-wk2): Output: https://webkit-queues.webkit.org/results/9307460 New failing tests: imported/mozilla/svg/text-layout-06.svg displaylists/replay-skip-clipped-rect.html fast/borders/hidpi-double-border-with-border-radius-always-produce-solid-line.html fast/css/cascade/box-shadow-and-webkit-box-shadow-cascade-order.html displaylists/extent-includes-shadow.html displaylists/extent-includes-transforms.html mathml/presentation/fractions-color.html svg/canvas/canvas-global-alpha-svg.html mathml/presentation/radical-bar-visibility.html displaylists/layer-dispay-list.html fast/text/simple-lines-float.html
EWS Watchlist
Comment 7 2018-09-21 21:39:40 PDT
Created attachment 350494 [details] Archive of layout-test-results from ews106 for mac-sierra-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews106 Port: mac-sierra-wk2 Platform: Mac OS X 10.12.6
EWS Watchlist
Comment 8 2018-09-22 00:20:17 PDT
Comment on attachment 350481 [details] Patch Attachment 350481 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: https://webkit-queues.webkit.org/results/9308548 Number of test failures exceeded the failure limit.
EWS Watchlist
Comment 9 2018-09-22 00:20:18 PDT
Created attachment 350500 [details] Archive of layout-test-results from ews126 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews126 Port: ios-simulator-wk2 Platform: Mac OS X 10.13.6
Said Abou-Hallawa
Comment 10 2018-09-22 18:43:44 PDT
Said Abou-Hallawa
Comment 11 2018-09-22 19:24:35 PDT
Said Abou-Hallawa
Comment 12 2018-09-22 20:24:19 PDT
Said Abou-Hallawa
Comment 13 2018-09-22 20:35:18 PDT
Said Abou-Hallawa
Comment 14 2018-09-22 20:41:41 PDT
Said Abou-Hallawa
Comment 15 2018-09-22 20:56:45 PDT
Said Abou-Hallawa
Comment 16 2018-09-22 21:04:06 PDT
EWS Watchlist
Comment 17 2018-09-22 22:01:35 PDT
Comment on attachment 350549 [details] Patch Attachment 350549 [details] did not pass mac-ews (mac): Output: https://webkit-queues.webkit.org/results/9317266 New failing tests: imported/mozilla/svg/text-layout-06.svg displaylists/replay-skip-clipped-rect.html displaylists/extent-includes-shadow.html fast/css/cascade/box-shadow-and-webkit-box-shadow-cascade-order.html fast/borders/hidpi-double-border-with-border-radius-always-produce-solid-line.html displaylists/extent-includes-transforms.html mathml/presentation/fractions-color.html svg/canvas/canvas-global-alpha-svg.html mathml/presentation/radical-bar-visibility.html displaylists/layer-dispay-list.html fast/text/simple-lines-float.html
EWS Watchlist
Comment 18 2018-09-22 22:01:37 PDT
Created attachment 350551 [details] Archive of layout-test-results from ews103 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews103 Port: mac-sierra Platform: Mac OS X 10.12.6
EWS Watchlist
Comment 19 2018-09-22 22:29:54 PDT
Comment on attachment 350549 [details] Patch Attachment 350549 [details] did not pass mac-wk2-ews (mac-wk2): Output: https://webkit-queues.webkit.org/results/9317360 New failing tests: imported/mozilla/svg/text-layout-06.svg displaylists/replay-skip-clipped-rect.html displaylists/extent-includes-shadow.html fast/css/cascade/box-shadow-and-webkit-box-shadow-cascade-order.html fast/borders/hidpi-double-border-with-border-radius-always-produce-solid-line.html displaylists/extent-includes-transforms.html mathml/presentation/fractions-color.html svg/canvas/canvas-global-alpha-svg.html mathml/presentation/radical-bar-visibility.html displaylists/layer-dispay-list.html fast/text/simple-lines-float.html
EWS Watchlist
Comment 20 2018-09-22 22:29:56 PDT
Created attachment 350552 [details] Archive of layout-test-results from ews104 for mac-sierra-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews104 Port: mac-sierra-wk2 Platform: Mac OS X 10.12.6
EWS Watchlist
Comment 21 2018-09-22 23:09:24 PDT
Comment on attachment 350549 [details] Patch Attachment 350549 [details] did not pass mac-debug-ews (mac): Output: https://webkit-queues.webkit.org/results/9317490 New failing tests: imported/mozilla/svg/text-layout-06.svg displaylists/replay-skip-clipped-rect.html fast/borders/hidpi-double-border-with-border-radius-always-produce-solid-line.html fast/css/cascade/box-shadow-and-webkit-box-shadow-cascade-order.html displaylists/extent-includes-shadow.html displaylists/extent-includes-transforms.html mathml/presentation/fractions-color.html http/tests/images/draw-pattern-slow-load-large-image.html svg/canvas/canvas-global-alpha-svg.html mathml/presentation/radical-bar-visibility.html displaylists/layer-dispay-list.html fast/text/simple-lines-float.html
EWS Watchlist
Comment 22 2018-09-22 23:09:26 PDT
Created attachment 350553 [details] Archive of layout-test-results from ews116 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews116 Port: mac-sierra Platform: Mac OS X 10.12.6
EWS Watchlist
Comment 23 2018-09-23 00:45:42 PDT
Comment on attachment 350549 [details] Patch Attachment 350549 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: https://webkit-queues.webkit.org/results/9318084 Number of test failures exceeded the failure limit.
EWS Watchlist
Comment 24 2018-09-23 00:45:44 PDT
Created attachment 350555 [details] Archive of layout-test-results from ews122 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews122 Port: ios-simulator-wk2 Platform: Mac OS X 10.13.6
EWS Watchlist
Comment 25 2018-09-23 01:06:49 PDT
Comment on attachment 350549 [details] Patch Attachment 350549 [details] did not pass mac-debug-ews (mac): Output: https://webkit-queues.webkit.org/results/9318135 New failing tests: imported/mozilla/svg/text-layout-06.svg displaylists/replay-skip-clipped-rect.html displaylists/extent-includes-shadow.html fast/css/cascade/box-shadow-and-webkit-box-shadow-cascade-order.html fast/borders/hidpi-double-border-with-border-radius-always-produce-solid-line.html displaylists/extent-includes-transforms.html mathml/presentation/fractions-color.html http/tests/images/draw-pattern-slow-load-large-image.html svg/canvas/canvas-global-alpha-svg.html mathml/presentation/radical-bar-visibility.html displaylists/layer-dispay-list.html fast/text/simple-lines-float.html
EWS Watchlist
Comment 26 2018-09-23 01:06:51 PDT
Created attachment 350556 [details] Archive of layout-test-results from ews113 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews113 Port: mac-sierra Platform: Mac OS X 10.12.6
Said Abou-Hallawa
Comment 27 2018-09-23 23:33:20 PDT
EWS Watchlist
Comment 28 2018-09-24 00:44:45 PDT
Comment on attachment 350612 [details] Patch Attachment 350612 [details] did not pass mac-ews (mac): Output: https://webkit-queues.webkit.org/results/9327476 New failing tests: imported/mozilla/svg/text-layout-06.svg displaylists/replay-skip-clipped-rect.html fast/css/cascade/box-shadow-and-webkit-box-shadow-cascade-order.html fast/borders/hidpi-double-border-with-border-radius-always-produce-solid-line.html displaylists/extent-includes-transforms.html mathml/presentation/fractions-color.html svg/canvas/canvas-global-alpha-svg.html mathml/presentation/radical-bar-visibility.html displaylists/layer-dispay-list.html fast/text/simple-lines-float.html
EWS Watchlist
Comment 29 2018-09-24 00:44:47 PDT
Created attachment 350619 [details] Archive of layout-test-results from ews102 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews102 Port: mac-sierra Platform: Mac OS X 10.12.6
EWS Watchlist
Comment 30 2018-09-24 01:40:25 PDT
Comment on attachment 350612 [details] Patch Attachment 350612 [details] did not pass mac-debug-ews (mac): Output: https://webkit-queues.webkit.org/results/9327579 New failing tests: imported/mozilla/svg/text-layout-06.svg displaylists/replay-skip-clipped-rect.html fast/css/cascade/box-shadow-and-webkit-box-shadow-cascade-order.html fast/borders/hidpi-double-border-with-border-radius-always-produce-solid-line.html displaylists/extent-includes-transforms.html mathml/presentation/fractions-color.html http/tests/images/draw-pattern-slow-load-large-image.html svg/canvas/canvas-global-alpha-svg.html mathml/presentation/radical-bar-visibility.html displaylists/layer-dispay-list.html fast/text/simple-lines-float.html
EWS Watchlist
Comment 31 2018-09-24 01:40:27 PDT
Created attachment 350623 [details] Archive of layout-test-results from ews116 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews116 Port: mac-sierra Platform: Mac OS X 10.12.6
EWS Watchlist
Comment 32 2018-09-24 03:10:47 PDT
Comment on attachment 350612 [details] Patch Attachment 350612 [details] did not pass ios-sim-ews (ios-simulator-wk2): Output: https://webkit-queues.webkit.org/results/9328257 Number of test failures exceeded the failure limit.
EWS Watchlist
Comment 33 2018-09-24 03:10:49 PDT
Created attachment 350629 [details] Archive of layout-test-results from ews126 for ios-simulator-wk2 The attached test failures were seen while running run-webkit-tests on the ios-sim-ews. Bot: ews126 Port: ios-simulator-wk2 Platform: Mac OS X 10.13.6
EWS Watchlist
Comment 34 2018-09-24 03:37:48 PDT
Comment on attachment 350612 [details] Patch Attachment 350612 [details] did not pass mac-debug-ews (mac): Output: https://webkit-queues.webkit.org/results/9328409 New failing tests: imported/mozilla/svg/text-layout-06.svg displaylists/replay-skip-clipped-rect.html fast/css/cascade/box-shadow-and-webkit-box-shadow-cascade-order.html fast/borders/hidpi-double-border-with-border-radius-always-produce-solid-line.html displaylists/extent-includes-transforms.html mathml/presentation/fractions-color.html http/tests/images/draw-pattern-slow-load-large-image.html svg/canvas/canvas-global-alpha-svg.html mathml/presentation/radical-bar-visibility.html displaylists/layer-dispay-list.html fast/text/simple-lines-float.html
EWS Watchlist
Comment 35 2018-09-24 03:37:50 PDT
Created attachment 350630 [details] Archive of layout-test-results from ews117 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews117 Port: mac-sierra Platform: Mac OS X 10.12.6
EWS Watchlist
Comment 36 2018-09-24 04:59:05 PDT
Comment on attachment 350612 [details] Patch Attachment 350612 [details] did not pass mac-wk2-ews (mac-wk2): Output: https://webkit-queues.webkit.org/results/9329005 New failing tests: imported/mozilla/svg/text-layout-06.svg fast/css/cascade/box-shadow-and-webkit-box-shadow-cascade-order.html fast/borders/hidpi-double-border-with-border-radius-always-produce-solid-line.html mathml/presentation/fractions-color.html svg/canvas/canvas-global-alpha-svg.html mathml/presentation/radical-bar-visibility.html fast/text/simple-lines-float.html
EWS Watchlist
Comment 37 2018-09-24 04:59:08 PDT
Created attachment 350632 [details] Archive of layout-test-results from ews104 for mac-sierra-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews104 Port: mac-sierra-wk2 Platform: Mac OS X 10.12.6
Said Abou-Hallawa
Comment 38 2018-09-24 07:44:08 PDT
The last patch confirms that all the layout tests pass when display list drawing is enabled except the following tests: imported/mozilla/svg/text-layout-06.svg fast/css/cascade/box-shadow-and-webkit-box-shadow-cascade-order.html fast/borders/hidpi-double-border-with-border-radius-always-produce-solid-line.html mathml/presentation/fractions-color.html svg/canvas/canvas-global-alpha-svg.html mathml/presentation/radical-bar-visibility.html fast/text/simple-lines-float.html These failures are mot related to form controls rendering. So I am going to disable display list drawing back and address these failures in a separate patch.
Said Abou-Hallawa
Comment 39 2018-09-24 07:59:02 PDT
EWS Watchlist
Comment 40 2018-09-24 09:10:16 PDT
Comment on attachment 350640 [details] Patch Attachment 350640 [details] did not pass mac-ews (mac): Output: https://webkit-queues.webkit.org/results/9331138 New failing tests: displaylists/replay-skip-clipped-rect.html displaylists/extent-includes-transforms.html displaylists/layer-dispay-list.html
EWS Watchlist
Comment 41 2018-09-24 09:10:18 PDT
Created attachment 350642 [details] Archive of layout-test-results from ews100 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews100 Port: mac-sierra Platform: Mac OS X 10.12.6
EWS Watchlist
Comment 42 2018-09-24 09:24:24 PDT
Comment on attachment 350640 [details] Patch Attachment 350640 [details] did not pass mac-wk2-ews (mac-wk2): Output: https://webkit-queues.webkit.org/results/9331171 New failing tests: displaylists/replay-skip-clipped-rect.html displaylists/layer-dispay-list.html
EWS Watchlist
Comment 43 2018-09-24 09:24:27 PDT
Created attachment 350645 [details] Archive of layout-test-results from ews107 for mac-sierra-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews107 Port: mac-sierra-wk2 Platform: Mac OS X 10.12.6
EWS Watchlist
Comment 44 2018-09-24 09:55:07 PDT
Comment on attachment 350640 [details] Patch Attachment 350640 [details] did not pass mac-debug-ews (mac): Output: https://webkit-queues.webkit.org/results/9331174 New failing tests: displaylists/replay-skip-clipped-rect.html displaylists/extent-includes-transforms.html displaylists/layer-dispay-list.html
EWS Watchlist
Comment 45 2018-09-24 09:55:10 PDT
Created attachment 350648 [details] Archive of layout-test-results from ews115 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews115 Port: mac-sierra Platform: Mac OS X 10.12.6
Said Abou-Hallawa
Comment 46 2018-09-24 11:20:38 PDT
EWS Watchlist
Comment 47 2018-09-24 12:31:59 PDT
Comment on attachment 350656 [details] Patch Attachment 350656 [details] did not pass mac-ews (mac): Output: https://webkit-queues.webkit.org/results/9333491 New failing tests: displaylists/extent-includes-transforms.html
EWS Watchlist
Comment 48 2018-09-24 12:32:01 PDT
Created attachment 350667 [details] Archive of layout-test-results from ews102 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews102 Port: mac-sierra Platform: Mac OS X 10.12.6
EWS Watchlist
Comment 49 2018-09-24 13:13:20 PDT
Comment on attachment 350656 [details] Patch Attachment 350656 [details] did not pass mac-debug-ews (mac): Output: https://webkit-queues.webkit.org/results/9333526 New failing tests: displaylists/extent-includes-transforms.html
EWS Watchlist
Comment 50 2018-09-24 13:13:22 PDT
Created attachment 350673 [details] Archive of layout-test-results from ews116 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews116 Port: mac-sierra Platform: Mac OS X 10.12.6
Said Abou-Hallawa
Comment 51 2018-09-24 15:54:22 PDT
Created attachment 350707 [details] Patch for review
Note You need to log in before you can comment on or make changes to this bug.