+2005-02-22 Maciej Stachowiak <mjs@apple.com>
+
+ Reviewed by Kevin.
+
+ <rdar://problem/4017066> crash in KJS::ValueImp::dispatchType() every time I load www.nytimes.com/pages/automobiles
+
+ When creating option elements, use lowercase "option" instead of
+ uppercase "OPTION" to create option elements, because only
+ lowercase works for XHTML.
+
+ * khtml/ecma/kjs_html.cpp:
+ (KJS::HTMLSelectCollection::tryPut):
+ (OptionConstructorImp::construct):
+
2005-02-22 Chris Blumenberg <cblu@apple.com>
WebCore fix for: <rdar://problem/3918056> Mail not line breaking my <pre> formatted emails on replies
if (diff < 0) { // add dummy elements
do {
- element.add(element.ownerDocument().createElement("OPTION"), DOM::HTMLElement());
+ element.add(element.ownerDocument().createElement("option"), DOM::HTMLElement());
} while (++diff);
}
else // remove elements
// out of array bounds ? first insert empty dummies
if (diff > 0) {
while (diff--) {
- element.add(element.ownerDocument().createElement("OPTION"), before);
+ element.add(element.ownerDocument().createElement("option"), before);
}
// replace an existing entry ?
} else if (diff < 0) {
Object OptionConstructorImp::construct(ExecState *exec, const List &args)
{
- DOM::Element el = doc.createElement("OPTION");
+ DOM::Element el = doc.createElement("option");
DOM::HTMLOptionElement opt = static_cast<DOM::HTMLOptionElement>(el);
int sz = args.size();
DOM::Text t = doc.createTextNode("");