RESOLVED FIXED 8024
"Go" button (search) doesn't work at fifa.com (document.all.InputName) 
https://bugs.webkit.org/show_bug.cgi?id=8024
Summary "Go" button (search) doesn't work at fifa.com (document.all.InputName)
anand krishna
Reported 2006-03-28 00:15:35 PST
"Go" button not working in safari browser alone while it works fine with other browsers.Iam using safari 1.2.2 (v 125.7)
Attachments
Reduction (389 bytes, text/html)
2006-03-28 11:52 PST, Geoffrey Garen 		no flags
Details
First attempt (2.44 KB, patch)
2006-05-24 11:27 PDT, Rob Buis 		no flags
DetailsFormatted DiffDiff
Patch including testcase (3.03 KB, patch)
2006-05-24 13:34 PDT, Rob Buis 		no flags
DetailsFormatted DiffDiff
Moved testcase (3.48 KB, patch)
2006-05-24 14:21 PDT, Rob Buis 		ggaren: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
David Kilzer (:ddkilzer)
Comment 1 2006-03-28 10:07:11 PST
*** Bug 8027 has been marked as a duplicate of this bug. ***
David Kilzer (:ddkilzer)
Comment 2 2006-03-28 10:07:23 PST
*** Bug 8025 has been marked as a duplicate of this bug. ***
Alexey Proskuryakov
Comment 3 2006-03-28 11:41:52 PST
Search doesn't work, confirmed with stock 10.4.5 and ToT. JavaScript console says "(event handler):Undefined value".
Geoffrey Garen
Comment 4 2006-03-28 11:52:53 PST
Created attachment 7362 [details] Reduction This is a case of indexing into the document.all collection by named form.
Geoffrey Garen
Comment 5 2006-03-28 11:56:22 PST
This is one of the issues mentioned in my uber test for bug 7532.
Rob Buis
Comment 6 2006-05-24 11:27:28 PDT
Created attachment 8517 [details] First attempt
Geoffrey Garen
Comment 7 2006-05-24 11:32:05 PDT
"named form" really meant "named input element"
Geoffrey Garen
Comment 8 2006-05-24 11:34:17 PDT
I've asked Rob to add a layout test to his patch.
Rob Buis
Comment 9 2006-05-24 13:34:35 PDT
Created attachment 8520 [details] Patch including testcase I hope this testcase is the right approach. I do not know whether it is in the right directory, just give me some pointers and I should be able to correct things hopefully quickly :) Cheers, Rob.
Geoffrey Garen
Comment 10 2006-05-24 13:43:30 PDT
Comment on attachment 8520 [details] Patch including testcase r=me The layout test is about form access through the DOM, so both fast/forms and fast/dom are good choices.
Rob Buis
Comment 11 2006-05-24 14:21:23 PDT
Created attachment 8521 [details] Moved testcase As discussed on irc, I moved the testcase. Cheers, Rob.
Geoffrey Garen
Comment 12 2006-05-24 14:57:35 PDT
Comment on attachment 8521 [details] Moved testcase r=me
Note You need to log in before you can comment on or make changes to this bug.