Switch Chromium to new Skia SkCanvas::drawPicture method

Switch Chromium to new Skia SkCanvas::drawPicture method

As of Skia 9b14f26d (Alter SkCanvas::drawPicture
 (devirtualize, take const SkPicture, take pointer) -
 https://codereview.chromium.org/313613004) 
SkCanvas::drawPicture has an alternate signature. 
The old entry point is deprecated.

Note: this patch cannot be landed until the 
corresponding Skia change is rolled into Chromium & Blink.

enne: cc
fmalita: skia

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=275155

[f(malita), 2014-06-04 15:16:55]

https://codereview.chromium.org/315963003/diff/1/skia/ext/benchmarking_canvas.cc
File skia/ext/benchmarking_canvas.cc (right):

https://codereview.chromium.org/315963003/diff/1/skia/ext/benchmarking_canvas...
skia/ext/benchmarking_canvas.cc:117: virtual void drawPicture(const SkPicture*
picture) OVERRIDE {
Override onDrawPicture() instead.

[robertphillips, 2014-06-04 15:42:41]

https://codereview.chromium.org/315963003/diff/1/skia/ext/benchmarking_canvas.cc
File skia/ext/benchmarking_canvas.cc (right):

https://codereview.chromium.org/315963003/diff/1/skia/ext/benchmarking_canvas...
skia/ext/benchmarking_canvas.cc:117: virtual void drawPicture(const SkPicture*
picture) OVERRIDE {
On 2014/06/04 15:16:55, Florin Malita wrote:
> Override onDrawPicture() instead.

Done.

[f(malita), 2014-06-04 16:00:29]

lgtm

[enne (OOO), 2014-06-04 17:45:31]

lgtm

Can you update your patch description to wrap at 72 columns?

[robertphillips, 2014-06-04 17:49:17]

wrapped

[robertphillips, 2014-06-05 11:58:17]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2014-06-05 11:59:32]

CQ is trying da patch. Follow status at

https://chromium-status.appspot.com/cq/robertphillips@google.com/315963003/20001

[commit-bot: I haz the power, 2014-06-05 16:35:43]

Change committed as 275155