From 20136f31941e3b036bf25537f1dac364a58483c9 Mon Sep 17 00:00:00 2001
From: Chris Craik <ccraik@google.com>
Date: Fri, 29 Apr 2016 17:38:31 -0700
Subject: [PATCH] Add LightingFilter behavior test

bug:28519669

Change-Id: I81ca4109bae04387f2eb73374d0f11821745fea1
---
 libs/hwui/tests/unit/SkiaBehaviorTests.cpp | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/libs/hwui/tests/unit/SkiaBehaviorTests.cpp b/libs/hwui/tests/unit/SkiaBehaviorTests.cpp
index 875e260c84cf1..e7e8af19b274c 100644
--- a/libs/hwui/tests/unit/SkiaBehaviorTests.cpp
+++ b/libs/hwui/tests/unit/SkiaBehaviorTests.cpp
@@ -18,6 +18,7 @@
 
 #include <gtest/gtest.h>
 #include <SkShader.h>
+#include <SkColorMatrixFilter.h>
 
 using namespace android;
 using namespace android::uirenderer;
@@ -28,7 +29,7 @@ using namespace android::uirenderer;
  */
 TEST(SkiaBehavior, CreateBitmapShader1x1) {
     SkBitmap origBitmap = TestUtils::createSkBitmap(1, 1);
-    std::unique_ptr<SkShader> s(SkShader::CreateBitmapShader(
+    SkAutoTUnref<SkShader> s(SkShader::CreateBitmapShader(
             origBitmap,
             SkShader::kClamp_TileMode,
             SkShader::kRepeat_TileMode));
@@ -48,3 +49,17 @@ TEST(SkiaBehavior, genIds) {
     bitmap.notifyPixelsChanged();
     EXPECT_NE(genId, bitmap.getGenerationID());
 }
+
+TEST(SkiaBehavior, lightingColorFilter_simplify) {
+    SkAutoTUnref<SkColorFilter> filter(SkColorMatrixFilter::CreateLightingFilter(0x11223344, 0));
+
+    SkColor observedColor;
+    SkXfermode::Mode observedMode;
+    ASSERT_TRUE(filter->asColorMode(&observedColor, &observedMode));
+    EXPECT_EQ(0xFF223344, observedColor);
+    EXPECT_EQ(SkXfermode::Mode::kModulate_Mode, observedMode);
+
+    SkAutoTUnref<SkColorFilter> failFilter(
+            SkColorMatrixFilter::CreateLightingFilter(0x11223344, 0x1));
+    EXPECT_FALSE(filter->asColorMode(nullptr, nullptr));
+}