Fixed comment about SkLightingShader::Context memory handling
authordvonbeck <dvonbeck@google.com>
Mon, 8 Aug 2016 18:25:13 +0000 (11:25 -0700)
committerCommit bot <commit-bot@chromium.org>
Mon, 8 Aug 2016 18:25:13 +0000 (11:25 -0700)
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2225093002

Review-Url: https://codereview.chromium.org/2225093002

src/core/SkLightingShader.cpp

index e35ede3..e8280b1 100644 (file)
@@ -58,8 +58,8 @@ public:
 
     class LightingShaderContext : public SkShader::Context {
     public:
-        // The context takes ownership of the states. It will call their destructors
-        // but will NOT free the memory.
+        // The context takes ownership of the context and provider. It will call their destructors
+        // and then indirectly free their memory by calling free() on heapAllocated
         LightingShaderContext(const SkLightingShaderImpl&, const ContextRec&,
                               SkShader::Context* diffuseContext, SkNormalSource::Provider*,
                               void* heapAllocated);