Extract most of the mutable state of SkShader into a separate Context object.

include/core/SkEmptyShader.h

 
 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 
 #ifndef SkEmptyShader_DEFINED
 #define SkEmptyShader_DEFINED
 
 #include "SkShader.h"
 
 /**
  *  \class SkEmptyShader
- *  A Shader that always draws nothing. Its setContext always returns false,
- *  so it never expects that its shadeSpan() methods will get called.
+ *  A Shader that always draws nothing. Its createContext always returns NULL.
  */
 class SK_API SkEmptyShader : public SkShader {
 public:
     SkEmptyShader() {}
 
-    virtual uint32_t getFlags() SK_OVERRIDE;
-    virtual uint8_t getSpan16Alpha() const SK_OVERRIDE;
-    virtual bool setContext(const SkBitmap&, const SkPaint&,
-                            const SkMatrix&) SK_OVERRIDE;
-    virtual void shadeSpan(int x, int y, SkPMColor span[], int count) SK_OVERRIDE;
-    virtual void shadeSpan16(int x, int y, uint16_t span[], int count) SK_OVERRIDE;
-    virtual void shadeSpanAlpha(int x, int y, uint8_t alpha[], int count) SK_OVERRIDE;
+    virtual size_t contextSize() const SK_OVERRIDE { return sizeof(SkShader::Context); }

[scroggo, 2014/03/24 21:24:46]

It seems misleading to me that this returns a non-zero size, since createContext
will *always* return NULL.

[Dominik Grewe, 2014/03/26 17:22:22]

I agree it's misleading but the problem is that we allocate space for the
context before calling createContext. And when allocating space,
SkSmallAllocator checks that the given size is >= sizeof(T) with T =
SkShader::Context.
I'll add a comment explaining this.

[scroggo, 2014/03/26 23:13:09]

Ah, that makes sense.

I'll see if I can find it, but I think you have another class that returns 0 for
contextSize().

[Dominik Grewe, 2014/03/27 14:27:20]

Only for SkGradientShader which isn't implement yet.

+
+    virtual SkShader::Context* createContext(const SkBitmap&, const SkPaint&,
+                                             const SkMatrix&, void*) const SK_OVERRIDE {
+        // validContext returns false.
+        return NULL;
+    }
 
     SK_DEVELOPER_TO_STRING()
     SK_DECLARE_PUBLIC_FLATTENABLE_DESERIALIZATION_PROCS(SkEmptyShader)
 
 protected:
     SkEmptyShader(SkReadBuffer& buffer) : INHERITED(buffer) {}
 
+    virtual bool validContext(const SkBitmap&, const SkPaint&,
+                             const SkMatrix&) SK_OVERRIDE { return false; }
 private:
     typedef SkShader INHERITED;
 };
 
 #endif