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

Gabor Rapcsanyi
Reported 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:
Attachments
Patch (1.41 KB, patch)
2012-08-15 13:57 PDT, Lauro Moura Maranhao Neto 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Kent Tamura
Comment 1 2012-01-11 04:22:40 PST
Does Qt DRT have GCController.collect()?
Gabor Rapcsanyi
Comment 2 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
Lauro Moura Maranhao Neto
Comment 3 2012-08-15 13:57:26 PDT
Created attachment 158634 [details] Patch Tested with newest Qt5, both WK1 and WK2 and it's working.
Csaba Osztrogonác
Comment 4 2012-08-15 14:06:51 PDT
Comment on attachment 158634 [details] Patch rs=me.
WebKit Review Bot
Comment 5 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>
WebKit Review Bot
Comment 6 2012-08-15 17:05:43 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.