Added PointLights to SkLights::Light

src/core/SkShadowShader.cpp

 /*
  * Copyright 2016 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 #include "SkLights.h"
 #include "SkReadBuffer.h"
@@ skipping 574 matching lines @@
     str->appendf("ShadowShader: ()");
 }
 #endif
 
 sk_sp<SkFlattenable> SkShadowShaderImpl::CreateProc(SkReadBuffer& buf) {
 
     // Discarding SkShader flattenable params
     bool hasLocalMatrix = buf.readBool();
     SkAssertResult(!hasLocalMatrix);
 
-    int numLights = buf.readInt();
-
-    SkLights::Builder builder;
-
-    for (int l = 0; l < numLights; ++l) {
-        bool isAmbient = buf.readBool();
-
-        SkColor3f color;
-        if (!buf.readScalarArray(&color.fX, 3)) {
-            return nullptr;
-        }
-
-        if (isAmbient) {
-            builder.add(SkLights::Light(color));
-        } else {
-            SkVector3 dir;
-            if (!buf.readScalarArray(&dir.fX, 3)) {
-                return nullptr;
-            }
-
-            sk_sp<SkImage> depthMap;
-            if (!(depthMap = sk_ref_sp<SkImage>(buf.readImage()))) {
-                return nullptr;
-            }
-
-            SkLights::Light light = SkLights::Light(color, dir);
-            light.setShadowMap(depthMap);
-
-            builder.add(light);
-        }
-    }
-
-    sk_sp<SkLights> lights(builder.finish());
+    sk_sp<SkLights> lights;

[robertphillips, 2016/08/11 14:56:18]

same here

[vjiaoblack, 2016/08/11 19:18:54]

Done.

+    lights->unFlatten(buf);
 
     int diffuseWidth = buf.readInt();
     int diffuseHeight = buf.readInt();
 
     sk_sp<SkShader> povDepthShader(buf.readFlattenable<SkShader>());
     sk_sp<SkShader> diffuseShader(buf.readFlattenable<SkShader>());
 
     return sk_make_sp<SkShadowShaderImpl>(std::move(povDepthShader),
                                           std::move(diffuseShader),
                                           std::move(lights),
                                           diffuseWidth, diffuseHeight);
 }
 
 void SkShadowShaderImpl::flatten(SkWriteBuffer& buf) const {
     this->INHERITED::flatten(buf);
 
-    buf.writeInt(fLights->numLights());
-
-    for (int l = 0; l < fLights->numLights(); ++l) {
-        const SkLights::Light& light = fLights->light(l);
-
-        bool isAmbient = SkLights::Light::kAmbient_LightType == light.type();
-
-        buf.writeBool(isAmbient);
-        buf.writeScalarArray(&light.color().fX, 3);
-        if (!isAmbient) {
-            buf.writeScalarArray(&light.dir().fX, 3);
-        }
-
-        buf.writeImage(light.getShadowMap());
-    }
+    fLights->flatten(buf);
 
     buf.writeInt(fDiffuseWidth);
     buf.writeInt(fDiffuseHeight);
 
     buf.writeFlattenable(fPovDepthShader.get());
     buf.writeFlattenable(fDiffuseShader.get());
 }
 
 size_t SkShadowShaderImpl::onContextSize(const ContextRec& rec) const {
     return sizeof(ShadowShaderContext);
@@ skipping 48 matching lines @@
 
 ///////////////////////////////////////////////////////////////////////////////
 
 SK_DEFINE_FLATTENABLE_REGISTRAR_GROUP_START(SkShadowShader)
     SK_DEFINE_FLATTENABLE_REGISTRAR_ENTRY(SkShadowShaderImpl)
 SK_DEFINE_FLATTENABLE_REGISTRAR_GROUP_END
 
 ///////////////////////////////////////////////////////////////////////////////
 
 #endif