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

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

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

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=91252

[wtc, 2011-06-30 21:51:20]

This is the CL that you reviewed in
http://codereview.chromium.org/7262011/

It was reverted because it broke ipc_tests (some
IPCSyncChannel tests timed out), and perhaps some other
unit tests.

Patch Set 1 is the version that was reverted.  Patch Set 2
has the fix -- we need to report we did work when being woken
up by the internal pipe, so that we will loop back to call
DoWork().  Otherwise we fall through into event_base_loop()
and fail to run tasks that the internal pipe woke us up to
run.

[Mark Mentovai, 2011-06-30 22:07:16]

LGTM

http://codereview.chromium.org/7291010/diff/4001/base/message_pump_libevent.cc
File base/message_pump_libevent.cc (right):

http://codereview.chromium.org/7291010/diff/4001/base/message_pump_libevent.c...
base/message_pump_libevent.cc:233: if (!keep_running_)
I was wondering about this. I should have spoken up.

[rvargas (doing something else), 2011-06-30 22:11:38]

LGTM

[commit-bot: I haz the power, 2011-07-01 01:55:31]

Change committed as 91252