Bug 11779

Summary:

REGRESSION: cannot change disabled SELECT value from JS (probable AJAX bug prevents booking at bmi flight site)

Product:

WebKit

Reporter:

tim bates <timothy.c.bates>

Component:

Forms

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Major

CC:

adele, ap

Priority:

Keywords:

InRadar, Regression

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

URL:

http://www.flybmi.com/bmi/en-gb/index.aspx

Attachments:

Description	Flags
test case	none

tim bates

Reported 2006-12-07 18:10:50 PST

1 go to the url 2. in the journey planner, select some trip (say LHR to EDI on 21 December 2006) 3. submit (it won't buy anything, don't worry :-) ) EXPECTED: costing for chosen journey OBTAINED: error dialog to select begin and arrival airports Works just fine on Firefox, On safari, the departure and arrival are selected, just not getting through to the pre-search checking algorithm.

Attachments
test case (427 bytes, text/html) 2007-01-07 10:56 PST, Alexey Proskuryakov	no flags	Details
View All Add attachment proposed patch, testcase, etc.

tim bates

Comment 1 2006-12-08 03:30:34 PST

there are other web-kit non-firefox behaviors here too: 1. leave the departure where it is (default LHR) 2. Select a destination (EDI) --> the departure airport resets (It should stay set as chosen)

Alexey Proskuryakov

Comment 2 2007-01-07 10:55:47 PST

This has regressed with r16663 (initial implementation of engine-based list box control): * html/HTMLOptionElement.cpp: (WebCore::HTMLOptionElement::setSelected): Doesn't allow selection to be changed here if the option is disabled. (WebCore::HTMLOptionElement::disabled): Added. Checks the parent's disabled status. * html/HTMLOptionElement.h: Added disabled method.

Alexey Proskuryakov

Comment 3 2007-01-07 10:56:34 PST

Created attachment 12280 [details] test case

Mark Rowe (bdash)

Comment 4 2007-01-16 19:14:58 PST

<rdar://problem/4928638>

Alexey Proskuryakov

Comment 5 2007-02-03 02:03:14 PST

Fixed by Maciej in r19383.

Note You need to log in before you can comment on or make changes to this bug.