chromium/src
0
Fork 0
Code Activity

Change MessagePumpLibevent::Run to process I/O events before doing idle

Browse Source 
work.

R=jar@chromium.org,mark@chromium.org,rvargas@chromium.org
BUG=87707
TEST=covered by existing unit tests

Review URL: http://codereview.chromium.org/7291010

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91252 0039d316-1c4b-4281-b951-d872f2087c98
This commit is contained in:
wtc@chromium.org
2011-07-01 01:55:29 +00:00
parent da08956f0d
commit 4b4a94f22b
2 changed files with 12 additions and 0 deletions

9
base/message_pump_libevent.cc

@ -107,6 +107,7 @@ void MessagePumpLibevent::FileDescriptorWatcher::OnFileCanWriteWithoutBlocking(
MessagePumpLibevent::MessagePumpLibevent()
: keep_running_(true),
in_run_(false),
processed_io_events_(false),
event_base_(event_base_new()),
wakeup_pipe_in_(-1),
wakeup_pipe_out_(-1) {
@ -226,6 +227,12 @@ void MessagePumpLibevent::Run(Delegate* delegate) {
if (!keep_running_)
break;
event_base_loop(event_base_, EVLOOP_NONBLOCK);
did_work |= processed_io_events_;
processed_io_events_ = false;
if (!keep_running_)
break;
did_work |= delegate->DoDelayedWork(&delayed_work_time_);
if (!keep_running_)
break;
@ -331,6 +338,7 @@ void MessagePumpLibevent::OnLibeventNotification(int fd, short flags,
static_cast<FileDescriptorWatcher*>(context);
MessagePumpLibevent* pump = controller->pump();
pump->processed_io_events_ = true;
if (flags & EV_WRITE) {
controller->OnFileCanWriteWithoutBlocking(fd, pump);
@ -351,6 +359,7 @@ void MessagePumpLibevent::OnWakeup(int socket, short flags, void* context) {
char buf;
int nread = HANDLE_EINTR(read(socket, &buf, 1));
DCHECK_EQ(nread, 1);
that->processed_io_events_ = true;
// Tell libevent to break out of inner loop.
event_base_loopbreak(that->event_base_);
}

3
base/message_pump_libevent.h

@ -144,6 +144,9 @@ class BASE_API MessagePumpLibevent : public MessagePump {
// This flag is set when inside Run.
bool in_run_;
// This flag is set if libevent has processed I/O events.
bool processed_io_events_;
// The time at which we should call DoDelayedWork.
TimeTicks delayed_work_time_;