Eliminate uncalled will___LiveResize and inLiveResize from RenderViewImpl through WebWidget down to…

third_party/WebKit/Source/core/layout/ImageQualityController.cpp

Index: third_party/WebKit/Source/core/layout/ImageQualityController.cpp
diff --git a/third_party/WebKit/Source/core/layout/ImageQualityController.cpp b/third_party/WebKit/Source/core/layout/ImageQualityController.cpp
index 75ab18561e39a6550264e630c8453dafc682c4f1..c08be45d358441a21a20f02bd6fd0edc6069dc67 100644
--- a/third_party/WebKit/Source/core/layout/ImageQualityController.cpp
+++ b/third_party/WebKit/Source/core/layout/ImageQualityController.cpp
@@ -131,13 +131,6 @@ void ImageQualityController::objectDestroyed(const LayoutObject& object)
 void ImageQualityController::highQualityRepaintTimerFired(Timer<ImageQualityController>*)
 {
     for (auto* layoutObject : m_objectLayerSizeMap.keys()) {
-        if (LocalFrame* frame = layoutObject->document().frame()) {
-            // If this layoutObject's containing FrameView is in live resize, punt the timer and hold back for now.
-            if (frame->view() && frame->view()->inLiveResize()) {
-                restartTimer();
-                return;
-            }
-        }
         ObjectLayerSizeMap::iterator i = m_objectLayerSizeMap.find(layoutObject);
         if (i != m_objectLayerSizeMap.end()) {
             // Only invalidate the object if it is animating.
@@ -188,22 +181,6 @@ bool ImageQualityController::shouldPaintAtLowQuality(const LayoutObject& object,
         }
     }
 
-    // If the containing FrameView is being resized, paint at low quality until resizing is finished.
-    if (LocalFrame* frame = object.document().frame()) {
-        bool frameViewIsCurrentlyInLiveResize = frame->view() && frame->view()->inLiveResize();
-        if (frameViewIsCurrentlyInLiveResize) {
-            set(object, innerMap, layer, layoutSize, true);
-            restartTimer();

[Fady Samuel, 2016/03/02 00:47:18]

What's this timer? Can we get rid of it?

[Mark Dittmer, 2016/03/02 12:54:55]

The timer is used to toggle between lo-fi and hi-fi scrolling on Mac. I don't
think we can git rid of it. There are two restartTimer() invocations that are
independent of the live resize logic being removed.

Here:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

and here:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit...

-            m_liveResizeOptimizationIsActive = true;

[Fady Samuel, 2016/03/02 00:47:18]

You can probably remove this.

[Mark Dittmer, 2016/03/02 12:54:55]

Done.

-            return true;
-        }
-        if (m_liveResizeOptimizationIsActive) {
-            // Live resize has ended, paint in HQ and remove this object from the list.
-            removeLayer(object, innerMap, layer);
-            return false;
-        }
-    }
-
     if (layoutSize == image->size()) {
         // There is no scale in effect. If we had a scale in effect before, we can just remove this object from the list.
         removeLayer(object, innerMap, layer);