Allow proper highlighting on universal overflow scroll.

Source/core/rendering/RenderLayer.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
  *
  * Portions are Copyright (C) 1998 Netscape Communications Corporation.
  *
  * Other contributors:
  *   Robert O'Callahan <roc+@cs.cmu.edu>
  *   David Baron <dbaron@fas.harvard.edu>
  *   Christian Biesinger <cbiesinger@web.de>
  *   Randall Jesup <rjesup@wgate.com>
@@ skipping 3162 matching lines @@
             }
 
             if (hitLayer)
                 return hitLayer;
         }
     }
 
     return 0;
 }
 
+void RenderLayer::blockSelectionGapsBoundsChanged()
+{
+    if (needsCompositedScrolling() && hasCompositedLayerMapping())

[Ian Vollick, 2014/06/12 14:54:57]

This looks like a chicken-and-egg thing, but I think it might be avoidable. Can
we schedule a compositing update of the composited-scroll variety and ensure
that the scrolling block selection stuff is freshened during those updates?

[hartmanng, 2014/06/13 14:51:37]

Done.

+        m_compositedLayerMapping->updateScrollingBlockSelection();

[abarth-chromium, 2014/06/13 04:31:05]

Yeah, you should be able to just:

setNeedsCompositingInputsUpdate();
compositor()->setNeedsCompositingUpdate(CompositingUpdateRebuildTree);

If CompositedLayerMapping::updateGraphicsLayerConfiguration were smart enough to
recognize layer change, then you could use
CompositingUpdateAfterCompositingInputChange instead, but the net result would
be the same.

[hartmanng, 2014/06/13 14:51:37]

Would
compositor()->setNeedsCompositingUpdate(CompositingUpdateOnCompositedScroll);
work now that I don't need to mess around with GraphicsLayerTreeBuilder?

+}
+
 void RenderLayer::addBlockSelectionGapsBounds(const LayoutRect& bounds)
 {
     m_blockSelectionGapsBounds.unite(enclosingIntRect(bounds));
+    blockSelectionGapsBoundsChanged();
 }
 
 void RenderLayer::clearBlockSelectionGapsBounds()
 {
     m_blockSelectionGapsBounds = IntRect();
     for (RenderLayer* child = firstChild(); child; child = child->nextSibling())
         child->clearBlockSelectionGapsBounds();
+    blockSelectionGapsBoundsChanged();
 }
 
 void RenderLayer::repaintBlockSelectionGaps()
 {
     for (RenderLayer* child = firstChild(); child; child = child->nextSibling())
         child->repaintBlockSelectionGaps();
 
     if (m_blockSelectionGapsBounds.isEmpty())
         return;
 
     LayoutRect rect = m_blockSelectionGapsBounds;
     if (renderer()->hasOverflowClip()) {
         RenderBox* box = renderBox();
         rect.move(-box->scrolledContentOffset());
         if (!scrollableArea()->usesCompositedScrolling())
             rect.intersect(box->overflowClipRect(LayoutPoint()));
     }
     if (renderer()->hasClip())
         rect.intersect(toRenderBox(renderer())->clipRect(LayoutPoint()));
     if (!rect.isEmpty())
         renderer()->invalidatePaintRectangle(rect);
 }
 
-bool RenderLayer::hasBlockSelectionGapBounds() const
+IntRect RenderLayer::blockSelectionGapsBounds() const
 {
-    return !m_blockSelectionGapsBounds.isEmpty();
+    if (!renderer()->isRenderBlock())
+        return IntRect();
+
+    RenderBlock* renderBlock = toRenderBlock(renderer());
+    LayoutRect gapRects = renderBlock->selectionGapRectsForRepaint(renderBlock);
+
+    return pixelSnappedIntRect(gapRects);
+}
+
+RenderLayer::PresenceOfBlockSelectionGaps RenderLayer::hasBlockSelectionGapBounds() const
+{
+    return blockSelectionGapsBounds().isEmpty() ? LayerDoesNotHaveBlockSelectionGaps : LayerHasBlockSelectionGaps;
 }
 
 bool RenderLayer::intersectsDamageRect(const LayoutRect& layerBounds, const LayoutRect& damageRect, const RenderLayer* rootLayer, const LayoutPoint* offsetFromRoot) const
 {
     // Always examine the canvas and the root.
     // FIXME: Could eliminate the isDocumentElement() check if we fix background painting so that the RenderView
     // paints the root's background.
     if (isRootLayer() || renderer()->isDocumentElement())
         return true;
 
@@ skipping 635 matching lines @@
     }
 }
 
 void showLayerTree(const WebCore::RenderObject* renderer)
 {
     if (!renderer)
         return;
     showLayerTree(renderer->enclosingLayer());
 }
 #endif