[Qt] animations/dynamic-stylesheet-loading.html fails with accelerated compositing

• Component: Tools / Tests
• Status: CLOSED FIXED

[Csaba Osztrogonác]

animations/dynamic-stylesheet-loading.html introduced in http://trac.webkit.org/changeset/57390 ,
but fails if accelerated compositing is enabled (enabled by default)

results: http://build.webkit.org/results/Qt%20Linux%20Release/r57407%20%289904%29/results.html

If I disabled accelerated compositing, the test would be pass: (r57365)
--- a/WebKit/qt/Api/qwebsettings.cpp
+++ b/WebKit/qt/Api/qwebsettings.cpp
@@ -451,7 +451,7 @@ QWebSettings::QWebSettings()
     d->attributes.insert(QWebSettings::LocalStorageEnabled, false);
     d->attributes.insert(QWebSettings::LocalContentCanAccessRemoteUrls, false);
     d->attributes.insert(QWebSettings::LocalContentCanAccessFileUrls, true);
-    d->attributes.insert(QWebSettings::AcceleratedCompositingEnabled, true);
+    d->attributes.insert(QWebSettings::AcceleratedCompositingEnabled, false);

[Csaba Osztrogonác]

It might be related with: https://bugs.webkit.org/show_bug.cgi?id=36191

Skipped until fix: http://trac.webkit.org/changeset/57409

[Jocelyn Turcotte]

We should try to un-skip this test as soon as possible: P1.

[Noam Rosenthal]

I can't reproduce it - I believe it was fixed as a side effect of http://trac.webkit.org/changeset/57961. Can someone double check?

[Csaba Osztrogonác]

(In reply to comment #3)
> I can't reproduce it - I believe it was fixed as a side effect of http://trac.webkit.org/changeset/57961. Can someone double check?

I double checked, it works now, so I unskipped it:
http://trac.webkit.org/changeset/59471

[Simon Hausmann]

<cherry-pick-for-backport: r59471>

[Simon Hausmann]

Revision r59471 cherry-picked into qtwebkit-2.0 with commit 4696beb87359fe9236d23e0791526eb38dab341d