Bug 65244 - [Qt] tst_QWebElement::style() fails because QWebElement::CascadedStyle doesn't work as expected
Summary: [Qt] tst_QWebElement::style() fails because QWebElement::CascadedStyle doesn'...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Jocelyn Turcotte
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks: 38654
  Show dependency tree / graph
 
Reported: 2011-07-27 03:41 PDT by Kristóf Kosztyó
Modified: 2012-11-12 07:30 PST (History)
4 users (show)

See Also:

Attachments
mark failing test cases as expected fail (2.59 KB, patch)
2011-08-02 01:31 PDT, Kristóf Kosztyó 		no flags Details | Formatted Diff | Diff
Patch (3.93 KB, patch)
2012-11-09 08:23 PST, Jocelyn Turcotte 		kenneth: review+
Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Kristóf Kosztyó 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)]
Comment 1 Kristóf Kosztyó 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)]
Comment 2 Kristóf Kosztyó 2011-08-02 01:31:31 PDT 
Created attachment 102626 [details]
mark failing test cases as expected fail
Comment 3 Csaba Osztrogonác 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 :)
Comment 4 Csaba Osztrogonác 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
Comment 5 Jocelyn Turcotte 2012-11-09 08:23:53 PST 
Created attachment 173312 [details]
Patch
Comment 6 Jocelyn Turcotte 2012-11-12 07:30:58 PST 
Committed r134232: <http://trac.webkit.org/changeset/134232>