Add --serial mode to dm, runs unit tests serially

tests/ColorSpaceTest.cpp

 /*
  * Copyright 2016 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "Resources.h"
 #include "SkCodec.h"
 #include "SkColorSpace.h"
@@ skipping 241 matching lines @@
     REPORTER_ASSERT(r, !SkColorSpace::Equals(nullptr, srgb.get()));
     REPORTER_ASSERT(r, !SkColorSpace::Equals(srgb.get(), nullptr));
     REPORTER_ASSERT(r, !SkColorSpace::Equals(adobe.get(), srgb.get()));
     REPORTER_ASSERT(r, !SkColorSpace::Equals(z30.get(), srgb.get()));
     REPORTER_ASSERT(r, !SkColorSpace::Equals(z32.get(), z30.get()));
     REPORTER_ASSERT(r, !SkColorSpace::Equals(upperLeft.get(), srgb.get()));
     REPORTER_ASSERT(r, !SkColorSpace::Equals(upperLeft.get(), upperRight.get()));
     REPORTER_ASSERT(r, !SkColorSpace::Equals(z30.get(), upperRight.get()));
     REPORTER_ASSERT(r, !SkColorSpace::Equals(upperRight.get(), adobe.get()));
 }
+
+// This test is only interesting on 32-bit builds.
+#if defined(SK_CPU_X86) || defined(SK_CPU_ARM32)

[mtklein, 2016/08/16 23:54:06]

Do we really need to not have the test exist?  Can't we just have it here and
not run it?

[msarett, 2016/08/17 12:40:45]

Sure, sgtm

+
+DEF_TEST(ColorSpaceCRBug, r) {
+    sk_sp<SkData> data = SkData::MakeFromFileName(
+            GetResourcePath("invalid_images/crbug636268.png").c_str());
+    SkAutoTDelete<SkCodec> codec(SkCodec::NewFromData(data.get()));
+    REPORTER_ASSERT(r, codec.get());
+}
+
+#endif