Simplify platform_canvas.h by recognizing that PlatformCanvas does not actually extend

chrome/renderer/automation/automation_renderer_helper.cc

Index: chrome/renderer/automation/automation_renderer_helper.cc
===================================================================
--- chrome/renderer/automation/automation_renderer_helper.cc	(revision 165342)
+++ chrome/renderer/automation/automation_renderer_helper.cc	(working copy)
@@ -67,9 +67,10 @@
   view->layout();
   frame->setScrollOffset(WebSize(0, 0));
 
-  skia::PlatformCanvas canvas(
-      new_size.width, new_size.height, true /* is_opaque */);
-  view->paint(webkit_glue::ToWebCanvas(&canvas),
+  SkCanvas* canvas = skia::CreatePlatformCanvas(new_size.width, new_size.height,

[sky, 2012/11/01 17:41:39]

Is it possible to return a value like we did before so that folks don't have to
remember to wrap in an SkAutoUnref?

[reed1, 2012/11/01 21:07:21]

Makes sense. I'll see what I can come up with.

[reed1, 2012/11/02 15:49:39]

ScopedPlatformCanvas

Done.

+                                                true /* is_opaque */);
+  SkAutoUnref au(canvas);
+  view->paint(webkit_glue::ToWebCanvas(canvas),
               gfx::Rect(0, 0, new_size.width, new_size.height));
 
   frame->setCanHaveScrollbars(true);
@@ -80,7 +81,7 @@
   frame->setScrollOffset(WebSize(old_scroll.width - min_scroll.width,
                                  old_scroll.height - min_scroll.height));
 
-  const SkBitmap& bmp = skia::GetTopDevice(canvas)->accessBitmap(false);
+  const SkBitmap& bmp = skia::GetTopDevice(*canvas)->accessBitmap(false);
   SkAutoLockPixels lock_pixels(bmp);
   // EncodeBGRA uses FORMAT_SkBitmap, which doesn't work on windows for some
   // cases dealing with transparency. See crbug.com/96317. Use FORMAT_BGRA.