Further plumb visual rect into cc:DisplayItemList.

ui/compositor/paint_recorder.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/compositor/paint_recorder.h"
 
 #include "cc/playback/display_item_list.h"
 #include "cc/playback/drawing_display_item.h"
 #include "third_party/skia/include/core/SkPictureRecorder.h"
 #include "ui/compositor/paint_cache.h"
 #include "ui/compositor/paint_context.h"
+#include "ui/gfx/geometry/rect.h"
 #include "ui/gfx/skia_util.h"
 
 namespace ui {
 
 PaintRecorder::PaintRecorder(const PaintContext& context,
                              const gfx::Size& recording_size,
                              PaintCache* cache)
     : context_(context),
       // The SkCanvas reference returned by beginRecording is shared with
       // the recorder_ so no need to store a RefPtr to it on this class, we just
       // store the gfx::Canvas.
       canvas_(skia::SharePtr(context.recorder_->beginRecording(
                                  gfx::RectToSkRect(gfx::Rect(recording_size))))
                   .get(),
               context.device_scale_factor_),
-      cache_(cache) {
+      cache_(cache),
+      visual_rect_(context.ToVisualRect(recording_size)) {

[danakj, 2015/11/06 00:32:45]

ditto, store the recording_size instead?

[wkorman, 2015/11/16 19:01:03]

Same as above, can explore subsequently.

 #if DCHECK_IS_ON()
   DCHECK(!context.inside_paint_recorder_);
   context.inside_paint_recorder_ = true;
 #endif
 }
 
 PaintRecorder::PaintRecorder(const PaintContext& context,
                              const gfx::Size& recording_size)
     : PaintRecorder(context, recording_size, nullptr) {
 }
 
 PaintRecorder::~PaintRecorder() {
 #if DCHECK_IS_ON()
   context_.inside_paint_recorder_ = false;
 #endif
 
-  auto* item = context_.list_->CreateAndAppendItem<cc::DrawingDisplayItem>();
+  auto* item =
+      context_.list_->CreateAndAppendItem<cc::DrawingDisplayItem>(visual_rect_);
   item->SetNew(skia::AdoptRef(context_.recorder_->endRecordingAsPicture()));
   if (cache_)
-    cache_->SetCache(item);
+    cache_->SetCache(visual_rect_, item);
 }
 
 }  // namespace ui