Title: [105776] branches/chromium/963
Revision
105776
Author
[email protected]
Date
2012-01-24 12:40:43 -0800 (Tue, 24 Jan 2012)

Log Message

Merge 105212 - Crash in in WebCore::EventHandler::mouseMoved.
BUG=110374
Review URL: https://chromiumcodereview.appspot.com/9160033

Modified Paths

Added Paths

Diff

Copied: branches/chromium/963/LayoutTests/fast/events/mouse-moved-remove-frame-crash-expected.txt (from rev 105212, trunk/LayoutTests/fast/events/mouse-moved-remove-frame-crash-expected.txt) (0 => 105776)


--- branches/chromium/963/LayoutTests/fast/events/mouse-moved-remove-frame-crash-expected.txt	                        (rev 0)
+++ branches/chromium/963/LayoutTests/fast/events/mouse-moved-remove-frame-crash-expected.txt	2012-01-24 20:40:43 UTC (rev 105776)
@@ -0,0 +1 @@
+PASS

Copied: branches/chromium/963/LayoutTests/fast/events/mouse-moved-remove-frame-crash.html (from rev 105212, trunk/LayoutTests/fast/events/mouse-moved-remove-frame-crash.html) (0 => 105776)


--- branches/chromium/963/LayoutTests/fast/events/mouse-moved-remove-frame-crash.html	                        (rev 0)
+++ branches/chromium/963/LayoutTests/fast/events/mouse-moved-remove-frame-crash.html	2012-01-24 20:40:43 UTC (rev 105776)
@@ -0,0 +1,27 @@
+<!DOCTYPE html>
+<html>
+<script>
+if (window.layoutTestController) {
+    layoutTestController.dumpAsText();
+    layoutTestController.waitUntilDone();
+}
+
+function onMouseMove()
+{
+    document.body.innerHTML = "PASS";
+
+    if (window.layoutTestController)
+        layoutTestController.notifyDone();
+}
+
+function runTest()
+{
+    root = document.getElementById('root').contentDocument;
+    root.addEventListener('mousemove', onMouseMove, 0);
+    eventSender.mouseMoveTo(1, 1);
+    eventSender.mouseMoveTo(0, 0);
+}
+</script>
+<style>body { margin: 0px; }</style>
+<object data="" id="root" _onload_="runTest()"></object>
+</html>

Copied: branches/chromium/963/LayoutTests/fast/events/resources/mouse-move.html (from rev 105212, trunk/LayoutTests/fast/events/resources/mouse-move.html) (0 => 105776)


--- branches/chromium/963/LayoutTests/fast/events/resources/mouse-move.html	                        (rev 0)
+++ branches/chromium/963/LayoutTests/fast/events/resources/mouse-move.html	2012-01-24 20:40:43 UTC (rev 105776)
@@ -0,0 +1,2 @@
+<div id="test"></div><iframe src=""
+

Modified: branches/chromium/963/Source/WebCore/page/EventHandler.cpp (105775 => 105776)


--- branches/chromium/963/Source/WebCore/page/EventHandler.cpp	2012-01-24 20:32:37 UTC (rev 105775)
+++ branches/chromium/963/Source/WebCore/page/EventHandler.cpp	2012-01-24 20:40:43 UTC (rev 105776)
@@ -1556,6 +1556,8 @@
 
 bool EventHandler::mouseMoved(const PlatformMouseEvent& event, bool onlyUpdateScrollbars)
 {
+    RefPtr<FrameView> protector(m_frame->view());
+
     HitTestResult hoveredNode = HitTestResult(LayoutPoint());
     bool result = handleMouseMoveEvent(event, &hoveredNode, onlyUpdateScrollbars);
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to