WebSocket Pepper API: allow to release in completion callbacks

ppapi/tests/test_websocket.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 PAPPI_TESTS_TEST_WEBSOCKET_H_
 #define PAPPI_TESTS_TEST_WEBSOCKET_H_
 
 #include <string>
 
 #include "ppapi/c/ppb_core.h"
 #include "ppapi/c/ppb_var.h"
 #include "ppapi/c/ppb_var_array_buffer.h"
 #include "ppapi/c/ppb_websocket.h"
 #include "ppapi/tests/test_case.h"
+#include "ppapi/tests/test_utils.h"
 
-class TestWebSocket : public TestCase {
+class TestWebSocket
+    : public TestCase,
+      public TestCompletionCallback::Delegate {

[dmichael (off chromium), 2012/06/25 15:50:51]

I think it would be cleaner to define a separate class to implement the
Delegate.

[Takashi Toyoshima, 2012/06/26 09:19:10]

OK, I will introduce class ReleaseResourceDelegate in test_websocket.cc.

  public:
   explicit TestWebSocket(TestingInstance* instance) : TestCase(instance) {}
 
+  // TestCompletionCallback::Delegate interfaces.

[dmichael (off chromium), 2012/06/25 15:50:51]

interfaces->implementation

[Takashi Toyoshima, 2012/06/26 09:19:10]

Done.

+  virtual void OnCallback(void* user_data, int32_t result);
+
  private:
   // TestCase implementation.
   virtual bool Init();
   virtual void RunTests(const std::string& filter);
 
   std::string GetFullURL(const char* url);
   PP_Var CreateVarString(const std::string& string);
   PP_Var CreateVarBinary(const std::vector<uint8_t>& binary);
   void ReleaseVar(const PP_Var& var);
   bool AreEqualWithString(const PP_Var& var, const std::string& string);
@@ skipping 31 matching lines @@
   std::string TestUtilityTextSendReceive();
   std::string TestUtilityBinarySendReceive();
   std::string TestUtilityBufferedAmount();
 
   // Keeps Pepper API interfaces. These are used by the tests that access the C
   // API directly.
   const PPB_WebSocket* websocket_interface_;
   const PPB_Var* var_interface_;
   const PPB_VarArrayBuffer* arraybuffer_interface_;
   const PPB_Core* core_interface_;
+
+  // This resource will be released in |OnCallback|.
+  PP_Resource resource_;
 };
 
 #endif  // PAPPI_TESTS_TEST_WEBSOCKET_H_