GM: remove deprecated --ignoreTests flag

gm/gmmain.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.
  */
 
 /*
  * Code for the "gm" (Golden Master) rendering comparison tool.
  *
@@ skipping 1407 matching lines @@
               "use the default. 0 for either disables the cache.");
 #endif
 DEFINE_bool(hierarchy, false, "Whether to use multilevel directory structure "
             "when reading/writing files.");
 DEFINE_string(ignoreErrorTypes, kDefaultIgnorableErrorTypes.asString(" ").c_str(),
               "Space-separated list of ErrorTypes that should be ignored. If any *other* error "
               "types are encountered, the tool will exit with a nonzero return value.");
 DEFINE_string(ignoreFailuresFile, "", "Path to file containing a list of tests for which we "
               "should ignore failures.\n"
               "The file should list one test per line, except for comment lines starting with #");
-DEFINE_string(ignoreTests, "", "Space delimited list of tests for which we should ignore "
-              "failures.\n"
-              "DEPRECATED in favor of --ignoreFailuresFile; see bug 1730");
 DEFINE_string(match, "", "[~][^]substring[$] [...] of test name to run.\n"
               "Multiple matches may be separated by spaces.\n"
               "~ causes a matching test to always be skipped\n"
               "^ requires the start of the test to match\n"
               "$ requires the end of the test to match\n"
               "^ and $ requires an exact match\n"
               "If a test does not match any list entry,\n"
               "it is skipped unless some list entry starts with ~");
 DEFINE_string(missingExpectationsPath, "", "Write images for tests without expectations "
               "into this directory.");
@@ skipping 614 matching lines @@
     return true;
 }
 
 /**
  * Replace contents of ignoreTestSubstrings with a list of testname/config substrings, indicating
  * which tests' failures should be ignored.
  */
 static bool parse_flags_ignore_tests(SkTArray<SkString> &ignoreTestSubstrings) {
     ignoreTestSubstrings.reset();
 
-    // Parse --ignoreTests
-    if (FLAGS_ignoreTests.count() > 0) {
-        gm_fprintf(stderr, "you are using deprecated --ignoreTests flag\n");
-        for (int i = 0; i < FLAGS_ignoreTests.count(); i++) {
-            ignoreTestSubstrings.push_back(SkString(FLAGS_ignoreTests[i]));
-        }
-    }
-
     // Parse --ignoreFailuresFile
     for (int i = 0; i < FLAGS_ignoreFailuresFile.count(); i++) {
         SkTArray<SkString> linesFromFile;
         if (!read_lines_from_file(FLAGS_ignoreFailuresFile[i], linesFromFile)) {
             return false;
         } else {
             for (int j = 0; j < linesFromFile.count(); j++) {
                 SkString thisLine = linesFromFile[j];
                 if (thisLine.isEmpty() || thisLine.startsWith('#')) {
                     // skip this line
@@ skipping 303 matching lines @@
     if (FLAGS_forceBWtext) {
         canvas->setDrawFilter(SkNEW(BWTextDrawFilter))->unref();
     }
 }
 
 #if !defined(SK_BUILD_FOR_IOS) && !defined(SK_BUILD_FOR_NACL)
 int main(int argc, char * const argv[]) {
     return tool_main(argc, (char**) argv);
 }
 #endif