Chromium Code Reviews| Index: webrtc/modules/utility/source/process_thread_impl_unittest.cc |
| diff --git a/webrtc/modules/utility/source/process_thread_impl_unittest.cc b/webrtc/modules/utility/source/process_thread_impl_unittest.cc |
| index 5b31870ac41438739105356ff49c33a72e5d5933..881a1ea9e8bdd2bb56499630f19c74b7eb0180d9 100644 |
| --- a/webrtc/modules/utility/source/process_thread_impl_unittest.cc |
| +++ b/webrtc/modules/utility/source/process_thread_impl_unittest.cc |
| @@ -13,6 +13,7 @@ |
| #include "testing/gmock/include/gmock/gmock.h" |
| #include "testing/gtest/include/gtest/gtest.h" |
| +#include "webrtc/base/task_queue.h" |
| #include "webrtc/base/timeutils.h" |
| #include "webrtc/modules/include/module.h" |
| #include "webrtc/modules/utility/source/process_thread_impl.h" |
| @@ -33,10 +34,10 @@ class MockModule : public Module { |
| MOCK_METHOD1(ProcessThreadAttached, void(ProcessThread*)); |
| }; |
| -class RaiseEventTask : public ProcessTask { |
| +class RaiseEventTask : public rtc::QueuedTask { |
| public: |
| RaiseEventTask(EventWrapper* event) : event_(event) {} |
| - void Run() override { event_->Set(); } |
| + bool Run() override { event_->Set(); return true; } |
|
magjed_webrtc
2016/05/27 14:40:36
nit: Is it really ok to have multiple statements o
tommi
2016/05/28 12:15:54
Ah, no. Thanks for catching.
|
| private: |
| EventWrapper* event_; |