Add clone to Stage. Rename place to mix and PolymorphicUnion to Stage. Cleanup.

src/core/SkBitmapProcShader.h

 /*
  * Copyright 2006 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkBitmapProcShader_DEFINED
 #define SkBitmapProcShader_DEFINED
 
@@ skipping 37 matching lines @@
                                 const SkBitmapProvider&, const ContextRec&, void* storage);
 
     typedef SkShader INHERITED;
 };
 
 // Commonly used allocator. It currently is only used to allocate up to 3 objects. The total
 // bytes requested is calculated using one of our large shaders, its context size plus the size of
 // an Sk3DBlitter in SkDraw.cpp
 // Note that some contexts may contain other contexts (e.g. for compose shaders), but we've not
 // yet found a situation where the size below isn't big enough.
-typedef SkSmallAllocator<3, 2100> SkTBlitterAllocator;
+typedef SkSmallAllocator<3, 2400> SkTBlitterAllocator;

[mtklein, 2016/04/08 19:30:29]

Think it's good idea to define a constant on SkLinearBitmapPipeline, or in its
.h, so we don't have to keep making sure to update these sites (and that we got
all of them)?

[herb_g, 2016/04/08 19:51:55]

This is good advice, but I'm going to ignore it for now. I would like to
experiment with a clean way of doing this. C++ should be able to calculate all
these things for us, but we need to get those calculations to work across all
compilers.

 
 // If alloc is non-nullptr, it will be used to allocate the returned SkShader, and MUST outlive
 // the SkShader.
 sk_sp<SkShader> SkMakeBitmapShader(const SkBitmap& src, SkShader::TileMode, SkShader::TileMode,
                                    const SkMatrix* localMatrix, SkTBlitterAllocator* alloc);
 
 #endif