Fix YUV horizontal and vertical sampling checks

src/codec/SkJpegCodec.cpp

Index: src/codec/SkJpegCodec.cpp
diff --git a/src/codec/SkJpegCodec.cpp b/src/codec/SkJpegCodec.cpp
index d36051511c664c14cc9bfeddc642426b7fc24df1..cbe8946b4733dfbf1a21807156a0a67df1c035a1 100644
--- a/src/codec/SkJpegCodec.cpp
+++ b/src/codec/SkJpegCodec.cpp
@@ -568,10 +568,10 @@ static bool is_yuv_supported(jpeg_decompress_struct* dinfo) {
     // that of Y would be an extremely difficult change, given that clients
     // allocate memory as if the size of the Y plane is always the size of the
     // image.  However, this case is very, very rare.
-    if (!(1 == dinfo->comp_info[1].h_samp_factor) &&
-         (1 == dinfo->comp_info[1].v_samp_factor) &&
-         (1 == dinfo->comp_info[2].h_samp_factor) &&
-         (1 == dinfo->comp_info[2].v_samp_factor)) {
+    if  (!((1 == dinfo->comp_info[1].h_samp_factor) &&

[msarett, 2016/03/04 00:34:01]

A couple characters makes a big difference :).

Whew I really needed an easy bug!

[mtklein, 2016/03/04 13:23:27]

This may indicate we should De Morgan the logic?

if (1 != dinfo->comp_info[1].h_samp_factor ||
    1 != dinfo->comp_info[1].v_samp_factor ||
    1 != dinfo->comp_info[2].h_samp_factor ||
    1 != dinfo->comp_info[2].v_samp_factor) {
   return false;
} 

[msarett, 2016/03/04 13:42:51]

Yeah I think that's clearer.

+           (1 == dinfo->comp_info[1].v_samp_factor) &&
+           (1 == dinfo->comp_info[2].h_samp_factor) &&
+           (1 == dinfo->comp_info[2].v_samp_factor))) {
         return false;
     }