Sky: Allow clients to specify tile mode for gradients (repeating or mirror).

sky/engine/core/painting/CanvasGradient.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef SKY_ENGINE_CORE_PAINTING_CANVASGRADIENT_H_
 #define SKY_ENGINE_CORE_PAINTING_CANVASGRADIENT_H_
 
 #include "sky/engine/core/painting/CanvasColor.h"
 #include "sky/engine/core/painting/Point.h"
 #include "sky/engine/core/painting/Shader.h"
 #include "sky/engine/tonic/dart_wrappable.h"
 #include "third_party/skia/include/effects/SkGradientShader.h"
 
 namespace blink {
 
 class CanvasGradient : public Shader {
     DEFINE_WRAPPERTYPEINFO();
  public:
   ~CanvasGradient() override;
   static PassRefPtr<CanvasGradient> create();
 
   void initLinear(const Vector<Point>& end_points,
                   const Vector<SkColor>& colors,
-                  const Vector<float>& color_stops);
+                  const Vector<float>& color_stops,
+                  unsigned tile_mode);

[eseidel, 2015/06/09 17:45:22]

Why can't the C++ use an enum?

[Matt Perry, 2015/06/09 19:01:25]

It can, but then I'd have to write code to do the conversion. I figured that
since we have type safety on the Dart side, we don't need the extra code on the
C++ side. WDYT?

   void initRadial(const Point& center,
                   double radius,
                   const Vector<SkColor>& colors,
-                  const Vector<float>& color_stops);
+                  const Vector<float>& color_stops,
+                  unsigned tile_mode);
 
  private:
   CanvasGradient();
 };
 
 } // namespace blink
 
 #endif  // SKY_ENGINE_CORE_PAINTING_CANVASGRADIENT_H_