add explicit filepaths to render_pictures JSON summary

tools/skimage_main.cpp

 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "gm_expectations.h"
 #include "SkBitmap.h"
 #include "SkColorPriv.h"
@@ skipping 446 matching lines @@
         return;
     }
     if (!lengthLessDigest.equals(digest)) {
         gDecodeFailures.push_back().appendf("Without using getLength, %s did not match digest "
                                             "that uses getLength\n", srcPath);
     }
 }
 #endif // defined(SK_BUILD_FOR_ANDROID) || defined(SK_BUILD_FOR_UNIX)
 
 /**
- *  Replace all instances of oldChar with newChar in str.
- *  TODO: Add this function to SkString and write tests for it.
+ * Replaces all instances of oldChar with newChar in str.
+ *
+ * TODO: This function appears here and in picture_utils.[cpp|h] ;
+ * we should add the implementation to src/core/SkString.cpp, write tests for it,
+ * and remove it from elsewhere.
  */
 static void replace_char(SkString* str, const char oldChar, const char newChar) {
     if (NULL == str) {
         return;
     }
     for (size_t i = 0; i < str->size(); ++i) {
         if (oldChar == str->operator[](i)) {
             str->operator[](i) = newChar;
         }
     }
@@ skipping 352 matching lines @@
     print_strings("Known failures", gKnownFailures);
 
     return failed ? -1 : 0;
 }
 
 #if !defined SK_BUILD_FOR_IOS
 int main(int argc, char * const argv[]) {
     return tool_main(argc, (char**) argv);
 }
 #endif