Bug 23911 – Exception thrown when opening a <select> pop-up that uses a web font

• Status: RESOLVED
• Resolution: FIXED
• Component: Forms

[Eddy Nigg]

Navigate to the URL https://www.startssl.com/?app=12 and click on register. In Safari the selects for country and states lock up. Code which could cause the problem is:

onchange="x_get_states_by_country_name(this.value,changeState);"

Which is AJAX call. The return function is:

function changeState(states) {
  stateElement = document.getElementById("State");
  stateElement.options.length = 0;
  var i = 0;

  for (var i in states){ 

    stateElement.options[i]=new Option(states[i], states[i], false, false);

  }
}

[Mark Rowe (bdash)]

<rdar://problem/6579204>

[Eddy Nigg]

Is there any resolution coming for this bug or is there a known workaround for the JavaScript code?

[Eddy Nigg]

The JavaScript from comment 0 works with all browsers except Safari. I'm trying to have this bug somehow moved forward. If anything is need in order to help, please ping me.

[Alexey Proskuryakov]

One thing that could help is a reduction - a self-contained short example that shows the problem in action.

[Eddy Nigg]

Thanks for following up. The example can be seen at https://www.startssl.com/?app=11&action=regform

The code used is exactly as in comment 0 in particular:

stateElement.options[i]=new Option(states[i], states[i], false, false);

or a bit clearer:

element.options[1]=new Option(stringLabel, stringValue, false, false);

Incidentally I'm reading again through some JavaScript specs and it appears to me that the last false value might not be compatible (with some versions). Trying that right now...

[Alexey Proskuryakov]

I was talking about a reduced test case - a dozen lines or so of code total.

That's of course not a requirement for filing a bug, I only suggested that because you specifically asked how you could help.

> Incidentally I'm reading again through some JavaScript specs and it appears to
> me that the last false value might not be compatible (with some versions).

If that works in both IE and Firefox, we should strongly consider making it work in WebKit.

[Eddy Nigg]

I tried right now with  new Option(stringLabel, stringValue, false); and it doesn't work either with Safari.

[Eddy Nigg]

Some more information with this, which could be another issue: The SELECT fields have onchange="doSomething();" as well. Could this be the problem?

[Eddy Nigg]

(In reply to comment #6)
> I was talking about a reduced test case - a dozen lines or so of code total.
> 
> That's of course not a requirement for filing a bug, I only suggested that
> because you specifically asked how you could help.

Thank you, I understand...but doesn't the page source help? See https://www.startssl.com/?app=11&action=regform in particular line 367 and 868 with:

<select name="Country" onchange="x_get_states_by_country_name(this.value,changeState); x_get_phonecode_by_country_name(this.value,changePhoneCode);">

Not sure how much time I'd need to produce a test case mimicking the same behaviour... :S

[Eddy Nigg]

I'm now almost 100% certain that the cause for this behavior is the ONCHANGE in the SELECT field. Other SELECT fields which have onchange="doSomething();" lock up as well - without relation to the new Option() function.

[David Kilzer (:ddkilzer)]

This still occurs.  The "Register" link is now "Sign-up" apparently.

This issue is also mentioned on an Ars Technica article now:

<http://arstechnica.com/security/news/2009/12/how-to-get-set-with-a-secure-sertificate-for-free.ars>

[Keith Anderson]

Created attachment 46180 [details]
Test Case For onChange Functionality

I ran into this bug on their web site and their web developer pointed me to this bug report. I knocked up a test case that makes an attempt to recreate this issue with ONLY an onChange and AJAX request. I cannot duplicate the behavior, so I don't think that whatever is happening on that site is related to onChange.

There is a lot of document fiddling happening onLoad and otherwise, though - and something is preventing the popup menus from even being activated. Something else is going on here.

I've included as attachments my test case showing that XHR and onChange on a select element work fine.

[Keith Anderson]

Created attachment 46181 [details]
Response for test case

Put this next to the HTML test case - it functions as the AJAX response.

[Alexey Proskuryakov]

Created attachment 46287 [details]
test case

You may need to click the menu a couple of times before the problem reproduces with this test case.

[Alexey Proskuryakov]

This is caused by having an online @font-face for menu item text.

#0  0x94bf6dd4 in +[NSException raise:format:] ()
#1  0x9625b1f8 in -[NSCFDictionary setObject:forKey:] ()
#2  0x058f8ab3 in WebCore::PopupMenu::populate (this=0x1f8a6f60) at /Users/ap/Safari/OpenSource/WebCore/platform/mac/PopupMenuMac.mm:85
#3  0x058f8d03 in WebCore::PopupMenu::show (this=0x1f8a6f60, r=@0xbfffebb0, v=0xa904000, index=0) at /Users/ap/Safari/OpenSource/WebCore/platform/mac/PopupMenuMac.mm:107
#4  0x059ac7c2 in WebCore::RenderMenuList::showPopup (this=0x1f8a76bc) at /Users/ap/Safari/OpenSource/WebCore/rendering/RenderMenuList.cpp:290
#5  0x05a620af in WebCore::SelectElement::menuListDefaultEventHandler (data=@0x1f8a6a3c, element=0x1f8a69f0, event=0x203f4a60, htmlForm=0x0) at /Users/ap/Safari/OpenSource/WebCore/dom/SelectElement.cpp:660

[Eddy Nigg]

Hallelujah! That's easy to work around for us and hopefully there will be a solution for WebKit based browsers soon.

Created attachment 47052 [details]
Use the system font for the pop-up if an NSFont cannot be obtained

I tried to use the CTFontRef (which can be obtained for custom fonts), as it is toll-free bridged with NSFont, however the pop-up just used the system font. This will take care of the exception for now.

[Darin Adler]

Comment on attachment 47052 [details]
Use the system font for the pop-up if an NSFont cannot be obtained

> +                    font = style.font().weight() < FontWeightBold ? [NSFont systemFontOfSize:size] :[NSFont boldSystemFontOfSize:size];

Need a space after the : but before [NSFont].

Fixed in <http://trac.webkit.org/projects/webkit/changeset/53565>

[Eddy Nigg]

Thanks to all for working on this.