Change all one-off lifecycle callers to FrameView::updateLayoutAndStyleForPainting

Source/core/page/EventHandler.cpp

Index: Source/core/page/EventHandler.cpp
diff --git a/Source/core/page/EventHandler.cpp b/Source/core/page/EventHandler.cpp
index c0ba3e05201a722358b65af8f51f4a76d27094a1..1e70b2cbf48cfb297e9744ad53f2f3e582e2f571 100644
--- a/Source/core/page/EventHandler.cpp
+++ b/Source/core/page/EventHandler.cpp
@@ -2423,9 +2423,6 @@ bool EventHandler::handleGestureTap(const GestureEventWithHitTestResults& target
     // co-ordinates (updating layout/style as hitTestResultAtPoint normally would).
     // FIXME: Use a hit-test cache to avoid unnecessary hit tests. http://crbug.com/398920
     if (currentHitTest.innerNode()) {
-        LocalFrame* mainFrame = m_frame->localFrameRoot();

[chrishtr, 2015/05/08 00:33:37]

These are not needed because of the code in LayoutView::hitTest().

-        if (mainFrame && mainFrame->view())
-            mainFrame->view()->updateLayoutAndStyleIfNeededRecursive();
         adjustedPoint = frameView->rootFrameToContents(gestureEvent.position());
         currentHitTest = hitTestResultInFrame(m_frame, adjustedPoint, hitType);
     }
@@ -2450,9 +2447,6 @@ bool EventHandler::handleGestureTap(const GestureEventWithHitTestResults& target
 
     // FIXME: Use a hit-test cache to avoid unnecessary hit tests. http://crbug.com/398920
     if (currentHitTest.innerNode()) {
-        LocalFrame* mainFrame = m_frame->localFrameRoot();
-        if (mainFrame && mainFrame->view())
-            mainFrame->view()->updateLayoutAndStyleIfNeededRecursive();
         adjustedPoint = frameView->rootFrameToContents(gestureEvent.position());
         currentHitTest = hitTestResultInFrame(m_frame, adjustedPoint, hitType);
     }