Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(35)

Side by Side Diff: src/core/SkShadowShader.h

Issue 2287553002: Moved ambient lights out of SkLight's light array (Closed) Base URL: https://skia.googlesource.com/skia@master
Patch Set: made req changes Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright 2016 Google Inc. 2 * Copyright 2016 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 8
9 #ifndef SkShadowShader_DEFINED 9 #ifndef SkShadowShader_DEFINED
10 #define SkShadowShader_DEFINED 10 #define SkShadowShader_DEFINED
11 11
12 #ifdef SK_EXPERIMENTAL_SHADOWING 12 #ifdef SK_EXPERIMENTAL_SHADOWING
13 13
14 class SkLights; 14 class SkLights;
15 class SkShader; 15 class SkShader;
16 16
17 class SK_API SkShadowShader { 17 class SK_API SkShadowShader {
18 public: 18 public:
19 /** This shader combines the diffuse color in 'diffuseShader' with the shado ws 19 /** This shader combines the diffuse color in 'diffuseShader' with the shado ws
20 * determined by the 'povDepthShader' and the shadow maps stored in each of the 20 * determined by the 'povDepthShader' and the shadow maps stored in each of the
21 * lights in 'lights' 21 * lights in 'lights'
22 */ 22 */
23 static sk_sp<SkShader> Make(sk_sp<SkShader> povDepthShader, 23 static sk_sp<SkShader> Make(sk_sp<SkShader> povDepthShader,
24 sk_sp<SkShader> diffuseShader, 24 sk_sp<SkShader> diffuseShader,
25 sk_sp<SkLights> lights, 25 sk_sp<SkLights> lights,
26 int diffuseWidth, int diffuseHeight, 26 int diffuseWidth, int diffuseHeight,
27 const SkShadowParams& params); 27 const SkShadowParams& params);
28 28
29 // The shadow shader supports any number of ambient lights, but only 29 // The shadow shader supports any number of ambient lights, but only
30 // 4 non-ambient lights (currently just refers to directional lights). 30 // 4 non-ambient lights (currently just refers to directional lights).
robertphillips 2016/08/26 17:31:29 I think the old version was more descriptive
31 static constexpr int kMaxNonAmbientLights = 4; 31 static constexpr int kMaxLights = 4;
32 32
33 SK_DECLARE_FLATTENABLE_REGISTRAR_GROUP() 33 SK_DECLARE_FLATTENABLE_REGISTRAR_GROUP()
34 }; 34 };
35 35
36 #endif 36 #endif
37 #endif 37 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698