looks good in spirit.  just one nit on the implementation.

https://codereview.chromium.org/21949007/diff/1/tests/PaintTest.cpp
File tests/PaintTest.cpp (right):

https://codereview.chromium.org/21949007/diff/1/tests/PaintTest.cpp#newcode139
tests/PaintTest.cpp:139: static bool PaintEquals(const SkPaint& a, const SkPaint
&b) {
I think we can ditch the class and just have a free function?

e.g. SkPaintShallowEqual(const SkPaint& a, const SkPaint& b) ?

looks good in spirit.  just one nit on the implementation.

Changed from friend class to friend function.

change is good, except I think we can simplify the unittest to not require
special knowledge.

https://codereview.chromium.org/21949007/diff/17001/tests/PaintTest.cpp
File tests/PaintTest.cpp (right):

https://codereview.chromium.org/21949007/diff/17001/tests/PaintTest.cpp#newco...
tests/PaintTest.cpp:136: bool SkPaintShallowEquals(const SkPaint& a, const
SkPaint &b) {
I think this guy should just test for the fields it cares about, using public
getters, just for this test. Then it need not be a friend of SkPaint.

Small static function that only does what the test needs.

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/sglez@google.com/21949007/21001