RESOLVED FIXED 55029
[Qt] tst_QWebView::setPalette(activeFG) fails 
https://bugs.webkit.org/show_bug.cgi?id=55029
Summary [Qt] tst_QWebView::setPalette(activeFG) fails
Csaba Osztrogonác
Reported 2011-02-23 03:17:14 PST
FAIL! : tst_QWebView::setPalette(activeBG) 'img1 != img2' returned FALSE. () Loc: [/home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/Source/WebKit/qt/tests/qwebview/tst_qwebview.cpp(426)] FAIL! : tst_QWebView::setPalette(activeFG) 'img1 != img2' returned FALSE. () Loc: [/home/webkitbuildbot/slaves/release32bit/buildslave/qt-linux-release/build/Source/WebKit/qt/tests/qwebview/tst_qwebview.cpp(426)]
Attachments
Patch to improve the auto-test. (2.20 KB, patch)
2011-02-23 09:37 PST, Alexis Menard (darktears) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alexis Menard (darktears)
Comment 1 2011-02-23 03:23:47 PST
It's not a regression, the tests were added with the patch :).
Csaba Osztrogonác
Comment 2 2011-02-23 03:25:22 PST
(In reply to comment #1) > It's not a regression, the tests were added with the patch :). OK, it's not a regression. :) But tests introduced in r79409 still fail.
Alexis Menard (darktears)
Comment 3 2011-02-23 03:29:37 PST
But I'm on it :)
Csaba Osztrogonác
Comment 4 2011-02-23 03:30:57 PST
(In reply to comment #3) > But I'm on it :) Cool, thanks. ;)
Alexis Menard (darktears)
Comment 5 2011-02-23 09:37:40 PST
Created attachment 83491 [details] Patch to improve the auto-test.
Ariya Hidayat
Comment 6 2011-02-23 10:45:00 PST
Comment on attachment 83491 [details] Patch to improve the auto-test. LGTM. re=me.
WebKit Commit Bot
Comment 7 2011-02-23 11:03:50 PST
Comment on attachment 83491 [details] Patch to improve the auto-test. Clearing flags on attachment: 83491 Committed r79463: <http://trac.webkit.org/changeset/79463>
WebKit Commit Bot
Comment 8 2011-02-23 11:03:55 PST
All reviewed patches have been landed. Closing bug.
Andreas Kling
Comment 9 2011-02-24 05:44:43 PST
Committed r79552: <http://trac.webkit.org/changeset/79552>
Note You need to log in before you can comment on or make changes to this bug.