Eliminate skia::PlatformCanvas - Step 1

webkit/glue/webthemeengine_impl_win.cc

Index: webkit/glue/webthemeengine_impl_win.cc
===================================================================
--- webkit/glue/webthemeengine_impl_win.cc	(revision 80026)
+++ webkit/glue/webthemeengine_impl_win.cc	(working copy)
@@ -1,4 +1,4 @@
-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// Copyright (c) 2011 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.
 
@@ -24,58 +24,67 @@
   return result;
 }
 
+static skia::PlatformCanvas* ToPlatformCanvas(WebCanvas* canvas) {
+  return static_cast<skia::PlatformCanvas*>(canvas);
+}
+

[reed1, 2011/04/01 17:44:42]

In preparation for the future, is it clear to create two helpers here?

static HDC PlatformBeginPaint(SkCanvas* canvas) {
    return <PlatformCanvas*>(canvas)->beginPlatformPaint();
}

static void PlatformEndPaint(SkCanvas* canvas) {
    ...
}

Its nearly the same, but it would seem to prepare the way going forward to
replace the cast with SkMetaData calls. Just a thought, not a strong suggestion.

[alokp, 2011/04/01 19:57:38]

Done.

 void WebThemeEngineImpl::paintButton(
     WebCanvas* canvas, int part, int state, int classic_state,
     const WebRect& rect) {
-  HDC hdc = canvas->beginPlatformPaint();
+  skia::PlatformCanvas* platform_canvas = ToPlatformCanvas(canvas);
+  HDC hdc = platform_canvas->beginPlatformPaint();
 
   RECT native_rect = WebRectToRECT(rect);
   gfx::NativeThemeWin::instance()->PaintButton(
       hdc, part, state, classic_state, &native_rect);
 
-  canvas->endPlatformPaint();
+  platform_canvas->endPlatformPaint();
 }
 
 void WebThemeEngineImpl::paintMenuList(
     WebCanvas* canvas, int part, int state, int classic_state,
     const WebRect& rect) {
-  HDC hdc = canvas->beginPlatformPaint();
+  skia::PlatformCanvas* platform_canvas = ToPlatformCanvas(canvas);
+  HDC hdc = platform_canvas->beginPlatformPaint();
 
   RECT native_rect = WebRectToRECT(rect);
   gfx::NativeThemeWin::instance()->PaintMenuList(
       hdc, part, state, classic_state, &native_rect);
 
-  canvas->endPlatformPaint();
+  platform_canvas->endPlatformPaint();
 }
 
 void WebThemeEngineImpl::paintScrollbarArrow(
     WebCanvas* canvas, int state, int classic_state,
     const WebRect& rect) {
-  HDC hdc = canvas->beginPlatformPaint();
+  skia::PlatformCanvas* platform_canvas = ToPlatformCanvas(canvas);
+  HDC hdc = platform_canvas->beginPlatformPaint();
 
   RECT native_rect = WebRectToRECT(rect);
   gfx::NativeThemeWin::instance()->PaintScrollbarArrow(
       hdc, state, classic_state, &native_rect);
 
-  canvas->endPlatformPaint();
+  platform_canvas->endPlatformPaint();
 }
 
 void WebThemeEngineImpl::paintScrollbarThumb(
     WebCanvas* canvas, int part, int state, int classic_state,
     const WebRect& rect) {
-  HDC hdc = canvas->beginPlatformPaint();
+  skia::PlatformCanvas* platform_canvas = ToPlatformCanvas(canvas);
+  HDC hdc = platform_canvas->beginPlatformPaint();
 
   RECT native_rect = WebRectToRECT(rect);
   gfx::NativeThemeWin::instance()->PaintScrollbarThumb(
       hdc, part, state, classic_state, &native_rect);
 
-  canvas->endPlatformPaint();
+  platform_canvas->endPlatformPaint();
 }
 
 void WebThemeEngineImpl::paintScrollbarTrack(
     WebCanvas* canvas, int part, int state, int classic_state,
     const WebRect& rect, const WebRect& align_rect) {
-  HDC hdc = canvas->beginPlatformPaint();
+  skia::PlatformCanvas* platform_canvas = ToPlatformCanvas(canvas);
+  HDC hdc = platform_canvas->beginPlatformPaint();
 
   RECT native_rect = WebRectToRECT(rect);
   RECT native_align_rect = WebRectToRECT(align_rect);
@@ -83,26 +92,28 @@
       hdc, part, state, classic_state, &native_rect, &native_align_rect,
       canvas);
 
-  canvas->endPlatformPaint();
+  platform_canvas->endPlatformPaint();
 }
 
 void WebThemeEngineImpl::paintSpinButton(
     WebCanvas* canvas, int part, int state, int classic_state,
     const WebRect& rect) {
-  HDC hdc = canvas->beginPlatformPaint();
+  skia::PlatformCanvas* platform_canvas = ToPlatformCanvas(canvas);
+  HDC hdc = platform_canvas->beginPlatformPaint();
 
   RECT native_rect = WebRectToRECT(rect);
   gfx::NativeThemeWin::instance()->PaintSpinButton(
       hdc, part, state, classic_state, &native_rect);
 
-  canvas->endPlatformPaint();
+  platform_canvas->endPlatformPaint();
 }
 
 void WebThemeEngineImpl::paintTextField(
     WebCanvas* canvas, int part, int state, int classic_state,
     const WebRect& rect, WebColor color, bool fill_content_area,
     bool draw_edges) {
-  HDC hdc = canvas->beginPlatformPaint();
+  skia::PlatformCanvas* platform_canvas = ToPlatformCanvas(canvas);
+  HDC hdc = platform_canvas->beginPlatformPaint();
 
   RECT native_rect = WebRectToRECT(rect);
   COLORREF c = skia::SkColorToCOLORREF(color);
@@ -111,32 +122,36 @@
       hdc, part, state, classic_state, &native_rect, c, fill_content_area,
       draw_edges);
 
-  canvas->endPlatformPaint();
+  platform_canvas->endPlatformPaint();
 }
 
 void WebThemeEngineImpl::paintTrackbar(
     WebCanvas* canvas, int part, int state, int classic_state,
     const WebRect& rect) {
-  HDC hdc = canvas->beginPlatformPaint();
+  skia::PlatformCanvas* platform_canvas = ToPlatformCanvas(canvas);
+  HDC hdc = platform_canvas->beginPlatformPaint();
 
   RECT native_rect = WebRectToRECT(rect);
   gfx::NativeThemeWin::instance()->PaintTrackbar(
       hdc, part, state, classic_state, &native_rect, canvas);
 
-  canvas->endPlatformPaint();
+  platform_canvas->endPlatformPaint();
 }
 
 void WebThemeEngineImpl::paintProgressBar(
     WebCanvas* canvas, const WebRect& barRect, const WebRect& valueRect,
     bool determinate, double animatedSeconds)
 {
-  HDC hdc = canvas->beginPlatformPaint();
+  skia::PlatformCanvas* platform_canvas = ToPlatformCanvas(canvas);
+  HDC hdc = platform_canvas->beginPlatformPaint();
+
   RECT native_bar_rect = WebRectToRECT(barRect);
   RECT native_value_rect = WebRectToRECT(valueRect);
   gfx::NativeThemeWin::instance()->PaintProgressBar(
       hdc, &native_bar_rect,
       &native_value_rect, determinate, animatedSeconds, canvas);
-  canvas->endPlatformPaint();
+
+  platform_canvas->endPlatformPaint();
 }
 
 }  // namespace webkit_glue