diff options
author | teravest@chromium.org <teravest@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-15 11:06:23 +0000 |
---|---|---|
committer | teravest@chromium.org <teravest@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-15 11:06:23 +0000 |
commit | 599aeffdcddcd73fc4dff854631016438fd922c2 (patch) | |
tree | 94d4f0b041368f7af5e92e7a0c2720274e910481 /ppapi/tests/test_udp_socket_private_disallowed.cc | |
parent | 9ffbb3ffc9220cd90e9aeaeb901963e1ba4922cc (diff) | |
download | chromium_src-599aeffdcddcd73fc4dff854631016438fd922c2.zip chromium_src-599aeffdcddcd73fc4dff854631016438fd922c2.tar.gz chromium_src-599aeffdcddcd73fc4dff854631016438fd922c2.tar.bz2 |
Cleanup: Remove operator from TestCompletionCallback.
This change removes TestCompletionCallback::operator pp::CompletionCallback(),
which made test code confusing to read, and allowed some strange static_cast<>
hacks in tests.
I tested this change by building ppapi_tests, browser_tests, and chrome.
BUG=
Review URL: https://chromiumcodereview.appspot.com/12824010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@188334 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi/tests/test_udp_socket_private_disallowed.cc')
-rw-r--r-- | ppapi/tests/test_udp_socket_private_disallowed.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ppapi/tests/test_udp_socket_private_disallowed.cc b/ppapi/tests/test_udp_socket_private_disallowed.cc index fa0c179..655ed35 100644 --- a/ppapi/tests/test_udp_socket_private_disallowed.cc +++ b/ppapi/tests/test_udp_socket_private_disallowed.cc @@ -37,7 +37,7 @@ std::string TestUDPSocketPrivateDisallowed::TestBind() { TestCompletionCallback callback(instance_->pp_instance()); int32_t rv = udp_socket_private_interface_->Bind(socket, &addr, - static_cast<pp::CompletionCallback>(callback).pp_completion_callback()); + callback.GetCallback().pp_completion_callback()); if (PP_OK_COMPLETIONPENDING == rv) rv = callback.WaitForResult(); |