Fix up all the easy virtual ... SK_OVERRIDE cases.

tests/Test.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 "SkCommandLineFlags.h"
@@ skipping 48 matching lines @@
 }
 
 class LocalReporter : public Reporter {
 public:
     explicit LocalReporter(Reporter* reporterToMimic) : fReporter(reporterToMimic) {}
 
     int numFailures() const { return fFailures.count(); }
     const skiatest::Failure& failure(int i) const { return fFailures[i]; }
 
 protected:
-    virtual void onReportFailed(const Failure& failure) SK_OVERRIDE {
+    void onReportFailed(const Failure& failure) SK_OVERRIDE {
         fFailures.push_back(failure);
     }
 
     // Proxy down to fReporter.  We assume these calls are threadsafe.
-    virtual bool allowExtendedTest() const SK_OVERRIDE {
+    bool allowExtendedTest() const SK_OVERRIDE {
         return fReporter->allowExtendedTest();
     }
 
-    virtual void bumpTestCount() SK_OVERRIDE {
+    void bumpTestCount() SK_OVERRIDE {
         fReporter->bumpTestCount();
     }
 
-    virtual bool verbose() const SK_OVERRIDE {
+    bool verbose() const SK_OVERRIDE {
         return fReporter->verbose();
     }
 
 private:
     Reporter* fReporter;  // Unowned.
     SkTArray<skiatest::Failure> fFailures;
 };
 
 void Test::run() {
     // Clear the Skia error callback before running any test, to ensure that tests
@@ skipping 16 matching lines @@
       fReporter->reportFailed(local.failure(i));
     }
     fReporter->endTest(this);
 
 }
 
 SkString Test::GetTmpDir() {
     const char* tmpDir = FLAGS_tmpDir.isEmpty() ? NULL : FLAGS_tmpDir[0];
     return SkString(tmpDir);
 }