Add ColorMatrix filter tests from Android CTS.

Add ColorMatrix filter tests from Android CTS.

BUG=skia:3848
Committed: https://skia.googlesource.com/skia/+/b500ffa1a253dca35a7ff846ebece9fe570e3565

[djsollen, 2015-05-19 19:18:57]

djsollen@google.com changed reviewers:
+ reed@google.com

[djsollen, 2015-05-19 20:03:00]

djsollen@google.com changed reviewers:
+ scroggo@google.com

[scroggo, 2015-05-19 20:10:44]

lgtm

https://codereview.chromium.org/1134753008/diff/1/tests/ColorMatrixTest.cpp
File tests/ColorMatrixTest.cpp (right):

https://codereview.chromium.org/1134753008/diff/1/tests/ColorMatrixTest.cpp#n...
tests/ColorMatrixTest.cpp:28: static inline void
test_colorMatrix(skiatest::Reporter* reporter) {
Why is this its own function? Why not just use DEF_TEST here?

[djsollen, 2015-05-19 20:22:48]

The CQ bit was checked by djsollen@google.com

[commit-bot: I haz the power, 2015-05-19 20:23:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1134753008/1

[djsollen, 2015-05-19 20:24:29]

https://codereview.chromium.org/1134753008/diff/1/tests/ColorMatrixTest.cpp
File tests/ColorMatrixTest.cpp (right):

https://codereview.chromium.org/1134753008/diff/1/tests/ColorMatrixTest.cpp#n...
tests/ColorMatrixTest.cpp:28: static inline void
test_colorMatrix(skiatest::Reporter* reporter) {
On 2015/05/19 20:10:44, scroggo wrote:
> Why is this its own function? Why not just use DEF_TEST here?

Personal preference.  My intention was that this method would just replicate the
CTS test and additional methods could be added for other colorMatrix tests. 
However, I should have named this test_colorMatrix_CTS or something like that.

[commit-bot: I haz the power, 2015-05-19 20:27:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-05-19 20:27:13]

Try jobs failed on following builders:
  Build-Win-MSVC-x86_64-Debug-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Win-MSVC-x86_6...)

[djsollen, 2015-05-19 20:41:59]

The CQ bit was checked by djsollen@google.com

[djsollen, 2015-05-19 20:42:00]

The patchset sent to the CQ was uploaded after l-g-t-m from scroggo@google.com
Link to the patchset: https://codereview.chromium.org/1134753008/#ps20001
(title: "windows fix")

[commit-bot: I haz the power, 2015-05-19 20:42:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1134753008/20001

[commit-bot: I haz the power, 2015-05-19 20:46:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-05-19 20:46:48]

Try jobs failed on following builders:
  Build-Win-MSVC-x86_64-Debug-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Win-MSVC-x86_6...)

[djsollen, 2015-06-04 17:16:02]

The CQ bit was checked by djsollen@google.com

[djsollen, 2015-06-04 17:16:02]

The patchset sent to the CQ was uploaded after l-g-t-m from scroggo@google.com
Link to the patchset: https://codereview.chromium.org/1134753008/#ps40001
(title: " ")

[commit-bot: I haz the power, 2015-06-04 17:16:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1134753008/40001

[commit-bot: I haz the power, 2015-06-04 19:16:31]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-04 19:16:33]

Try jobs failed on following builders:
  Build-Mac10.8-Clang-x86_64-Release-Trybot on client.skia.compile
(JOB_TIMED_OUT, no build URL)

[djsollen, 2015-06-05 16:38:05]

The CQ bit was checked by djsollen@google.com

[commit-bot: I haz the power, 2015-06-05 16:38:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1134753008/40001

[commit-bot: I haz the power, 2015-06-05 16:41:24]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/b500ffa1a253dca35a7ff846ebece9fe570e3565