CL removing inheritance of SkDevice from PlatformDevice. Flavours of PlatformDevice classes now ...

skia/ext/platform_device.h

Index: skia/ext/platform_device.h
===================================================================
--- skia/ext/platform_device.h	(revision 96545)
+++ skia/ext/platform_device.h	(working copy)
@@ -6,21 +6,23 @@
 #define SKIA_EXT_PLATFORM_DEVICE_H_
 #pragma once
 
-// This file provides an easy way to include the appropriate PlatformDevice
-// header file for your platform.
-
 #if defined(WIN32)
 #include <windows.h>
+#include <vector>
 #endif
 
 #include "third_party/skia/include/core/SkPreConfig.h"
+#include "third_party/skia/include/core/SkDevice.h"
 #include "third_party/skia/include/core/SkColor.h"
 
+class SkMatrix;
+class SkPath;
+class SkRegion;
 
-class SkDevice;
 struct SkIRect;
 
-#if defined(__linux__) || defined(__FreeBSD__) || defined(__OpenBSD__) || defined(__sun)
+#if defined(__linux__) || defined(__FreeBSD__) || defined(__OpenBSD__) || \
+    defined(__sun)
 typedef struct _cairo cairo_t;
 typedef struct _cairo_rectangle cairo_rectangle_t;
 #elif defined(__APPLE__)
@@ -35,7 +37,8 @@
 #if defined(WIN32)
 typedef HDC PlatformSurface;
 typedef RECT PlatformRect;
-#elif defined(__linux__) || defined(__FreeBSD__) || defined(__OpenBSD__) || defined(__sun)
+#elif defined(__linux__) || defined(__FreeBSD__) || defined(__OpenBSD__) || \
+      defined(__sun)
 typedef cairo_t* PlatformSurface;
 typedef cairo_rectangle_t PlatformRect;
 #elif defined(__APPLE__)
@@ -59,15 +62,112 @@
                               PlatformDevice* platform_device);
 SK_API PlatformDevice* GetPlatformDevice(SkDevice* device);
 
-}  // namespace skia
 
 #if defined(WIN32)

[vandebo (ex-Chrome), 2011/08/23 17:40:35]

The standard os defines are OS_WIN, OS_MACOSX, OS_LINUX, and OS_POSIX (for mac +
linux) - see build/build_config.h.  Not sure why these aren't used above.

[Jeff Timanus, 2011/08/23 21:13:02]

Done.

-#include "skia/ext/platform_device_win.h"
+// Initializes the default settings and colors in a device context.
+SK_API void InitializeDC(HDC context);
+#elif defined (__APPLE__)
+// Returns the CGContext that backing the SkDevice.  Forwards to the bound
+// PlatformDevice.  Returns NULL if no PlatformDevice is bound.
+SK_API CGContextRef GetBitmapContext(SkDevice* device);
+#endif
+
+// A SkDevice is basically a wrapper around SkBitmap that provides a surface for
+// SkCanvas to draw into. PlatformDevice provides a surface Windows can also
+// write to. It also provides functionality to play well with GDI drawing
+// functions. This class is abstract and must be subclassed. It provides the
+// basic interface to implement it either with or without a bitmap backend.
+//
+// PlatformDevice provides an interface which sub-classes of SkDevice can also
+// provide to allow for drawing by the native platform into the device.
+class SK_API PlatformDevice {
+ public:
+#if defined(WIN32)
+  typedef HDC PlatformSurface;
 #elif defined(__APPLE__)
-#include "skia/ext/platform_device_mac.h"
+  typedef CGContextRef PlatformSurface;
 #elif defined(__linux__) || defined(__FreeBSD__) || defined(__OpenBSD__) || \
       defined(__sun)
-#include "skia/ext/platform_device_linux.h"
+  typedef cairo_t* PlatformSurface;
 #endif
 
+  virtual ~PlatformDevice() {}
+
+#if defined(__APPLE__)
+  // The CGContext that corresponds to the bitmap, used for CoreGraphics
+  // operations drawing into the bitmap. This is possibly heavyweight, so it
+  // should exist only during one pass of rendering.
+  virtual CGContextRef GetBitmapContext() = 0;
 #endif
+
+  // The DC that corresponds to the bitmap, used for GDI operations drawing
+  // into the bitmap. This is possibly heavyweight, so it should be existant
+  // only during one pass of rendering.
+  virtual PlatformSurface BeginPlatformPaint();
+
+  // Finish a previous call to beginPlatformPaint.
+  virtual void EndPlatformPaint();
+
+  // Draws to the given screen DC, if the bitmap DC doesn't exist, this will
+  // temporarily create it. However, if you have created the bitmap DC, it will
+  // be more efficient if you don't free it until after this call so it doesn't
+  // have to be created twice.  If src_rect is null, then the entirety of the
+  // source device will be copied.
+  virtual void DrawToNativeContext(PlatformSurface surface, int x, int y,

[vandebo (ex-Chrome), 2011/08/23 17:40:35]

No platform has a meaningful implementation of this, make it pure virtual?

[Jeff Timanus, 2011/08/23 21:13:02]

The routine was previously pure virtual on windows and mac, but not on linux. 
The linux VectorPlatformDevice, and BitmapPlatformDevice did not have
implementations.

I shuffled the code around so that the routine is pure virtual, and the linux
vector and bitmap classes provide implementations that SkASSERT, which does seem
a bit cleaner, and more explicit.

+                                   const PlatformRect* src_rect);
+
+  // Sets the opacity of each pixel in the specified region to be opaque.
+  virtual void MakeOpaque(int x, int y, int width, int height) { }
+
+  // Returns if GDI is allowed to render text to this device.
+  virtual bool IsNativeFontRenderingAllowed();
+
+  // True if AlphaBlend() was called during a
+  // BeginPlatformPaint()/EndPlatformPaint() pair.
+  // Used by the printing subclasses.  See |VectorPlatformDeviceEmf|.
+  virtual bool AlphaBlendUsed() const;
+
+#if defined(WIN32)
+  // Loads a SkPath into the GDI context. The path can there after be used for
+  // clipping or as a stroke. Returns false if the path failed to be loaded.
+  static bool LoadPathToDC(HDC context, const SkPath& path);
+
+  // Loads a SkRegion into the GDI context.
+  static void LoadClippingRegionToDC(HDC context, const SkRegion& region,
+                                     const SkMatrix& transformation);
+#elif defined(__APPLE__)
+  // Loads a SkPath into the CG context. The path can there after be used for
+  // clipping or as a stroke.
+  static void LoadPathToCGContext(CGContextRef context, const SkPath& path);
+
+  // Loads a SkRegion into the CG context.
+  static void LoadClippingRegionToCGContext(CGContextRef context,
+                                            const SkRegion& region,
+                                            const SkMatrix& transformation);
+#endif
+
+ protected:
+#if defined(WIN32)
+  // Arrays must be inside structures.
+  struct CubicPoints {
+    SkPoint p[4];
+  };
+  typedef std::vector<CubicPoints> CubicPath;
+  typedef std::vector<CubicPath> CubicPaths;
+
+  // Loads the specified Skia transform into the device context, excluding
+  // perspective (which GDI doesn't support).
+  static void LoadTransformToDC(HDC dc, const SkMatrix& matrix);
+
+  // Transforms SkPath's paths into a series of cubic path.
+  static bool SkPathToCubicPaths(CubicPaths* paths, const SkPath& skpath);
+#elif defined(__APPLE__)
+  // Loads the specified Skia transform into the device context
+  static void LoadTransformToCGContext(CGContextRef context,
+                                       const SkMatrix& matrix);
+#endif
+};
+
+}  // namespace skia
+
+#endif