occasional crash in Chromium in dispatching keyEvent

https://bugs.webkit.org/show_bug.cgi?id=67941

Patch by Scott Graham <scottmg@chromium.org> on 2011-09-23
Reviewed by Darin Fisher.

Use focusedWebCoreNode() to handle null document for frame when
retrieving focused node in handling keyEvent.

* src/WebViewImpl.cpp:
(WebKit::WebViewImpl::keyEvent):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@95861 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 41ed8679
2011-09-23 Scott Graham <scottmg@chromium.org>
occasional crash in Chromium in dispatching keyEvent
https://bugs.webkit.org/show_bug.cgi?id=67941
Reviewed by Darin Fisher.
Use focusedWebCoreNode() to handle null document for frame when
retrieving focused node in handling keyEvent.
* src/WebViewImpl.cpp:
(WebKit::WebViewImpl::keyEvent):
2011-09-23 Fady Samuel <fsamuel@chromium.org>
Refactor paintOverhangAreas to allow non-Mac Chromium platforms to reuse code
......
......@@ -654,7 +654,7 @@ bool WebViewImpl::keyEvent(const WebKeyboardEvent& event)
if (WebInputEvent::RawKeyDown == event.type) {
// Suppress the next keypress event unless the focused node is a plug-in node.
// (Flash needs these keypress events to handle non-US keyboards.)
Node* node = frame->document()->focusedNode();
Node* node = focusedWebCoreNode();
if (!node || !node->renderer() || !node->renderer()->isEmbeddedObject())
m_suppressNextKeypressEvent = true;
}
......
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