Bug 65244

Summary: [Qt] tst_QWebElement::style() fails because QWebElement::CascadedStyle doesn't work as expected
Product: WebKit Reporter: Kristóf Kosztyó <kkristof>
Component: WebKit QtAssignee: Jocelyn Turcotte <jturcotte>
Status: RESOLVED FIXED    
Severity: Normal CC: jturcotte, kenneth, ossy, tonikitoo
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 38654    
Attachments:
Description Flags
mark failing test cases as expected fail
none
Patch kenneth: review+

Kristóf Kosztyó
Reported 2011-07-27 03:41:10 PDT
FAIL! : tst_QWebElement::style() Compared values are not the same Actual (p.styleProperty("color", QWebElement::CascadedStyle)): red Expected (QLatin1String("yellow")): yellow Loc: [/ramdisk/qt-linux-release/build/Source/WebKit/qt/tests/qwebelement/tst_qwebelement.cpp(478)]
Attachments
mark failing test cases as expected fail (2.59 KB, patch)
2011-08-02 01:31 PDT, Kristóf Kosztyó 		no flags
DetailsFormatted DiffDiff
Patch (3.93 KB, patch)
2012-11-09 08:23 PST, Jocelyn Turcotte 		kenneth: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Kristóf Kosztyó
Comment 1 2011-08-02 01:30:30 PDT
FAIL! : tst_QWebElement::style() Compared values are not the same Actual (p.styleProperty("color", QWebElement::CascadedStyle)): Expected (QLatin1String("red")): red Loc: [/home/kosztyo/WebKit/Source/WebKit/qt/tests/qwebelement/tst_qwebelement.cpp(544)] FAIL! : tst_QWebElement::style() Compared values are not the same Actual (p.styleProperty("color", QWebElement::CascadedStyle)): blue Expected (QLatin1String("black")): black Loc: [/home/kosztyo/WebKit/Source/WebKit/qt/tests/qwebelement/tst_qwebelement.cpp(565)] FAIL! : tst_QWebElement::style() Compared values are not the same Actual (p.styleProperty("color", QWebElement::CascadedStyle)): blue Expected (QLatin1String("black")): black Loc: [/home/kosztyo/WebKit/Source/WebKit/qt/tests/qwebelement/tst_qwebelement.cpp(583)]
Kristóf Kosztyó
Comment 2 2011-08-02 01:31:31 PDT
Created attachment 102626 [details] mark failing test cases as expected fail
Csaba Osztrogonác
Comment 3 2011-08-02 01:39:14 PDT
Comment on attachment 102626 [details] mark failing test cases as expected fail r=me, but cq-, because webkit-patch doesn't like my name :)
Csaba Osztrogonác
Comment 4 2011-08-02 01:47:14 PDT
Comment on attachment 102626 [details] mark failing test cases as expected fail Landed in https://trac.webkit.org/changeset/92169
Jocelyn Turcotte
Comment 5 2012-11-09 08:23:53 PST
Created attachment 173312 [details] Patch
Jocelyn Turcotte
Comment 6 2012-11-12 07:30:58 PST
Committed r134232: <http://trac.webkit.org/changeset/134232>
Note You need to log in before you can comment on or make changes to this bug.