Avoid updating compositor animations in hitTest path

Source/core/frame/FrameView.cpp

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Dirk Mueller <mueller@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  *           (C) 2006 Graham Dennis (graham.dennis@gmail.com)
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
@@ skipping 2440 matching lines @@
 void FrameView::updateAllLifecyclePhases()
 {
     frame().localFrameRoot()->view()->updateAllLifecyclePhasesInternal();
 }
 
 // TODO(chrishtr): add a scrolling update lifecycle phase, after compositing and before invalidation.
 void FrameView::updateLifecycleToCompositingCleanPlusScrolling()
 {
     frame().localFrameRoot()->view()->updateStyleAndLayoutIfNeededRecursive();
     LayoutView* view = layoutView();
-    if (view)
+    if (view && !view->compositor()->inCompositingMode())
         view->compositor()->updateIfNeededRecursive();

[chrishtr, 2015/07/22 14:14:20]

updateIfNeededRecursive() should be a no-op if called when already clean.
Instead you should fix whatever is not a no-op. Do you know which code there is
broken?

     scrollContentsIfNeededRecursive();
 
     ASSERT(lifecycle().state() >= DocumentLifecycle::CompositingClean);
 }
 
 void FrameView::updateAllLifecyclePhasesInternal()
 {
     // This must be called from the root frame, since it recurses down, not up. Otherwise the lifecycles of the frames might be out of sync.
     ASSERT(frame() == page()->mainFrame() || (!frame().tree().parent()->isLocalFrame()));
 
@@ skipping 1471 matching lines @@
 
     if (!graphicsLayer)
         return;
 
     DeprecatedPaintLayer::mapRectToPaintInvalidationBacking(localFrame->contentLayoutObject(), paintInvalidationContainer, viewRect);
 
     graphicsLayerTimingRequests.add(graphicsLayer, Vector<std::pair<int64_t, WebRect>>()).storedValue->value.append(std::make_pair(m_frame->frameID(), enclosingIntRect(viewRect)));
 }
 
 } // namespace blink