diff options
author | adamk@chromium.org <adamk@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-05 20:30:11 +0000 |
---|---|---|
committer | adamk@chromium.org <adamk@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-05 20:30:11 +0000 |
commit | e45f526c066073bfd63c115d5b843333c8d1a2cd (patch) | |
tree | 0986f2d415158baa6675850379c48ff788a6791a /webkit/glue | |
parent | 8fd673a1f1baf1aac390b2b635e4636ed357886c (diff) | |
download | chromium_src-e45f526c066073bfd63c115d5b843333c8d1a2cd.zip chromium_src-e45f526c066073bfd63c115d5b843333c8d1a2cd.tar.gz chromium_src-e45f526c066073bfd63c115d5b843333c8d1a2cd.tar.bz2 |
Implement WebThread::{add,remove}TaskObserver.
Depends on http://trac.webkit.org/changeset/101418
Review URL: http://codereview.chromium.org/8586038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113010 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/glue')
-rw-r--r-- | webkit/glue/webthread_impl.cc | 45 | ||||
-rw-r--r-- | webkit/glue/webthread_impl.h | 30 |
2 files changed, 71 insertions, 4 deletions
diff --git a/webkit/glue/webthread_impl.cc b/webkit/glue/webthread_impl.cc index 236497e..1354bcb 100644 --- a/webkit/glue/webthread_impl.cc +++ b/webkit/glue/webthread_impl.cc @@ -10,9 +10,45 @@ #include "base/bind.h" #include "base/bind_helpers.h" #include "base/message_loop.h" +#include "base/threading/platform_thread.h" namespace webkit_glue { +class WebThreadBase::TaskObserverAdapter : public MessageLoop::TaskObserver { +public: + TaskObserverAdapter(WebThread::TaskObserver* observer) + : observer_(observer) { } + + // WebThread::TaskObserver does not have a willProcessTask method. + virtual void WillProcessTask(base::TimeTicks) OVERRIDE { } + + virtual void DidProcessTask(base::TimeTicks) OVERRIDE { + observer_->didProcessTask(); + } + +private: + WebThread::TaskObserver* observer_; +}; + +void WebThreadBase::addTaskObserver(TaskObserver* observer) { + CHECK(IsCurrentThread()); + std::pair<TaskObserverMap::iterator, bool> result = task_observer_map_.insert( + std::make_pair(observer, static_cast<TaskObserverAdapter*>(NULL))); + if (result.second) + result.first->second = new TaskObserverAdapter(observer); + MessageLoop::current()->AddTaskObserver(result.first->second); +} + +void WebThreadBase::removeTaskObserver(TaskObserver* observer) { + CHECK(IsCurrentThread()); + TaskObserverMap::iterator iter = task_observer_map_.find(observer); + if (iter == task_observer_map_.end()) + return; + MessageLoop::current()->RemoveTaskObserver(iter->second); + delete iter->second; + task_observer_map_.erase(iter); +} + WebThreadImpl::WebThreadImpl(const char* name) : thread_(new base::Thread(name)) { thread_->Start(); @@ -22,6 +58,7 @@ void WebThreadImpl::postTask(Task* task) { thread_->message_loop()->PostTask( FROM_HERE, base::Bind(&WebKit::WebThread::Task::run, base::Owned(task))); } + void WebThreadImpl::postDelayedTask( Task* task, long long delay_ms) { thread_->message_loop()->PostDelayedTask( @@ -30,6 +67,10 @@ void WebThreadImpl::postDelayedTask( delay_ms); } +bool WebThreadImpl::IsCurrentThread() const { + return thread_->thread_id() == base::PlatformThread::CurrentId(); +} + WebThreadImpl::~WebThreadImpl() { thread_->Stop(); } @@ -52,6 +93,10 @@ void WebThreadImplForMessageLoop::postDelayedTask( delay_ms); } +bool WebThreadImplForMessageLoop::IsCurrentThread() const { + return message_loop_->BelongsToCurrentThread(); +} + WebThreadImplForMessageLoop::~WebThreadImplForMessageLoop() { } diff --git a/webkit/glue/webthread_impl.h b/webkit/glue/webthread_impl.h index 2112536..a26de79 100644 --- a/webkit/glue/webthread_impl.h +++ b/webkit/glue/webthread_impl.h @@ -4,6 +4,8 @@ #ifndef WEBKIT_GLUE_WEBTHREAD_IMPL_H_ #define WEBKIT_GLUE_WEBTHREAD_IMPL_H_ +#include <map> + #include "base/threading/thread.h" #include "base/memory/scoped_ptr.h" #include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebThread.h" @@ -11,9 +13,23 @@ namespace webkit_glue { -class WebThreadImpl : public WebKit::WebThread { +class WebThreadBase : public WebKit::WebThread { public: - WEBKIT_GLUE_EXPORT WebThreadImpl(const char* name); + virtual void addTaskObserver(TaskObserver* observer); + virtual void removeTaskObserver(TaskObserver* observer); + + protected: + virtual bool IsCurrentThread() const = 0; + + private: + class TaskObserverAdapter; + typedef std::map<TaskObserver*, TaskObserverAdapter*> TaskObserverMap; + TaskObserverMap task_observer_map_; +}; + +class WebThreadImpl : public WebThreadBase { + public: + WEBKIT_GLUE_EXPORT explicit WebThreadImpl(const char* name); WEBKIT_GLUE_EXPORT virtual ~WebThreadImpl(); virtual void postTask(Task* task) OVERRIDE; @@ -22,18 +38,24 @@ class WebThreadImpl : public WebKit::WebThread { MessageLoop* message_loop() const { return thread_->message_loop(); } protected: + virtual bool IsCurrentThread() const; + + private: scoped_ptr<base::Thread> thread_; }; -class WebThreadImplForMessageLoop : public WebKit::WebThread { +class WebThreadImplForMessageLoop : public WebThreadBase { public: - WebThreadImplForMessageLoop(base::MessageLoopProxy* message_loop); + explicit WebThreadImplForMessageLoop(base::MessageLoopProxy* message_loop); virtual ~WebThreadImplForMessageLoop(); virtual void postTask(Task* task) OVERRIDE; virtual void postDelayedTask(Task* task, long long delay_ms) OVERRIDE; protected: + virtual bool IsCurrentThread() const; + + private: scoped_refptr<base::MessageLoopProxy> message_loop_; }; |