Fix PPAPI TestCase so individual tests can be run more than once.

ppapi/tests/test_case.h

 // Copyright (c) 2012 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.
 
 #ifndef PPAPI_TESTS_TEST_CASE_H_
 #define PPAPI_TESTS_TEST_CASE_H_
 
 #include <cmath>
 #include <limits>
 #include <map>
@@ skipping 206 matching lines @@
 
   static void DoQuitMainMessageLoop(void* pp_instance, int32_t result);
 
   // Passed when creating completion callbacks in some tests. This determines
   // what kind of callback we use for the test.
   CallbackType callback_type_;
 
   // Var ids that should be ignored when checking for leaks on shutdown.
   std::set<int64_t> ignored_leaked_vars_;
 
-  // The tests that were found in test_filter but have not yet been run. The
-  // bool indicates whether the test should be run (i.e., it will be false if
-  // the test name was prefixed in the test_filter string).
+  // The tests that were found in test_filter. The bool indicates whether the
+  // test should be run (i.e., it will be false if the test name was prefixed in
+  // the test_filter string).
   //
-  // This is initialized lazily the first time that ShouldRunTest is called by
-  // RunTests. When RunTests is finished, this should be empty. Any remaining
-  // tests are tests that were listed in the test_filter but didn't match
-  // any calls to ShouldRunTest, meaning it was probably a typo. TestingInstance
-  // should log this and consider it a failure.
+  // This is initialized lazily the first time that ShouldRunTest is called.
+  std::map<std::string, bool> filter_tests_;
+  // Flag indicating whether we have populated filter_tests_ yet.
+  bool have_populated_filter_tests_;
+  // This is initialized with the contents of filter_tests_. As each test is
+  // run, it is removed from remaining_tests_. When RunTests is finished,
+  // remaining_tests_ should be empty. Any remaining tests are tests that were
+  // listed in the test_filter but didn't match any calls to ShouldRunTest,
+  // meaning it was probably a typo. TestingInstance should log this and
+  // consider it a failure.
   std::map<std::string, bool> remaining_tests_;
-  // Flag indicating whether we have populated remaining_tests_ yet.
-  bool have_populated_remaining_tests_;
 
   // If ShouldRunTest is called but the given test name doesn't match anything
   // in the test_filter, the test name will be added here. This allows
   // TestingInstance to detect when not all tests were listed.
   std::set<std::string> skipped_tests_;
 
 #if !(defined __native_client__)
   // Holds the test object, if any was retrieved from CreateTestObject.
   pp::VarPrivate test_object_;
 #endif
@@ skipping 135 matching lines @@
 #define ASSERT_SUBTEST_SUCCESS(function) \
   do { \
     std::string result = (function); \
     if (!result.empty()) \
       return result; \
   } while (false)
 
 #define PASS() return std::string()
 
 #endif  // PPAPI_TESTS_TEST_CASE_H_