Bug 8024 - "Go" button (search) doesn't work at fifa.com (document.all.InputName)
Summary: "Go" button (search) doesn't work at fifa.com (document.all.InputName)
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Nobody
URL: http://www.fifa.com/en/index.html
Keywords: HasReduction
: 8025 8027 (view as bug list)
Depends on:
Blocks: 7532
  Show dependency treegraph
 
Reported: 2006-03-28 00:15 PST by anand krishna
Modified: 2006-05-31 21:57 PDT (History)
2 users (show)

See Also:


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 Details | Formatted Diff | Diff
Patch including testcase (3.03 KB, patch)
2006-05-24 13:34 PDT, Rob Buis
no flags Details | Formatted Diff | Diff
Moved testcase (3.48 KB, patch)
2006-05-24 14:21 PDT, Rob Buis
ggaren: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description anand krishna 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)
Comment 1 David Kilzer (:ddkilzer) 2006-03-28 10:07:11 PST
*** Bug 8027 has been marked as a duplicate of this bug. ***
Comment 2 David Kilzer (:ddkilzer) 2006-03-28 10:07:23 PST
*** Bug 8025 has been marked as a duplicate of this bug. ***
Comment 3 Alexey Proskuryakov 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".
Comment 4 Geoffrey Garen 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.
Comment 5 Geoffrey Garen 2006-03-28 11:56:22 PST
This is one of the issues mentioned in my uber test for bug 7532.
Comment 6 Rob Buis 2006-05-24 11:27:28 PDT
Created attachment 8517 [details]
First attempt
Comment 7 Geoffrey Garen 2006-05-24 11:32:05 PDT
"named form" really meant "named input element"
Comment 8 Geoffrey Garen 2006-05-24 11:34:17 PDT
I've asked Rob to add a layout test to his patch.
Comment 9 Rob Buis 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.
Comment 10 Geoffrey Garen 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.
Comment 11 Rob Buis 2006-05-24 14:21:23 PDT
Created attachment 8521 [details]
Moved testcase

As discussed on irc, I moved the testcase.
Cheers,

Rob.
Comment 12 Geoffrey Garen 2006-05-24 14:57:35 PDT
Comment on attachment 8521 [details]
Moved testcase

r=me