Bug 76051

Summary: [Qt] new test fast/forms/radio/radio-group.html introduced in r104668 fails
Product: WebKit Reporter: Gabor Rapcsanyi <rgabor>
Component: Tools / TestsAssignee: Lauro Moura Maranhao Neto <lauro.neto>
Status: RESOLVED FIXED    
Severity: Normal CC: lauro.neto, ossy, tkent, webkit.review.bot
Priority: P2 Keywords: Qt, QtTriaged
Version: 528+ (Nightly build)   
Hardware: All   
OS: Linux   
Bug Depends on:    
Bug Blocks: 45719, 87008    
Attachments:
Description Flags
Patch none

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.