Fix how --gtest_filter interacts with PRE_ tests

base/test/launcher/test_launcher.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/test/launcher/test_launcher.h"
 
 #if defined(OS_POSIX)
 #include <fcntl.h>
 #endif
 
@@ skipping 787 matching lines @@
 
       const CommandLine* command_line = CommandLine::ForCurrentProcess();
       if (test_name.find("DISABLED") != std::string::npos) {
         results_tracker_.AddDisabledTest(test_name);
 
         // Skip disabled tests unless explicitly requested.
         if (!command_line->HasSwitch(kGTestRunDisabledTestsFlag))
           continue;
       }
 
+      if (!launcher_delegate_->ShouldRunTest(test_case, test_info))
+        continue;
+
       // Skip the test that doesn't match the filter (if given).
       if (!positive_test_filter_.empty()) {
         bool found = false;
         for (size_t k = 0; k < positive_test_filter_.size(); ++k) {
           if (MatchPattern(test_name, positive_test_filter_[k])) {
             found = true;
             break;
           }
         }
 
         if (!found)
           continue;
       }
       bool excluded = false;
       for (size_t k = 0; k < negative_test_filter_.size(); ++k) {
         if (MatchPattern(test_name, negative_test_filter_[k])) {
           excluded = true;
           break;
         }
       }
       if (excluded)
         continue;
 
-      if (!launcher_delegate_->ShouldRunTest(test_case, test_info))
-        continue;
-
       if (base::Hash(test_name) % total_shards_ !=
           static_cast<uint32>(shard_index_)) {
         continue;
       }
 
       test_names.push_back(test_name);
     }
   }
 
   test_started_count_ = launcher_delegate_->RunTests(this, test_names);
@@ skipping 250 matching lines @@
 
     g_live_processes.Get().erase(process_handle);
   }
 
   base::CloseProcessHandle(process_handle);
 
   return exit_code;
 }
 
 }  // namespace base