Raster display item lists via a visual rect RTree.

third_party/WebKit/Source/web/LinkHighlightImpl.cpp

 /*
  * Copyright (C) 2012 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:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 252 matching lines @@
 {
     return gfx::Rect(0, 0, contentLayer()->layer()->bounds().width, contentLayer()->layer()->bounds().height);
 }
 
 void LinkHighlightImpl::paintContents(WebDisplayItemList* webDisplayItemList, WebContentLayerClient::PaintingControlSetting paintingControl)
 {
     if (!m_node || !m_node->layoutObject())
         return;
 
     SkPictureRecorder recorder;
-    SkCanvas* canvas = recorder.beginRecording(paintableRegion().width(), paintableRegion().height());
+    gfx::Rect visualRect = paintableRegion();
+    SkCanvas* canvas = recorder.beginRecording(visualRect.width(), visualRect.height());
 
     SkPaint paint;
     paint.setStyle(SkPaint::kFill_Style);
     paint.setFlags(SkPaint::kAntiAlias_Flag);
     paint.setColor(m_node->layoutObject()->style()->tapHighlightColor().rgb());
     canvas->drawPath(m_path.skPath(), paint);
 
     RefPtr<const SkPicture> picture = adoptRef(recorder.endRecording());
-    // TODO(wkorman): Pass actual visual rect with the drawing item.
-    webDisplayItemList->appendDrawingItem(IntRect(), picture.get());
+    // TODO(wkorman): Consider using m_path.skPath().getBounds() as the visual rect below.
+    webDisplayItemList->appendDrawingItem(WebRect(visualRect.x(), visualRect.y(), visualRect.width(), visualRect.height()), picture.get());
 }
 
 void LinkHighlightImpl::startHighlightAnimationIfNeeded()
 {
     if (m_isAnimating)
         return;
 
     m_isAnimating = true;
     const float startOpacity = 1;
     // FIXME: Should duration be configurable?
@@ skipping 89 matching lines @@
 {
     return clipLayer();
 }
 
 WebCompositorAnimationPlayer* LinkHighlightImpl::compositorPlayer() const
 {
     return m_compositorPlayer.get();
 }
 
 } // namespace blink