Added PointLights to SkLights::Light

include/core/SkLights.h

 
 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkLights_DEFINED
 #define SkLights_DEFINED
 
 #include "SkPoint3.h"
 #include "SkRefCnt.h"
 #include "../private/SkTArray.h"
 #include "SkImage.h"
 
 class SK_API SkLights  : public SkRefCnt {
 public:
     class Light {
     public:
         enum LightType {
             kAmbient_LightType,       // only 'fColor' is used
-            kDirectional_LightType
+            kDirectional_LightType,
+            kPoint_LightType
         };
 
         Light(const Light& other)
             : fType(other.fType)
             , fColor(other.fColor)
             , fDirection(other.fDirection)
             , fShadowMap(other.fShadowMap) {
         }
 
         Light(Light&& other)
             : fType(other.fType)
             , fColor(other.fColor)
             , fDirection(other.fDirection)
             , fShadowMap(std::move(other.fShadowMap)) {
         }
 
-        Light(const SkColor3f& color)
-            : fType(kAmbient_LightType)
-            , fColor(color) {
-            fDirection.set(0.0f, 0.0f, 1.0f);
+        static Light MakeAmbient(const SkColor3f& color) {

[robertphillips, 2016/08/10 17:48:39]

just "return Light(...);"
no need for the temporary variable

[vjiaoblack, 2016/08/11 14:27:23]

Done.

+            Light light(kAmbient_LightType, color, SkVector3::Make(0.0f, 0.0f, 1.0f));
+            return light;
         }
 
-        Light(const SkColor3f& color, const SkVector3& dir)
-            : fType(kDirectional_LightType)
-            , fColor(color)
-            , fDirection(dir) {
-            if (!fDirection.normalize()) {
-                fDirection.set(0.0f, 0.0f, 1.0f);
+        static Light MakeDirectional(const SkColor3f& color, const SkVector3& dir) {
+            Light light(kDirectional_LightType, color, dir);
+            if (!light.fDirection.normalize()) {
+                light.fDirection.set(0.0f, 0.0f, 1.0f);
             }
+            return light;
+        }
+
+        static Light MakePoint(const SkColor3f& color, const SkPoint3& pos) {

[robertphillips, 2016/08/10 17:48:39]

Same here

[vjiaoblack, 2016/08/11 14:27:23]

Done.

+            Light light(kPoint_LightType, color, pos);
+            return light;
         }
 
         LightType type() const { return fType; }
         const SkColor3f& color() const { return fColor; }
         const SkVector3& dir() const { 
-            SkASSERT(kAmbient_LightType != fType);
+            SkASSERT(kDirectional_LightType == fType);
             return fDirection; 
         }
+        const SkPoint3& pos() const {
+            SkASSERT(kPoint_LightType == fType);
+            return fDirection;
+        }
 
         void setShadowMap(sk_sp<SkImage> shadowMap) {
             fShadowMap = std::move(shadowMap);
         }
 
         SkImage* getShadowMap() const {
             return fShadowMap.get();
         }
 
         Light& operator= (const Light& b) {
             if (this == &b) {
                 return *this;
             }
 
             fColor = b.fColor;
             fType = b.fType;
             fDirection = b.fDirection;
             fShadowMap = b.fShadowMap;
             return *this;
         }
 
     private:
         LightType   fType;
         SkColor3f   fColor;           // linear (unpremul) color. Range is 0..1 in each channel.

[robertphillips, 2016/08/10 17:48:39]

Fix comment

[vjiaoblack, 2016/08/11 14:27:23]

Done.

         SkVector3   fDirection;       // direction towards the light (+Z is out of the screen).
                                       // If degenerate, it will be replaced with (0, 0, 1).
         sk_sp<SkImage> fShadowMap;
+
+        Light(LightType type, const SkColor3f color, const SkVector3 dir) {

[robertphillips, 2016/08/10 17:48:39]

Don't need "this->" on member variables

[vjiaoblack, 2016/08/11 14:27:23]

Done.

+            this->fType = type;
+            this->fColor = color;
+            this->fDirection = dir;
+        }
     };
 
     class Builder {
     public:
         Builder() : fLights(new SkLights) { }
     
         void add(const Light& light) {
             if (fLights) {
                 fLights->fLights.push_back(light);
             }
@@ skipping 27 matching lines @@
 
 private:
     SkLights() {}
 
     SkTArray<Light> fLights;
 
     typedef SkRefCnt INHERITED;
 };
 
 #endif