Bug 76051 - [Qt] new test fast/forms/radio/radio-group.html introduced in r104668 fails
Summary: [Qt] new test fast/forms/radio/radio-group.html introduced in r104668 fails
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All Linux
: P2 Normal
Assignee: Lauro Moura Maranhao Neto
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks: 45719 87008
  Show dependency tree / graph
 
Reported: 2012-01-11 04:20 PST by Gabor Rapcsanyi
Modified: 2012-08-15 17:05 PDT (History)
4 users (show)

See Also:

Attachments
Patch (1.41 KB, patch)
2012-08-15 13:57 PDT, Lauro Moura Maranhao Neto 		no flags Details | Formatted Diff | Diff
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 Gabor Rapcsanyi 2012-01-11 04:20:11 PST 
--- /ramdisk/qt-linux-release/build/layout-test-results/fast/forms/radio/radio-group-expected.txt 
+++ /ramdisk/qt-linux-release/build/layout-test-results/fast/forms/radio/radio-group-actual.txt 
@@ -35,7 +35,7 @@
 PASS $("radio1-3").checked is true
 PASS $("radio1-4").checked is true
 (The following test depends on gc(). It might fail on a real browser.)
-PASS $("radio1-1").checked is false
+FAIL $("radio1-1").checked should be false. Was true.
 PASS $("radio1-3").checked is true
 
 Adding a radio button to an orphan tree:
Comment 1 Kent Tamura 2012-01-11 04:22:40 PST 
Does Qt DRT have GCController.collect()?
Comment 2 Gabor Rapcsanyi 2012-01-11 04:50:17 PST 
(In reply to comment #1)
> Does Qt DRT have GCController.collect()?

Yes it has, but anyway I skipped it while somebody fix it to make the bot green.
http://trac.webkit.org/changeset/104700
Comment 3 Lauro Moura Maranhao Neto 2012-08-15 13:57:26 PDT 
Created attachment 158634 [details]
Patch

Tested with newest Qt5, both WK1 and WK2 and it's working.
Comment 4 Csaba Osztrogonác 2012-08-15 14:06:51 PDT 
Comment on attachment 158634 [details]
Patch

rs=me.
Comment 5 WebKit Review Bot 2012-08-15 17:05:38 PDT 
Comment on attachment 158634 [details]
Patch

Clearing flags on attachment: 158634

Committed r125724: <http://trac.webkit.org/changeset/125724>
Comment 6 WebKit Review Bot 2012-08-15 17:05:43 PDT 
All reviewed patches have been landed.  Closing bug.