Bug 8024

Summary:

"Go" button (search) doesn't work at fifa.com (document.all.InputName)

Product:

WebKit

Reporter:

anand krishna <snklanandeee>

Component:

JavaScriptCore

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

andersca, snklanandeee

Priority:

Keywords:

HasReduction

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

URL:

http://www.fifa.com/en/index.html

Bug Depends on:

Bug Blocks:

7532

Attachments:

Description	Flags
Reduction	none
First attempt	none
Patch including testcase	none
Moved testcase	ggaren: review+

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	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
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.