Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(170)

Side by Side Diff: Source/WebCore/page/EventHandler.cpp

Issue 9160033: Merge 105212 - Crash in in WebCore::EventHandler::mouseMoved. (Closed) Base URL: http://svn.webkit.org/repository/webkit/branches/chromium/963/
Patch Set: Created 8 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « LayoutTests/fast/events/resources/mouse-move.html ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserv ed. 2 * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserv ed.
3 * Copyright (C) 2006 Alexey Proskuryakov (ap@webkit.org) 3 * Copyright (C) 2006 Alexey Proskuryakov (ap@webkit.org)
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions 6 * modification, are permitted provided that the following conditions
7 * are met: 7 * are met:
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 1538 matching lines...) Expand 10 before | Expand all | Expand 10 after
1549 1549
1550 RenderLayer* layer = renderer->enclosingLayer(); 1550 RenderLayer* layer = renderer->enclosingLayer();
1551 if (!layer) 1551 if (!layer)
1552 return 0; 1552 return 0;
1553 1553
1554 return layer; 1554 return layer;
1555 } 1555 }
1556 1556
1557 bool EventHandler::mouseMoved(const PlatformMouseEvent& event, bool onlyUpdateSc rollbars) 1557 bool EventHandler::mouseMoved(const PlatformMouseEvent& event, bool onlyUpdateSc rollbars)
1558 { 1558 {
1559 RefPtr<FrameView> protector(m_frame->view());
1560
1559 HitTestResult hoveredNode = HitTestResult(LayoutPoint()); 1561 HitTestResult hoveredNode = HitTestResult(LayoutPoint());
1560 bool result = handleMouseMoveEvent(event, &hoveredNode, onlyUpdateScrollbars ); 1562 bool result = handleMouseMoveEvent(event, &hoveredNode, onlyUpdateScrollbars );
1561 1563
1562 if (onlyUpdateScrollbars) 1564 if (onlyUpdateScrollbars)
1563 return result; 1565 return result;
1564 1566
1565 Page* page = m_frame->page(); 1567 Page* page = m_frame->page();
1566 if (!page) 1568 if (!page)
1567 return result; 1569 return result;
1568 1570
(...skipping 1761 matching lines...) Expand 10 before | Expand all | Expand 10 after
3330 } 3332 }
3331 3333
3332 return defaultPrevented; 3334 return defaultPrevented;
3333 } 3335 }
3334 3336
3335 3337
3336 3338
3337 #endif 3339 #endif
3338 3340
3339 } 3341 }
OLDNEW
« no previous file with comments | « LayoutTests/fast/events/resources/mouse-move.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698