Change loop bounds to be correct (and much less confusing)

gm/recordopts.cpp

Index: gm/recordopts.cpp
diff --git a/gm/recordopts.cpp b/gm/recordopts.cpp
index 25a88f73e54defdb3b57ff46ae03f5b414652ded..4a5248d0eab557d2a74c850aaacc017893cffdb9 100644
--- a/gm/recordopts.cpp
+++ b/gm/recordopts.cpp
@@ -190,7 +190,7 @@ DEF_SIMPLE_GM(recordopts, canvas, (kTestRectSize+1)*2, (kTestRectSize+1)*15) {
 
     for (size_t i = 0; i < SK_ARRAY_COUNT(shapeColors); ++i) {
         shapeColor = shapeColors[i];
-        for (size_t j = 0; j < SK_ARRAY_COUNT(shapeColors); ++j) {
+        for (size_t j = 0; j < SK_ARRAY_COUNT(detectorInstallFuncs); ++j) {

[reed1, 2016/06/23 17:43:29]

can we use the fancy style (and be perhaps even clearer)?

e.g.
for (auto shapeColor : shapeColors) {
   for (auto installFunc : detectorInstallFuncs) {
      ...

             InstallDetectorFunc detectorInstallFunc = detectorInstallFuncs[j];
             for (size_t k = 0; k < SK_ARRAY_COUNT(funcs); ++k) {
                 TestVariantSequence drawTestSequence = funcs[k];