Bug 191688 - [GTK] Several compositing tests failing since r238090 "Make compositing updates incremental"
Summary: [GTK] Several compositing tests failing since r238090 "Make compositing updat...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-11-15 07:24 PST by Miguel Gomez
Modified: 2018-11-15 11:24 PST (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Miguel Gomez 2018-11-15 07:24:10 PST
Several compositing related tests started failing since r238090:

compositing/backing/no-backing-for-clip-overlap.html [ Failure ]
compositing/backing/no-backing-for-clip.html [ Failure ]
compositing/backing/no-backing-for-perspective.html [ Failure ]
compositing/layer-creation/overlap-animation.html [ Failure ]
compositing/layer-creation/overlap-transforms.html [ Failure ]

Curiously, all of this tests produce almost the expected outcome except that some of the layers have the drawsContent flag set to 1 when it's not expected. These are the diffs:

compositing/backing/no-backing-for-clip-overlap.html:
https://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20(Tests)/r238220%20(8857)/compositing/backing/no-backing-for-clip-overlap-diff.txt

compositing/backing/no-backing-for-clip.html:
https://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20(Tests)/r238220%20(8857)/compositing/backing/no-backing-for-clip-diff.txt

compositing/backing/no-backing-for-perspective.html:
https://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20(Tests)/r238220%20(8857)/compositing/backing/no-backing-for-perspective-diff.txt

compositing/layer-creation/overlap-animation.html:
https://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20(Tests)/r238220%20(8857)/compositing/layer-creation/overlap-animation-diff.txt

compositing/layer-creation/overlap-transforms.html:
https://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20(Tests)/r238220%20(8857)/compositing/layer-creation/overlap-transforms-diff.txt
Comment 1 Miguel Gomez 2018-11-15 07:26:35 PST
compositing/overflow/clip-descendents.html [ Failure ]

This seems to be related to r230890 as well, but the diff is different from the other ones:

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/compositing/repaint/absolute-painted-into-composited-ancestor-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/compositing/repaint/absolute-painted-into-composited-ancestor-actual.txt
@@ -1,5 +1,4 @@
 (repaint rects
-  (rect 0 0 200 200)
   (rect 300 0 200 200)
 )
 (GraphicsLayer
Comment 2 Miguel Gomez 2018-11-15 07:28:57 PST
And (In reply to Miguel Gomez from comment #1)
> compositing/overflow/clip-descendents.html [ Failure ]
> 
> This seems to be related to r230890 as well, but the diff is different from
> the other ones:

Sorry, this is failing due to the drawsContent flag as well, this is the diff

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/compositing/overflow/clip-descendents-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/compositing/overflow/clip-descendents-actual.txt
@@ -36,6 +36,7 @@
           (offsetFromRenderer width=20 height=10)
           (position 48.00 230.00)
           (bounds 60.00 70.00)
+          (drawsContent 1)
           (children 1
             (GraphicsLayer
               (offsetFromRenderer width=20 height=10)
@@ -55,6 +56,7 @@
           (offsetFromRenderer width=20 height=10)
           (position 240.00 230.00)
           (bounds 60.00 70.00)
+          (drawsContent 1)
           (children 1
             (GraphicsLayer
               (offsetFromRenderer width=20 height=10)
Comment 3 Miguel Gomez 2018-11-15 07:30:20 PST
compositing/repaint/absolute-painted-into-composited-ancestor.html [ Failure ]

This is the one that happens also due to r238090 but is not related to the drawsContent flag. This is the diff

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/compositing/repaint/absolute-painted-into-composited-ancestor-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/compositing/repaint/absolute-painted-into-composited-ancestor-actual.txt
@@ -1,5 +1,4 @@
 (repaint rects
-  (rect 0 0 200 200)
   (rect 300 0 200 200)
 )
 (GraphicsLayer
Comment 4 Simon Fraser (smfr) 2018-11-15 09:44:20 PST
(In reply to Miguel Gomez from comment #1)
> compositing/overflow/clip-descendents.html [ Failure ]
> 
> This seems to be related to r230890 as well, but the diff is different from
> the other ones:
> 
> ---
> /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/
> compositing/repaint/absolute-painted-into-composited-ancestor-expected.txt
> +++
> /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/
> compositing/repaint/absolute-painted-into-composited-ancestor-actual.txt
> @@ -1,5 +1,4 @@
>  (repaint rects
> -  (rect 0 0 200 200)
>    (rect 300 0 200 200)
>  )
>  (GraphicsLayer

I changed repaint stuff in r237942, but that should have added repaints, not removed them.
Comment 5 Michael Catanzaro 2018-11-15 11:24:22 PST
(In reply to Miguel Gomez from comment #0)
> Curiously, all of this tests produce almost the expected outcome except that
> some of the layers have the drawsContent flag set to 1 when it's not
> expected.

Probably unrelated, but reminds me of bug #187268, where we keep backing store attached when it shouldn't be....