diff options
| author | joth@chromium.org <joth@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-07 16:37:34 +0000 |
|---|---|---|
| committer | joth@chromium.org <joth@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-07 16:37:34 +0000 |
| commit | 1f7a8052898f2163b152ef39802b43b5af4b4e0c (patch) | |
| tree | f15cdd3f60b90664c225b237717c38381b27dd59 /base/message_loop_proxy.h | |
| parent | 7a2f8f90696c8301bcf3bbb5047f614d8c9c4b70 (diff) | |
| download | chromium_src-1f7a8052898f2163b152ef39802b43b5af4b4e0c.zip chromium_src-1f7a8052898f2163b152ef39802b43b5af4b4e0c.tar.gz chromium_src-1f7a8052898f2163b152ef39802b43b5af4b4e0c.tar.bz2 | |
Clarify documentation about when a task may fail to post to a message loop
BUG=None
TEST=still builds...
Review URL: http://codereview.chromium.org/6812016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80794 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/message_loop_proxy.h')
| -rw-r--r-- | base/message_loop_proxy.h | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/base/message_loop_proxy.h b/base/message_loop_proxy.h index e750f2a..07bca64 100644 --- a/base/message_loop_proxy.h +++ b/base/message_loop_proxy.h @@ -27,8 +27,10 @@ class BASE_API MessageLoopProxy // either post the Task to the MessageLoop (if it's still alive), or to // delete the Task otherwise. // They return true iff the thread existed and the task was posted. Note that - // even if the task is posted, there's no guarantee that it will run, since - // the target thread may already have a Quit message in its queue. + // even if the task is posted, there's no guarantee that it will run; for + // example the target loop may already be quitting, or in the case of a + // delayed task a Quit message may preempt it in the message loop queue. + // Conversely, a return value of false is a guarantee the task will not run. virtual bool PostTask(const tracked_objects::Location& from_here, Task* task) = 0; virtual bool PostDelayedTask(const tracked_objects::Location& from_here, |