Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1634)

Unified Diff: cc/resources/drawing_display_item.cc

Issue 952893003: Update from https://crrev.com/317530 (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Fix gn for nacl Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « cc/resources/drawing_display_item.h ('k') | cc/resources/gpu_rasterizer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/drawing_display_item.cc
diff --git a/cc/resources/drawing_display_item.cc b/cc/resources/drawing_display_item.cc
index 1ef149fa0177ef0c781c9ae0de711fbd8a8f1cfb..6dffad9aa8595641f085eafa7403ee0c37e8f2b8 100644
--- a/cc/resources/drawing_display_item.cc
+++ b/cc/resources/drawing_display_item.cc
@@ -18,9 +18,8 @@
namespace cc {
-DrawingDisplayItem::DrawingDisplayItem(skia::RefPtr<SkPicture> picture,
- gfx::PointF location)
- : picture_(picture), location_(location) {
+DrawingDisplayItem::DrawingDisplayItem(skia::RefPtr<SkPicture> picture)
+ : picture_(picture) {
}
DrawingDisplayItem::~DrawingDisplayItem() {
@@ -29,7 +28,6 @@ DrawingDisplayItem::~DrawingDisplayItem() {
void DrawingDisplayItem::Raster(SkCanvas* canvas,
SkDrawPictureCallback* callback) const {
canvas->save();
- canvas->translate(location_.x(), location_.y());
if (callback)
picture_->playback(canvas, callback);
else
@@ -39,7 +37,6 @@ void DrawingDisplayItem::Raster(SkCanvas* canvas,
void DrawingDisplayItem::RasterForTracing(SkCanvas* canvas) const {
canvas->save();
- canvas->translate(location_.x(), location_.y());
// The picture debugger in about:tracing doesn't drill down into |drawPicture|
// operations. Calling |playback()| rather than |drawPicture()| causes the
// skia operations in |picture_| to appear individually in the picture
@@ -54,7 +51,7 @@ bool DrawingDisplayItem::IsSuitableForGpuRasterization() const {
}
int DrawingDisplayItem::ApproximateOpCount() const {
- return picture_->approximateOpCount() + sizeof(gfx::PointF);
+ return picture_->approximateOpCount();
}
size_t DrawingDisplayItem::PictureMemoryUsage() const {
@@ -66,9 +63,11 @@ void DrawingDisplayItem::AsValueInto(
base::trace_event::TracedValue* array) const {
array->BeginDictionary();
array->SetString("name", "DrawingDisplayItem");
- array->SetString("location",
- base::StringPrintf("[%f,%f]", picture_->cullRect().x(),
- picture_->cullRect().y()));
+ array->SetString(
+ "cullRect",
+ base::StringPrintf("[%f,%f,%f,%f]", picture_->cullRect().x(),
+ picture_->cullRect().y(), picture_->cullRect().width(),
+ picture_->cullRect().height()));
std::string b64_picture;
PictureDebugUtil::SerializeAsBase64(picture_.get(), &b64_picture);
array->SetString("skp64", b64_picture);
« no previous file with comments | « cc/resources/drawing_display_item.h ('k') | cc/resources/gpu_rasterizer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698