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

Issue 22385011: Removing deprecated constructors and setDevice from SkDeferredCanvas (Closed)

Created:
7 years, 4 months ago by Justin Novosad
Modified:
7 years, 4 months ago
CC:
skia-review_googlegroups.com, iannucci
Visibility:
Public.

Description

Removing deprecated constructors and setDevice from SkDeferredCanvas R=senorblanco@google.com BUR=crbug.com/270143

Patch Set 1 #

Total comments: 1

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -181 lines) Patch
M bench/DeferredCanvasBench.cpp View 1 chunk +1 line, -6 lines 0 comments Download
M bench/DeferredSurfaceCopyBench.cpp View 1 chunk +1 line, -6 lines 0 comments Download
M bench/benchmain.cpp View 1 chunk +1 line, -6 lines 0 comments Download
M gm/gmmain.cpp View 2 chunks +0 lines, -8 lines 0 comments Download
M include/utils/SkDeferredCanvas.h View 1 2 chunks +1 line, -38 lines 0 comments Download
M src/utils/SkDeferredCanvas.cpp View 1 2 chunks +0 lines, -27 lines 0 comments Download
M tests/CanvasTest.cpp View 1 chunk +1 line, -6 lines 0 comments Download
M tests/DeferredCanvasTest.cpp View 14 chunks +14 lines, -84 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
Justin Novosad
PTAL N.B.: Must wait for Blink patch to land before committing this: https://codereview.chromium.org/22634006/
7 years, 4 months ago (2013-08-08 19:48:43 UTC) #1
Stephen White
LGTM https://codereview.chromium.org/22385011/diff/1/include/utils/SkDeferredCanvas.h File include/utils/SkDeferredCanvas.h (left): https://codereview.chromium.org/22385011/diff/1/include/utils/SkDeferredCanvas.h#oldcode50 include/utils/SkDeferredCanvas.h:50: /** DEPRACATED, use create instead So is it ...
7 years, 4 months ago (2013-08-08 20:15:33 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://skia-tree-status.appspot.com/cq/junov@chromium.org/22385011/1
7 years, 4 months ago (2013-08-09 13:45:53 UTC) #3
commit-bot: I haz the power
Retried try job too often on Build-Mac10.7-Clang-x86-Release-Trybot for step(s) BuildTests http://108.170.217.252:10117/buildstatus?builder=Build-Mac10.7-Clang-x86-Release-Trybot&number=367
7 years, 4 months ago (2013-08-09 13:56:40 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://skia-tree-status.appspot.com/cq/junov@chromium.org/22385011/16003
7 years, 4 months ago (2013-08-09 14:08:58 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://skia-tree-status.appspot.com/cq/junov@chromium.org/22385011/16003
7 years, 4 months ago (2013-08-09 14:25:16 UTC) #6
commit-bot: I haz the power
Failed to apply patch for bench/DeferredCanvasBench.cpp: While running patch -p0 --forward --force --no-backup-if-mismatch; patching file ...
7 years, 4 months ago (2013-08-09 14:25:22 UTC) #7
rmistry
7 years, 4 months ago (2013-08-09 17:17:08 UTC) #8
+iannucci

Looks like this CL was successfully submitted by the CQ
(https://code.google.com/p/skia/source/detail?r=10657) but the last message was
a CQ patch failure. Any ideas Robbie?

Thanks!

Powered by Google App Engine
This is Rietveld 408576698