Fix "unreachable code" warnings (MSVC warning 4702) in ppapi/.

ppapi/tests/test_talk_private.cc

 // 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.
 
 #include "ppapi/tests/test_talk_private.h"
 
 #include <stdio.h>
 #include <string.h>
 #include <string>
 
@@ skipping 37 matching lines @@
 std::string TestTalkPrivate::TestGetPermission() {
   if (!talk_private_interface_1) {
     PASS();
   }
 
   if (!testing_interface_->IsOutOfProcess()) {
     // We only support out-of-process access to this API, so skip in-process
     PASS();
   }
 
-#if defined(USE_ASH)
   // Under Ash, this will prompt the user so the test cannot run in an automated
-  // fashion. To manually test under Ash, comment this out.
-  PASS();
-#endif
-
+  // fashion. To manually test under Ash, replace "!defined(USE_ASH)" with 1.
+#if !defined(USE_ASH)
   PP_Resource talk_resource = talk_private_interface_1->Create(
       instance_->pp_instance());
 
   TestCompletionCallback callback(instance_->pp_instance(), callback_type());
   callback.WaitForResult(talk_private_interface_1->GetPermission(talk_resource,
       callback.GetCallback().pp_completion_callback()));
   CHECK_CALLBACK_BEHAVIOR(callback);
 
 #if defined(USE_ASH)
   // Under Ash, this test will actually prompt the user and return either true
   // or false depending on their choice.
   if (callback.result() != 0 && callback.result() != 1)
     return "Unexpected result";
 #else
   // Currently not implemented without Ash, bur always returns false.
   if (callback.result() != 0)
     return "Unexpected non-zero result";
 #endif
-
+#endif
   PASS();
 }