Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(268)

Unified Diff: ppapi/tests/test_flash_message_loop.cc

Issue 9651002: Add C++ wrappers for output parameters. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: With swapping Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ppapi/tests/test_flash_message_loop.cc
diff --git a/ppapi/tests/test_flash_message_loop.cc b/ppapi/tests/test_flash_message_loop.cc
index 5143fe78d170f09743845d964b37e87555e3e30b..e5b7fe8f8f29cf7f99841db761863cdd57fb001f 100644
--- a/ppapi/tests/test_flash_message_loop.cc
+++ b/ppapi/tests/test_flash_message_loop.cc
@@ -31,9 +31,8 @@ void TestFlashMessageLoop::RunTests(const std::string& filter) {
std::string TestFlashMessageLoop::TestBasics() {
message_loop_ = new pp::flash::MessageLoop(instance_);
- pp::CompletionCallback callback =
- callback_factory_.NewRequiredCallback(
- &TestFlashMessageLoop::QuitMessageLoopTask);
+ pp::CompletionCallback callback = callback_factory_.NewCallback(
+ &TestFlashMessageLoop::QuitMessageLoopTask);
pp::Module::Get()->core()->CallOnMainThread(0, callback);
int32_t result = message_loop_->Run();
@@ -48,9 +47,8 @@ std::string TestFlashMessageLoop::TestBasics() {
std::string TestFlashMessageLoop::TestRunWithoutQuit() {
message_loop_ = new pp::flash::MessageLoop(instance_);
- pp::CompletionCallback callback =
- callback_factory_.NewRequiredCallback(
- &TestFlashMessageLoop::DestroyMessageLoopResourceTask);
+ pp::CompletionCallback callback = callback_factory_.NewCallback(
+ &TestFlashMessageLoop::DestroyMessageLoopResourceTask);
pp::Module::Get()->core()->CallOnMainThread(0, callback);
int32_t result = message_loop_->Run();

Powered by Google App Engine
This is Rietveld 408576698