Bug 167769 - [EFL][CoordinatedGraphics] Broken painting of background-attachment:fixed after scrolling
Summary: [EFL][CoordinatedGraphics] Broken painting of background-attachment:fixed aft...
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Fujii Hironori
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-02-02 18:07 PST by Fujii Hironori
Modified: 2017-03-11 10:52 PST (History)
6 users (show)

See Also:


Attachments
test content of background-attachment:fixed (5.23 KB, text/html)
2017-02-02 18:07 PST, Fujii Hironori
no flags Details
screenshot of EFL port (50.49 KB, image/png)
2017-02-02 18:11 PST, Fujii Hironori
no flags Details
Patch (4.05 KB, patch)
2017-02-05 23:20 PST, Fujii Hironori
gyuyoung.kim: review-
buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ews102 for mac-elcapitan (2.68 MB, application/zip)
2017-02-06 00:23 PST, Build Bot
no flags Details
Archive of layout-test-results from ews105 for mac-elcapitan-wk2 (2.62 MB, application/zip)
2017-02-06 00:27 PST, Build Bot
no flags Details
Archive of layout-test-results from ews112 for mac-elcapitan (3.44 MB, application/zip)
2017-02-06 00:40 PST, Build Bot
no flags Details
Archive of layout-test-results from ews122 for ios-simulator-wk2 (16.39 MB, application/zip)
2017-02-06 01:03 PST, Build Bot
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Fujii Hironori 2017-02-02 18:07:02 PST
Created attachment 300479 [details]
test content of background-attachment:fixed

[EFL][CoordinatedGraphics] Broken painting of background-attachment:fixed after scrolling

This bug happens only in multiprocess CoordinatedGraphics which is used only in EFL port.
Comment 1 Fujii Hironori 2017-02-02 18:11:01 PST
Created attachment 300481 [details]
screenshot of EFL port
Comment 2 Fujii Hironori 2017-02-05 23:20:51 PST
Created attachment 300688 [details]
Patch

Try EWS.
Comment 3 Fujii Hironori 2017-02-05 23:23:10 PST
(In reply to comment #0)
> This bug happens only in multiprocess CoordinatedGraphics which is used only
> in EFL port.

This is wrong.
This bug happens only in CoordinatedGraphics fixed layout mode.
AFAIK, other CoordinatedGraphics ports (ie. GTK port) don't use the fixed layout mode.
Comment 4 Build Bot 2017-02-06 00:23:46 PST
Comment on attachment 300688 [details]
Patch

Attachment 300688 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/3012269

New failing tests:
fast/backgrounds/background-attachment-fixed.html
Comment 5 Build Bot 2017-02-06 00:23:50 PST
Created attachment 300694 [details]
Archive of layout-test-results from ews102 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 6 Build Bot 2017-02-06 00:27:16 PST
Comment on attachment 300688 [details]
Patch

Attachment 300688 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/3012272

New failing tests:
fast/backgrounds/background-attachment-fixed.html
Comment 7 Build Bot 2017-02-06 00:27:20 PST
Created attachment 300695 [details]
Archive of layout-test-results from ews105 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6
Comment 8 Build Bot 2017-02-06 00:40:47 PST
Comment on attachment 300688 [details]
Patch

Attachment 300688 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/3012276

New failing tests:
fast/backgrounds/background-attachment-fixed.html
Comment 9 Build Bot 2017-02-06 00:40:51 PST
Created attachment 300696 [details]
Archive of layout-test-results from ews112 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews112  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 10 Build Bot 2017-02-06 01:03:49 PST
Comment on attachment 300688 [details]
Patch

Attachment 300688 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/3012307

New failing tests:
fast/backgrounds/background-attachment-fixed.html
Comment 11 Build Bot 2017-02-06 01:03:53 PST
Created attachment 300697 [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.11.6
Comment 12 Gyuyoung Kim 2017-02-06 21:05:15 PST
Comment on attachment 300688 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=300688&action=review

> LayoutTests/ChangeLog:8
> +        * fast/backgrounds/background-attachment-fixed-expected.html: Added.

This tests only should be run on ports which use CoordinatedGraphics. Probably EFL and GTK ports. You need to update TestExpectations files for non-coordinated-grahpics ports.
Comment 13 Fujii Hironori 2017-02-06 21:55:12 PST
I found that this test does not work as expected even on EFL port WebKitTestRunner.
Only EFL MiniBrowser can reproduce this bug with the test.
I guess fixed layout mode of EFL port WTR doesn't work as expected.

In this change <https://trac.webkit.org/changeset/187571>,
you mentioned "EFL WTR doesn't work with fixed layout and delegatesScrolling yet".
Are this sentence still true?
Comment 14 Gyuyoung Kim 2017-02-06 22:06:48 PST
(In reply to comment #13)
> I found that this test does not work as expected even on EFL port
> WebKitTestRunner.
> Only EFL MiniBrowser can reproduce this bug with the test.
> I guess fixed layout mode of EFL port WTR doesn't work as expected.
> 
> In this change <https://trac.webkit.org/changeset/187571>,
> you mentioned "EFL WTR doesn't work with fixed layout and delegatesScrolling
> yet".
> Are this sentence still true?

Unfortunately yes. Nobody adds an option to run WTR with fixed layout yet. I think we might enable the fixed layout on WTR using testRunner.Foo function. For instance, testRunner.enableFixedLayout(true). But I don't start to support it yet. :(
Comment 15 Fujii Hironori 2017-02-10 06:30:47 PST
Comment on attachment 300688 [details]
Patch

This patch causes flickering. GTK port doesn't flicker. This is not a right fix.
Comment 16 Michael Catanzaro 2017-03-11 10:52:16 PST
Closing this bug because the EFL port has been removed from trunk.

If you feel this bug applies to a different upstream WebKit port and was closed in error, please either update the title and reopen the bug, or leave a comment to request this.