Remove MessageLoop::os_modal_loop_.

Remove MessageLoop::os_modal_loop_.

This member is always false.

BUG=

Committed: https://crrev.com/1c8fc4c2731c8a42b36678bdc6be7b45d7541296
Cr-Commit-Position: refs/heads/master@{#395711}

[fdoray, 2016-05-19 17:21:47]

fdoray@chromium.org changed reviewers:
+ thestig@chromium.org

[fdoray, 2016-05-19 17:21:48]

Can you review this CL? Thanks.

[fdoray, 2016-05-19 17:21:53]

The CQ bit was checked by fdoray@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-19 17:22:36]

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

[Lei Zhang, 2016-05-19 17:31:07]

thestig@chromium.org changed reviewers:
+ ananta@chromium.org

[Lei Zhang, 2016-05-19 17:31:07]

+ananta

[commit-bot: I haz the power, 2016-05-19 18:57:33]

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

[commit-bot: I haz the power, 2016-05-19 18:57:33]

Dry run: This issue passed the CQ dry run.

[fdoray, 2016-05-24 15:02:47]

ananta@: Can you review this CL?

Context: We plan to use MessagePumps in base/task_scheduler/.
MessageLoop::current() will return nullptr on our worker threads.

We need to get rid of MessageLoop::current()->os_modal_loop() in
MessagePumpForUI (would make sense since it's always false) OR change the
condition to "if (MessageLoop::current() &&
MessageLoop::current()->os_modal_loop())".

[ananta, 2016-05-24 19:02:46]

lgtm

[Lei Zhang, 2016-05-24 19:07:21]

rslgtm

[fdoray, 2016-05-24 19:44:24]

On 2016/05/24 19:07:21, Lei Zhang wrote:
> rslgtm

thestig@: Is that an lgtm?

[fdoray, 2016-05-24 19:44:55]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-05-24 19:45:27]

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

[commit-bot: I haz the power, 2016-05-24 19:54:02]

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

[commit-bot: I haz the power, 2016-05-24 19:54:02]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Lei Zhang, 2016-05-24 20:12:10]

rs lgtm

[fdoray, 2016-05-24 20:12:23]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-05-24 20:13:02]

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

[commit-bot: I haz the power, 2016-05-24 22:29:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-24 22:32:20]

Description was changed from

==========
Remove MessageLoop::os_modal_loop_.

This member is always false.

BUG=
==========

to

==========
Remove MessageLoop::os_modal_loop_.

This member is always false.

BUG=

Committed: https://crrev.com/1c8fc4c2731c8a42b36678bdc6be7b45d7541296
Cr-Commit-Position: refs/heads/master@{#395711}
==========

[commit-bot: I haz the power, 2016-05-24 22:32:21]

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