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

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

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 113 matching lines @@
 {
     m_objectLayerSizeMap.remove(&object);
     if (m_objectLayerSizeMap.isEmpty()) {
         m_timer->stop();
     }
 }
 
 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.
             if (i->value.isResizing) {
                 // TODO(wangxianzhu): Use LayoutObject::getMutableForPainting().
                 const_cast<LayoutObject*>(layoutObject)->setShouldDoFullPaintInvalidation();
             }
             i->value.isResizing = false;
         }
     }
@@ skipping 30 matching lines @@
     LayoutSize oldSize;
     bool isFirstResize = true;
     if (innerMap) {
         LayerSizeMap::iterator j = innerMap->find(layer);
         if (j != innerMap->end()) {
             isFirstResize = false;
             oldSize = j->value;
         }
     }
 
-    // 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);
         return false;
     }
 
     // If an animated resize is active for this object, paint in low quality and kick the timer ahead.
     if (objectIsResizing) {
         bool sizesChanged = oldSize != layoutSize;
         set(object, innerMap, layer, layoutSize, sizesChanged);
@@ skipping 17 matching lines @@
     }
     // This object has been resized to two different sizes while the timer
     // is active, so draw at low quality, set the flag for animated resizes and
     // the object to the list for high quality redraw.
     set(object, innerMap, layer, layoutSize, true);
     restartTimer();
     return true;
 }
 
 } // namespace blink