diff options
author | skyostil <skyostil@chromium.org> | 2015-06-05 08:26:08 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-06-05 15:26:51 +0000 |
commit | 74f13dc2b62abb34e947093fe2600f9c539e60ef (patch) | |
tree | c298260fa8c56717943ff45efda601f312cc74c4 /content/browser/gamepad | |
parent | a95af720637c609f33ed76d89d658041f354a222 (diff) | |
download | chromium_src-74f13dc2b62abb34e947093fe2600f9c539e60ef.zip chromium_src-74f13dc2b62abb34e947093fe2600f9c539e60ef.tar.gz chromium_src-74f13dc2b62abb34e947093fe2600f9c539e60ef.tar.bz2 |
Revert "content: Remove use of MessageLoopProxy and deprecated MessageLoop APIs"
This reverts commit 5d18b2493af36d77d3c491854e3d56f46f5f747c.
Reason: Broke iOS build: http://build.chromium.org/p/chromium.mac/builders/iOS_Simulator_%28dbg%29/builds/27843
NOTRY=true
NOTREECHECKS=true
BUG=465354
TBR=sievers@chromium.org,nick@chromium.org
Review URL: https://codereview.chromium.org/1153503009
Cr-Commit-Position: refs/heads/master@{#333047}
Diffstat (limited to 'content/browser/gamepad')
-rw-r--r-- | content/browser/gamepad/gamepad_provider.cc | 37 | ||||
-rw-r--r-- | content/browser/gamepad/gamepad_provider.h | 7 |
2 files changed, 24 insertions, 20 deletions
diff --git a/content/browser/gamepad/gamepad_provider.cc b/content/browser/gamepad/gamepad_provider.cc index eef9001..e16339c 100644 --- a/content/browser/gamepad/gamepad_provider.cc +++ b/content/browser/gamepad/gamepad_provider.cc @@ -7,11 +7,10 @@ #include <vector> #include "base/bind.h" -#include "base/location.h" #include "base/logging.h" -#include "base/single_thread_task_runner.h" +#include "base/message_loop/message_loop.h" +#include "base/message_loop/message_loop_proxy.h" #include "base/third_party/dynamic_annotations/dynamic_annotations.h" -#include "base/thread_task_runner_handle.h" #include "base/threading/thread.h" #include "base/threading/thread_restrictions.h" #include "content/browser/gamepad/gamepad_data_fetcher.h" @@ -30,8 +29,9 @@ namespace content { GamepadProvider::ClosureAndThread::ClosureAndThread( const base::Closure& c, - const scoped_refptr<base::SingleThreadTaskRunner>& m) - : closure(c), task_runner(m) { + const scoped_refptr<base::MessageLoopProxy>& m) + : closure(c), + message_loop(m) { } GamepadProvider::ClosureAndThread::~ClosureAndThread() { @@ -83,7 +83,7 @@ void GamepadProvider::Pause() { is_paused_ = true; } base::MessageLoop* polling_loop = polling_thread_->message_loop(); - polling_loop->task_runner()->PostTask( + polling_loop->PostTask( FROM_HERE, base::Bind(&GamepadProvider::SendPauseHint, Unretained(this), true)); } @@ -97,18 +97,18 @@ void GamepadProvider::Resume() { } base::MessageLoop* polling_loop = polling_thread_->message_loop(); - polling_loop->task_runner()->PostTask( + polling_loop->PostTask( FROM_HERE, base::Bind(&GamepadProvider::SendPauseHint, Unretained(this), false)); - polling_loop->task_runner()->PostTask( + polling_loop->PostTask( FROM_HERE, base::Bind(&GamepadProvider::ScheduleDoPoll, Unretained(this))); } void GamepadProvider::RegisterForUserGesture(const base::Closure& closure) { base::AutoLock lock(user_gesture_lock_); - user_gesture_observers_.push_back( - ClosureAndThread(closure, base::MessageLoop::current()->task_runner())); + user_gesture_observers_.push_back(ClosureAndThread( + closure, base::MessageLoop::current()->message_loop_proxy())); } void GamepadProvider::OnDevicesChanged(base::SystemMonitor::DeviceType type) { @@ -144,9 +144,11 @@ void GamepadProvider::Initialize(scoped_ptr<GamepadDataFetcher> fetcher) { #endif polling_thread_->StartWithOptions(base::Thread::Options(kMessageLoopType, 0)); - polling_thread_->task_runner()->PostTask( - FROM_HERE, base::Bind(&GamepadProvider::DoInitializePollingThread, - base::Unretained(this), base::Passed(&fetcher))); + polling_thread_->message_loop()->PostTask( + FROM_HERE, + base::Bind(&GamepadProvider::DoInitializePollingThread, + base::Unretained(this), + base::Passed(&fetcher))); } void GamepadProvider::DoInitializePollingThread( @@ -262,8 +264,9 @@ void GamepadProvider::ScheduleDoPoll() { return; } - base::ThreadTaskRunnerHandle::Get()->PostDelayedTask( - FROM_HERE, base::Bind(&GamepadProvider::DoPoll, Unretained(this)), + base::MessageLoop::current()->PostDelayedTask( + FROM_HERE, + base::Bind(&GamepadProvider::DoPoll, Unretained(this)), base::TimeDelta::FromMilliseconds(kDesiredSamplingIntervalMs)); have_scheduled_do_poll_ = true; } @@ -310,8 +313,8 @@ void GamepadProvider::CheckForUserGesture() { if (GamepadsHaveUserGesture(pads)) { ever_had_user_gesture_ = true; for (size_t i = 0; i < user_gesture_observers_.size(); i++) { - user_gesture_observers_[i].task_runner->PostTask( - FROM_HERE, user_gesture_observers_[i].closure); + user_gesture_observers_[i].message_loop->PostTask(FROM_HERE, + user_gesture_observers_[i].closure); } user_gesture_observers_.clear(); } diff --git a/content/browser/gamepad/gamepad_provider.h b/content/browser/gamepad/gamepad_provider.h index ce9c11a..a81061a 100644 --- a/content/browser/gamepad/gamepad_provider.h +++ b/content/browser/gamepad/gamepad_provider.h @@ -13,13 +13,14 @@ #include "base/memory/scoped_ptr.h" #include "base/memory/shared_memory.h" #include "base/memory/weak_ptr.h" +#include "base/message_loop/message_loop_proxy.h" #include "base/synchronization/lock.h" #include "base/system_monitor/system_monitor.h" #include "content/common/content_export.h" #include "third_party/WebKit/public/platform/WebGamepads.h" namespace base { -class SingleThreadTaskRunner; +class MessageLoopProxy; class Thread; } @@ -104,11 +105,11 @@ class CONTENT_EXPORT GamepadProvider : base::Lock user_gesture_lock_; struct ClosureAndThread { ClosureAndThread(const base::Closure& c, - const scoped_refptr<base::SingleThreadTaskRunner>& m); + const scoped_refptr<base::MessageLoopProxy>& m); ~ClosureAndThread(); base::Closure closure; - scoped_refptr<base::SingleThreadTaskRunner> task_runner; + scoped_refptr<base::MessageLoopProxy> message_loop; }; typedef std::vector<ClosureAndThread> UserGestureObserverVector; UserGestureObserverVector user_gesture_observers_; |