Couple of small cleanups to MessagePumpMojo.

Small set of cleanups to MessagePumpMojo.

These are:
1. Change RemoveInvalidHandle to SignalHandleError to mirror SignalHandleReady.
2. Use SignalHandleError wherever handler->OnHandleError is called.

Committed: https://crrev.com/edd0504f025875837561cdc2a71a9939b8479504
Cr-Commit-Position: refs/heads/master@{#369389}

[Anand Mistry (off Chromium), 2016-01-13 22:29:15]

amistry@chromium.org changed reviewers:
+ sky@chromium.org

[Anand Mistry (off Chromium), 2016-01-13 22:29:15]

No behaviour changes. Just a bit of consistency and code re-use cleanup.

[sky, 2016-01-14 00:16:18]

LGTM

[Anand Mistry (off Chromium), 2016-01-14 00:29:47]

The CQ bit was checked by amistry@chromium.org

[commit-bot: I haz the power, 2016-01-14 00:34:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1573383002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1573383002/1

[commit-bot: I haz the power, 2016-01-14 02:39:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-01-14 02:39:08]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on tryserver.chromium.android (JOB_TIMED_OUT, no
build URL)
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_TIMED_OUT,
no build URL)
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_TIMED_OUT,
no build URL)
  android_clang_dbg_recipe on tryserver.chromium.android (JOB_TIMED_OUT, no
build URL)
  android_compile_dbg on tryserver.chromium.android (JOB_TIMED_OUT, no build
URL)
  cast_shell_android on tryserver.chromium.android (JOB_TIMED_OUT, no build URL)
  linux_android_rel_ng on tryserver.chromium.android (JOB_TIMED_OUT, no build
URL)

[Anand Mistry (off Chromium), 2016-01-14 10:46:24]

The CQ bit was checked by amistry@chromium.org

[commit-bot: I haz the power, 2016-01-14 10:46:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1573383002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1573383002/1

[commit-bot: I haz the power, 2016-01-14 10:51:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-14 10:52:22]

Description was changed from

==========
Small set of cleanups to MessagePumpMojo.

These are:
1. Change RemoveInvalidHandle to SignalHandleError to mirror SignalHandleReady.
2. Use SignalHandleError wherever handler->OnHandleError is called.
==========

to

==========
Small set of cleanups to MessagePumpMojo.

These are:
1. Change RemoveInvalidHandle to SignalHandleError to mirror SignalHandleReady.
2. Use SignalHandleError wherever handler->OnHandleError is called.

Committed: https://crrev.com/edd0504f025875837561cdc2a71a9939b8479504
Cr-Commit-Position: refs/heads/master@{#369389}
==========

[commit-bot: I haz the power, 2016-01-14 10:52:22]

Patchset 1 (id:??) landed as
https://crrev.com/edd0504f025875837561cdc2a71a9939b8479504
Cr-Commit-Position: refs/heads/master@{#369389}