6908 2006-01-29 02:51:41 -0800 REGRESSION: Replacing existing <option> shuffles it to the end of the list 2006-02-07 07:32:19 -0800 1 1 1 Unclassified WebKit Forms 420+ Mac OS X 10.4 VERIFIED FIXED http://www.qantas.com.au EasyFix, HasReduction, Regression P1 Major --- 6870 1 andrew darin darin oldest_to_newest 30488 0 andrew 2006-01-29 02:51:41 -0800 Go to www.qantas.com.au. In the tab labelled "Domestic" choose a Month to depart from (choose March, for example). Once you release the mouse button from choosing that month, notice that the month displayed is not what you selected. Attempt to select another month from the same popup menu and notice that the order of items in the list is now very strange. This does not occur with Apple's latest release of Safari. 30492 1 joost 2006-01-29 03:34:39 -0800 Confirmed, this site is good for bugfinding. :) 30497 2 ddkilzer 2006-01-29 04:32:55 -0800 Adding Regression keyword per reporter. Regressions are P1. Bug 6870 had to be fixed before this could be tested, therefore adding it as a dependency. 30499 3 6070 ap 2006-01-29 05:46:22 -0800 Created attachment 6070 test case Safari 2.0.3 and Opera leave the first element selected; WinIE and Firefox select the second one. ToT replaces a wrong (third) element). 30500 4 ddkilzer 2006-01-29 05:51:48 -0800 Actually ToT removes the element in the list then adds the new element to the end of the list. It should simply replace the existing element in the list with the new element. 31093 5 darin 2006-02-02 09:51:00 -0800 The problem is caused by the fact that KJS::HTMLSelectCollection::put relies on the "options" collection, which is not implemented. In fact, the binding for "options" calls optionsHTMLCollection() instead of options(). This particular bug can almost certainly be fixed just by changing the function call inside KJS::HTMLSelectCollection::put from options() to optionsHTMLCollection(). I tried it and it seems to work. 31411 6 6262 darin 2006-02-05 00:44:41 -0800 Created attachment 6262 patch to fix the problem, including change log and test 31725 7 ddkilzer 2006-02-07 07:32:19 -0800 Verified fixed. 6070 2006-01-29 05:46:22 -0800 2006-01-29 05:46:22 -0800 test case select.html text/html 319 ap PGhlYWQ+CjxzY3JpcHQ+CmZ1bmN0aW9uIGYoKQp7Cglkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgn dGVzdDInKS5vcHRpb25zWzBdPW5ldyBPcHRpb24oIjEiLCIxIik7Cn0KPC9zY3JpcHQ+CjwvaGVh ZD4KPGJvZHkgb25Mb2FkPSJqYXZhc2NpcHQ6ZigpOyI+ClNob3VsZCBiZTogIjEiLCAiT3B0aW9u IDIiLCAiT3B0aW9uIDMiOgo8Zm9ybT4KPHNlbGVjdCBpZD0idGVzdDIiPgo8b3B0aW9uPk9wdGlv biAxPC9vcHRpb24+CjxvcHRpb24+T3B0aW9uIDI8L29wdGlvbj4KPG9wdGlvbj5PcHRpb24gMzwv b3B0aW9uPgo8L3NlbGVjdD4KPC9mb3JtPgoKPC9ib2R5Pg== 6262 2006-02-05 00:44:41 -0800 2006-02-05 01:04:14 -0800 patch to fix the problem, including change log and test SelectOptionPatch.txt text/plain 4860 darin SW5kZXg6IFdlYkNvcmUva2h0bWwvZWNtYS9ranNfaHRtbC5jcHAKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2Vi Q29yZS9raHRtbC9lY21hL2tqc19odG1sLmNwcAkocmV2aXNpb24gMTI1NjgpCisrKyBXZWJDb3Jl L2todG1sL2VjbWEva2pzX2h0bWwuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0yLDcgKzIsNyBAQAog LyoKICAqICBUaGlzIGZpbGUgaXMgcGFydCBvZiB0aGUgS0RFIGxpYnJhcmllcwogICogIENvcHly aWdodCAoQykgMTk5OS0yMDAwIEhhcnJpIFBvcnRlbiAocG9ydGVuQGtkZS5vcmcpCi0gKiAgQ29w eXJpZ2h0IChDKSAyMDA0IEFwcGxlIENvbXB1dGVyLCBJbmMuCisgKiAgQ29weXJpZ2h0IChDKSAy MDA0LCAyMDA1LCAyMDA2IEFwcGxlIENvbXB1dGVyLCBJbmMuCiAgKgogICogIFRoaXMgbGlicmFy eSBpcyBmcmVlIHNvZnR3YXJlOyB5b3UgY2FuIHJlZGlzdHJpYnV0ZSBpdCBhbmQvb3IKICAqICBt b2RpZnkgaXQgdW5kZXIgdGhlIHRlcm1zIG9mIHRoZSBHTlUgTGVzc2VyIEdlbmVyYWwgUHVibGlj CkBAIC0zNDQzLDcgKzM0NDMsNyBAQAogICAgIH0KICAgICAvLyByZXBsYWNlIGFuIGV4aXN0aW5n IGVudHJ5ID8KICAgfSBlbHNlIGlmIChkaWZmIDwgMCkgewotICAgIGJlZm9yZSA9IHN0YXRpY19j YXN0PEhUTUxFbGVtZW50SW1wbCAqPihtX2VsZW1lbnQtPm9wdGlvbnMoKS0+aXRlbSh1KzEpKTsK KyAgICBiZWZvcmUgPSBzdGF0aWNfY2FzdDxIVE1MRWxlbWVudEltcGwgKj4obV9lbGVtZW50LT5v cHRpb25zSFRNTENvbGxlY3Rpb24oKS0+aXRlbSh1KzEpKTsKICAgICBtX2VsZW1lbnQtPnJlbW92 ZSh1KTsKICAgfQogICAvLyBmaW5hbGx5IGFkZCB0aGUgbmV3IGVsZW1lbnQKSW5kZXg6IFdlYkNv cmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9nCShyZXZpc2lvbiAx MjU2OSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTUg QEAKKzIwMDYtMDItMDUgIERhcmluIEFkbGVyICA8ZGFyaW5AYXBwbGUuY29tPgorCisgICAgICAg IFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIC0gZml4IGh0dHA6Ly9idWd6 aWxsYS5vcGVuZGFyd2luLm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NjkwOAorICAgICAgICAgIFJFR1JF U1NJT046IFJlcGxhY2luZyBleGlzdGluZyA8b3B0aW9uPiBzaHVmZmxlcyBpdCB0byB0aGUgZW5k IG9mIHRoZSBsaXN0CisKKyAgICAgICAgVGVzdDogZmFzdC9mb3Jtcy9zZWxlY3QtcmVwbGFjZS1v cHRpb24uaHRtbAorCisgICAgICAgICoga2h0bWwvZWNtYS9ranNfaHRtbC5jcHA6IChLSlM6OktK Uzo6SFRNTFNlbGVjdENvbGxlY3Rpb246OnB1dCk6CisgICAgICAgIFVzZSBvcHRpb25zSFRNTENv bGxlY3Rpb24sIGJlY2F1c2Ugb3B0aW9ucyBpc24ndCByZWFsbHkgaW1wbGVtZW50ZWQgeWV0Lgor CiAyMDA2LTAyLTA1ICBEYXZpZCBLaWx6ZXIgIDxkZGtpbHplckBraWx6ZXIubmV0PgogCiAgICAg ICAgIFJldmlld2VkIGJ5IERhcmluLCBsYW5kZWQgYnkgYXAgKHdpdGggYSB0d2VhayBzdWdnZXN0 ZWQgYnkgRGFyaW4pLgpJbmRleDogTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCj09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0t IExheW91dFRlc3RzL0NoYW5nZUxvZwkocmV2aXNpb24gMTI1NjgpCisrKyBMYXlvdXRUZXN0cy9D aGFuZ2VMb2cJKHdvcmtpbmcgY29weSkKQEAgLTEsMyArMSwxMyBAQAorMjAwNi0wMi0wNSAgRGFy aW4gQWRsZXIgIDxkYXJpbkBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ IChPT1BTISkuCisKKyAgICAgICAgLSB0ZXN0IGZvciBodHRwOi8vYnVnemlsbGEub3BlbmRhcndp bi5vcmcvc2hvd19idWcuY2dpP2lkPTY5MDgKKyAgICAgICAgICBSRUdSRVNTSU9OOiBSZXBsYWNp bmcgZXhpc3RpbmcgPG9wdGlvbj4gc2h1ZmZsZXMgaXQgdG8gdGhlIGVuZCBvZiB0aGUgbGlzdAor CisgICAgICAgICogZmFzdC9mb3Jtcy9zZWxlY3QtcmVwbGFjZS1vcHRpb24uaHRtbDogQWRkZWQu CisgICAgICAgICogZmFzdC9mb3Jtcy9zZWxlY3QtcmVwbGFjZS1vcHRpb24tZXhwZWN0ZWQudHh0 OiBBZGRlZC4KKwogMjAwNi0wMi0wNCAgTWFjaWVqIFN0YWNob3dpYWsgIDxtanNAYXBwbGUuY29t PgogCiAgICAgICAgIC0gcmV2ZXJ0ZWQgQ1NTIG1lZGlhIHF1ZXJpZXMgcGF0Y2gsIGFzIGl0IGNh dXNlcyByYW5kb20gY3Jhc2hlcyBvbiB0aGUKSW5kZXg6IExheW91dFRlc3RzL2Zhc3QvZm9ybXMv c2VsZWN0LXJlcGxhY2Utb3B0aW9uLWV4cGVjdGVkLnR4dAo9PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRU ZXN0cy9mYXN0L2Zvcm1zL3NlbGVjdC1yZXBsYWNlLW9wdGlvbi1leHBlY3RlZC50eHQJKHJldmlz aW9uIDApCisrKyBMYXlvdXRUZXN0cy9mYXN0L2Zvcm1zL3NlbGVjdC1yZXBsYWNlLW9wdGlvbi1l eHBlY3RlZC50eHQJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsMTAgQEAKK1RoaXMgdGVzdHMgcmVw bGFjaW5nIGFuIG9wdGlvbiB1c2luZyBKYXZhU2NyaXB0LiBJZiBzdWNjZXNzZnVsLCB0aGUgb3B0 aW9ucyBzaG91bGQgYmUgIlJlcGxhY2VtZW50IE9wdGlvbiAwIiwgIk9wdGlvbiAxIiwgYW5kICJP cHRpb24gMiIsIGFuZCBzZWxlY3RlZCBpbmRleCBzaG91bGQgYmUgMC4KKworCitvcHRpb25zWzBd IGlzIE9wdGlvbiAwCitvcHRpb25zWzFdIGlzIFJlcGxhY2VtZW50IE9wdGlvbiAxCitvcHRpb25z WzJdIGlzIE9wdGlvbiAyCitzZWxlY3RlZCBpbmRleCBpcyAwCisKKworCgpQcm9wZXJ0eSBjaGFu Z2VzIG9uOiBMYXlvdXRUZXN0cy9mYXN0L2Zvcm1zL3NlbGVjdC1yZXBsYWNlLW9wdGlvbi1leHBl Y3RlZC50eHQKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fXwpOYW1lOiBzdm46ZW9sLXN0eWxlCiAgICsgbmF0aXZlCgpJbmRl eDogTGF5b3V0VGVzdHMvZmFzdC9mb3Jtcy9zZWxlY3QtcmVwbGFjZS1vcHRpb24uaHRtbAo9PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9mYXN0L2Zvcm1zL3NlbGVjdC1yZXBsYWNlLW9wdGlvbi5o dG1sCShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMvZmFzdC9mb3Jtcy9zZWxlY3QtcmVwbGFj ZS1vcHRpb24uaHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwzMyBAQAorPGhlYWQ+Cis8c2Ny aXB0PgorZnVuY3Rpb24gdGVzdCgpCit7CisgICAgaWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29udHJv bGxlcikKKyAgICAgICAgbGF5b3V0VGVzdENvbnRyb2xsZXIuZHVtcEFzVGV4dCgpOworICAgIHZh ciBzZWxlY3QgPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgnc2VsZWN0Jyk7CisgICAgc2VsZWN0 Lm9wdGlvbnNbMV0gPSBuZXcgT3B0aW9uKCJSZXBsYWNlbWVudCBPcHRpb24gMSIsICJSZXBsYWNl bWVudCBPcHRpb24gMSIpOworICAgIHZhciBsZW5ndGggPSBzZWxlY3Qub3B0aW9ucy5sZW5ndGg7 CisgICAgZm9yICh2YXIgaSA9IDA7IGkgPCBsZW5ndGg7ICsraSkgeworICAgICAgICB2YXIgbWVz c2FnZSA9ICJvcHRpb25zWyIgKyBpICsgIl0gaXMgIiArIHNlbGVjdC5vcHRpb25zW2ldLmlubmVy SFRNTDsKKyAgICAgICAgdmFyIHBhcmFncmFwaCA9IGRvY3VtZW50LmNyZWF0ZUVsZW1lbnQoJ2Rp dicpOworICAgICAgICBwYXJhZ3JhcGguYXBwZW5kQ2hpbGQoZG9jdW1lbnQuY3JlYXRlVGV4dE5v ZGUobWVzc2FnZSkpOworICAgICAgICBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgnY29uc29sZScp LmFwcGVuZENoaWxkKHBhcmFncmFwaCk7CisgICAgfQorICAgIHZhciBtZXNzYWdlID0gInNlbGVj dGVkIGluZGV4IGlzICIgKyBzZWxlY3Quc2VsZWN0ZWRJbmRleDsKKyAgICB2YXIgcGFyYWdyYXBo ID0gZG9jdW1lbnQuY3JlYXRlRWxlbWVudCgncCcpOworICAgIHBhcmFncmFwaC5hcHBlbmRDaGls ZChkb2N1bWVudC5jcmVhdGVUZXh0Tm9kZShtZXNzYWdlKSk7CisgICAgZG9jdW1lbnQuZ2V0RWxl bWVudEJ5SWQoJ2NvbnNvbGUnKS5hcHBlbmRDaGlsZChwYXJhZ3JhcGgpOworfQorPC9zY3JpcHQ+ Cis8L2hlYWQ+Cis8Ym9keSBvbmxvYWQ9InRlc3QoKTsiPgorPHA+VGhpcyB0ZXN0cyByZXBsYWNp bmcgYW4gb3B0aW9uIHVzaW5nIEphdmFTY3JpcHQuIElmIHN1Y2Nlc3NmdWwsIHRoZSBvcHRpb25z IHNob3VsZCBiZSAiUmVwbGFjZW1lbnQgT3B0aW9uIDAiLCAiT3B0aW9uIDEiLCBhbmQgIk9wdGlv biAyIiwgYW5kIHNlbGVjdGVkIGluZGV4IHNob3VsZCBiZSAwLjxmb3JtPgorPGZvcm0+Cis8c2Vs ZWN0IGlkPSJzZWxlY3QiPgorPG9wdGlvbj5PcHRpb24gMDwvb3B0aW9uPgorPG9wdGlvbj5PcHRp b24gMTwvb3B0aW9uPgorPG9wdGlvbj5PcHRpb24gMjwvb3B0aW9uPgorPC9zZWxlY3Q+Cis8L2Zv cm0+Cis8cCBpZD0iY29uc29sZSI+PC9wPgorPC9ib2R5PgoKUHJvcGVydHkgY2hhbmdlcyBvbjog TGF5b3V0VGVzdHMvZmFzdC9mb3Jtcy9zZWxlY3QtcmVwbGFjZS1vcHRpb24uaHRtbApfX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fCk5hbWU6IHN2bjptaW1lLXR5cGUKICAgKyB0ZXh0L2h0bWwKTmFtZTogc3ZuOmVvbC1zdHls ZQogICArIG5hdGl2ZQoK