Refactor: clean up some unused or mostly-unused API I saw here.

tests/StreamTest.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 "Test.h"
 #include "SkRandom.h"
 #include "SkOSFile.h"
@@ skipping 26 matching lines @@
 static void test_filestreams(skiatest::Reporter* reporter, const char* tmpDir) {
     SkString path = SkOSPath::SkPathJoin(tmpDir, "wstream_test");
 
     const char s[] = "abcdefghijklmnopqrstuvwxyz";
 
     {
         SkFILEWStream writer(path.c_str());
         if (!writer.isValid()) {
             SkString msg;
             msg.printf("Failed to create tmp file %s\n", path.c_str());
-            reporter->reportFailed(msg.c_str());
+            reporter->reportFailed(msg);
             return;
         }
 
         for (int i = 0; i < 100; ++i) {
             writer.write(s, 26);
         }
     }
 
     {
         SkFILEStream stream(path.c_str());
@@ skipping 95 matching lines @@
 }
 
 static void TestStreams(skiatest::Reporter* reporter) {
     TestWStream(reporter);
     TestPackedUInt(reporter);
     TestNullData();
 }
 
 #include "TestClassDef.h"
 DEFINE_TESTCLASS("Stream", StreamTestClass, TestStreams)