Move SkPaint mangement for 2D canvas into CanvasRenderingContext2DState

Source/platform/graphics/GraphicsTypes.h

Index: Source/platform/graphics/GraphicsTypes.h
diff --git a/Source/platform/graphics/GraphicsTypes.h b/Source/platform/graphics/GraphicsTypes.h
index d2a07eb4be0130b17ce6feba85144172e3ca4672..bf58c1ea94138b86a5bd6051559c82e9c4bbd954 100644
--- a/Source/platform/graphics/GraphicsTypes.h
+++ b/Source/platform/graphics/GraphicsTypes.h
@@ -73,6 +73,17 @@ enum CompositeOperator {
 enum ShadowMode {
     DrawShadowAndForeground,
     DrawShadowOnly,
+    DrawForegroundOnly
+};
+
+enum CanvasPaintType {

[Stephen White, 2015/03/19 20:35:17]

Could this (and the CanvasShadowStrategy below) be in
CanvasRenderingContext2DState?

[Justin Novosad, 2015/03/19 21:03:36]

They could...

These enums are used in the declaration of
CanvasRenderingContext2D::fullCanvasCompositedDraw
If I move them to CanvasRenderingContext2DState.h, that would add a new include
dependency. Currently, CanvasRenderingContext2DState only needs to be forward
declared in CanvasRenderingContext2D.h

WDYT?

[Stephen White, 2015/03/19 22:37:26]

I don't mind the dependency. I think keeping canvas-specific stuff localized to
canvas headers is a nice property.

+    FillPaintType,
+    StrokePaintType,
+};
+
+enum CanvasShadowStrategy {
+    DrawLooperShadowStrategy,
+    ImageFilterShadowStrategy,
 };
 
 enum AntiAliasingMode {