Add means for running some tests only o-o-p, add messaging o-o-p test.

ppapi/tests/test_case.h

Index: ppapi/tests/test_case.h
diff --git a/ppapi/tests/test_case.h b/ppapi/tests/test_case.h
index 0a3cdc65a6e72c2c220cbc2340a37899f5d091ec..06ebcb1ccd1529511bded3900b8d0a536da66543 100644
--- a/ppapi/tests/test_case.h
+++ b/ppapi/tests/test_case.h
@@ -130,18 +130,24 @@ class TestCaseFactory {
 // RunTest function. This assumes the function name is TestFoo where Foo is the
 // test |name|.
 #define RUN_TEST(name) \
-  force_async_ = false; \
-  instance_->LogTest(#name, Test##name());
+  do { \
+    force_async_ = false; \
+    instance_->LogTest(#name, Test##name()); \
+  } while (false)
 
 // Like RUN_TEST above but forces functions taking callbacks to complete
 // asynchronously on success or error.
 #define RUN_TEST_FORCEASYNC(name) \
-  force_async_ = true; \
-  instance_->LogTest(#name"ForceAsync", Test##name());
+  do { \
+    force_async_ = true; \
+    instance_->LogTest(#name"ForceAsync", Test##name()); \
+  } while (false)
 
 #define RUN_TEST_FORCEASYNC_AND_NOT(name) \
-  RUN_TEST_FORCEASYNC(name); \
-  RUN_TEST(name);
+  do { \
+    RUN_TEST_FORCEASYNC(name); \
+    RUN_TEST(name); \
+  } while (false)

[dmichael (off chromium), 2011/08/16 19:42:26]

This is just so you can safely do stuff like:
if (blah)
  RUN_TEST(blab);
(I could have used curly braces, but wanted to foolproof it)

[polina, 2011/08/18 20:02:33]

it's better this way
we use this approach in nacl_macros.h, etc.
thanks for making the improvement

 
 
 // Helper macros for checking values in tests, and returning a location