ThreadedTestRunner::Run contained a PP_DCHECK with a side effect. Some tests failed because of it.

This CL fixes the problem.

BUG=None

Review URL: https://chromiumcodereview.appspot.com/19790011

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213386 0039d316-1c4b-4281-b951-d872f2087c98

ppapi/tests/test_case.h

@@ -187,7 +187,9 @@ class TestCase {
 
    private:
     void Run() {
-      PP_DCHECK(PP_OK == loop_.AttachToCurrentThread());
+      int32_t result = loop_.AttachToCurrentThread();
+      static_cast<void>(result); // result is not used in the RELEASE build.
+      PP_DCHECK(PP_OK == result);
       result_ = (test_case_->*test_to_run_)();
       // Now give the loop a chance to clean up.
       loop_.PostQuit(true /* should_destroy */);