Commit c1d48e13 authored by adele's avatar adele
Browse files

Reviewed by Darin.

        - fixed <rdar://problem/4181058> 8C45: Safari repro crash with document.write (DOM::NodeImpl::dispatchEvent)

        Test cases added:
        * manual-tests/textfield-onblur.html: Added.

        * kwq/KWQTextField.mm: (-[KWQTextFieldController setHasFocus:]): added nil check for widget, since it can get deleted during an event.


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@9846 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 7f7be48c
2005-07-20 Adele Peterson <adele@apple.com>
Reviewed by Darin.
- fixed <rdar://problem/4181058> 8C45: Safari repro crash with document.write (DOM::NodeImpl::dispatchEvent)
Test cases added:
* manual-tests/textfield-onblur.html: Added.
* kwq/KWQTextField.mm: (-[KWQTextFieldController setHasFocus:]): added nil check for widget, since it can get deleted during an event.
2005-07-20 Maciej Stachowiak <mjs@apple.com>
 
Patch from Trey Matteson <trey@usa.net>, reviewed by me.
......@@ -441,7 +441,8 @@
if (widget) {
QFocusEvent event(QEvent::FocusOut);
const_cast<QObject *>(widget->eventFilterObject())->eventFilter(widget, &event);
[KWQKHTMLPart::bridgeForWidget(widget) formControlIsResigningFirstResponder:field];
if (widget)
[KWQKHTMLPart::bridgeForWidget(widget) formControlIsResigningFirstResponder:field];
}
}
}
......
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN"
"http://www.w3.org/TR/html4/strict.dtd">
<html lang="en">
<head>
<script>
function init () {
document.getElementById("test").focus();
}
function foo () {
document.open();
document.write('Test has passed.');
document.close();
}
</script>
</head>
<body onload="init();">
<p><b>BUG ID:</b> <a href="rdar://problem/4181058">4181058</a> 8C45: Safari repro crash with document.write (DOM::NodeImpl::dispatchEvent)</p>
<p id="test" style="background-color:skyblue; padding:3px;"><b>STEPS TO TEST:</b>
<ul>
<li> Press the Tab key. This will remove focus from the text field, and should call document.write without crashing.
</ul>
</p>
<p id="success" style="background-color:palegreen; padding:3px;"><b>TEST PASS:</b>
You should see text that says "Test has passed".
</p>
<p id="failure" style="background-color:#FF3300; padding:3px;"><b>TEST FAIL:</b>
If the test fails, the browser will crash.
</p>
<input type="text" id="test" onblur="foo();" />
</body>
</html>
\ No newline at end of file
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment