Implementation for cc::Picture::IsCheapInRect().

cc/picture.cc

 // Copyright 2012 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 "base/debug/trace_event.h"
 #include "cc/content_layer_client.h"
 #include "cc/picture.h"
 #include "cc/rendering_stats.h"
 #include "skia/ext/analysis_canvas.h"
 #include "third_party/skia/include/core/SkCanvas.h"
@@ skipping 83 matching lines @@
   canvas->restore();
   picture_->endRecording();
 
   opaque_rect_ = gfx::ToEnclosedRect(opaque_layer_rect);
 }
 
 void Picture::Raster(
     SkCanvas* canvas,
     gfx::Rect content_rect,
     float contents_scale) {
-char strbuf[256];
-if (IsCheapInRect(content_rect)) {
-sprintf(strbuf, "CHEAP Picture::Raster");
-} else {
-sprintf(strbuf, "EXPENSIVE Picture::Raster");
-}
-SkBitmap emptyBitmap;
-emptyBitmap.setConfig(SkBitmap::kARGB_8888_Config, picture_->width(),
-                      picture_->height());
-skia::AnalysisDevice device(emptyBitmap);
-skia::AnalysisCanvas aCanvas(&device, gfx::RectToSkRect(content_rect));
-aCanvas.drawPicture(*picture_);
-TRACE_EVENT1("cc", strdup(strbuf), "cost", aCanvas.getEstimatedCost());
-  //TRACE_EVENT2("cc", strdup(strbuf),
-               //"width", layer_rect_.width(), "height", layer_rect_.height());
+  TRACE_EVENT2("cc", "Picture::Raster",
+               "width", layer_rect_.width(), "height", layer_rect_.height());
   DCHECK(picture_);
 
+  bool isCheap = IsCheapInRect(content_rect, contents_scale);
   canvas->save();
   canvas->clipRect(gfx::RectToSkRect(content_rect));
   canvas->scale(contents_scale, contents_scale);
   canvas->translate(layer_rect_.x(), layer_rect_.y());
   canvas->drawPicture(*picture_);
   canvas->restore();
 }
 
-bool Picture::IsCheapInRect(const gfx::Rect& layer_rect) {
-TRACE_EVENT0("cc", "Picture::IsCheapInRect");
+bool Picture::IsCheapInRect(const gfx::Rect& layer_rect,
+                            float contents_scale) const {
+  TRACE_EVENT0("cc", "Picture::IsCheapInRect");

[nduca, 2013/02/06 08:42:25]

is your git branch mucked up? this looks like its based on your previous patch?

+
   SkBitmap emptyBitmap;
-  emptyBitmap.setConfig(SkBitmap::kNo_Config, picture_->width(),
-                        picture_->height());
+  emptyBitmap.setConfig(SkBitmap::kNo_Config, layer_rect.width(),
+                        layer_rect.height());
   skia::AnalysisDevice device(emptyBitmap);
-  skia::AnalysisCanvas canvas(&device, gfx::RectToSkRect(layer_rect));
-  // If a picture has low estimated cost, go ahead and
-  // wait until we have more pictures to draw (to amortize startup costs).
-  // If estimated cost is high, start it now so that compositing isn't
-  // blocked waiting for it.
+  skia::AnalysisCanvas canvas(&device);
+
+  canvas.clipRect(gfx::RectToSkRect(layer_rect));

[nduca, 2013/02/06 08:42:25]

this line looks a little different than the call in ::Raster, any reason why?
namely ,we use layer_rect here but content_rect up above

+  canvas.scale(contents_scale, contents_scale);
+  canvas.translate(layer_rect_.x(), layer_rect_.y());
   canvas.drawPicture(*picture_);
   return canvas.isCheap();
 }
 
 void Picture::GatherPixelRefs(const gfx::Rect& layer_rect,
                               std::list<skia::LazyPixelRef*>& pixel_ref_list) {
   DCHECK(picture_);
   SkData* pixel_refs = SkPictureUtils::GatherPixelRefs(
       picture_.get(), SkRect::MakeXYWH(layer_rect.x(),
                                        layer_rect.y(),
@@ skipping 13 matching lines @@
     if (*refs && (*refs)->getURI() && !strncmp(
         (*refs)->getURI(), labelLazyDecoded, 4)) {
       pixel_ref_list.push_back(static_cast<skia::LazyPixelRef*>(*refs));
     }
     refs++;
   }
   pixel_refs->unref();
 }
 
 }  // namespace cc