Remove calls to MessageLoop::Run/RunUntilIdle.

Remove calls to MessageLoop::Run/RunUntilIdle.

These methods are deprecated. RunLoop should be used instead.

BUG=616447
TBR=eugenebut@chromium.org

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

[fdoray, 2016-09-26 14:43:53]

fdoray@chromium.org changed reviewers:
+ eugenebut@chromium.org, jochen@chromium.org, ryansturm@chromium.org,
tommi@chromium.org, xiyuan@chromium.org

[fdoray, 2016-09-26 14:43:54]

jochen@chromium.org: Please review changes in chrome/utility

xiyuan@chromium.org: Please review changes in chrome/browser/chromeos

ryansturm@chromium.org: Please review changes in components/previews

eugenebut@chromium.org: Please review changes in ios/web

tommi@chromium.org: Please review changes in media/ and chrome/browser/media/

[fdoray, 2016-09-26 14:44:12]

Description was changed from

==========
Remove calls to MessageLoop::Run/RunUntilIdle.

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

to

==========
Remove calls to MessageLoop::Run/RunUntilIdle.

These methods are deprecated. RunLoop should be used instead.

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

[fdoray, 2016-09-26 14:44:23]

Description was changed from

==========
Remove calls to MessageLoop::Run/RunUntilIdle.

These methods are deprecated. RunLoop should be used instead.

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

to

==========
Remove calls to MessageLoop::Run/RunUntilIdle.

These methods are deprecated. RunLoop should be used instead.

BUG=616447
==========

[tommi (sloooow) - chröme, 2016-09-26 14:52:40]

lgtm

[jochen (gone - plz use gerrit), 2016-09-26 15:11:32]

utility lgtm with comment

https://codereview.chromium.org/2361203002/diff/60001/chrome/utility/importer...
File chrome/utility/importer/firefox_importer_unittest_utils_mac.cc (right):

https://codereview.chromium.org/2361203002/diff/60001/chrome/utility/importer...
chrome/utility/importer/firefox_importer_unittest_utils_mac.cc:194: return true;
just make the method void()

[RyanSturm, 2016-09-26 15:22:46]

Previews lgtm

[xiyuan, 2016-09-26 16:11:37]

c/b/chromeos lgtm

[fdoray, 2016-09-26 17:21:18]

Description was changed from

==========
Remove calls to MessageLoop::Run/RunUntilIdle.

These methods are deprecated. RunLoop should be used instead.

BUG=616447
==========

to

==========
Remove calls to MessageLoop::Run/RunUntilIdle.

These methods are deprecated. RunLoop should be used instead.

BUG=616447
TBR=eugenebut@chromium.org
==========

[fdoray, 2016-09-26 17:22:12]

TBR eugenebut@chromium.org because there longer we wait to land this, the more
chances there are that other devs will add calls to these methods in their code.

[fdoray, 2016-09-26 17:22:16]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-09-26 17:22:44]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[fdoray, 2016-09-26 17:22:46]

TBR eugenebut@chromium.org because the longer we wait to land this, the more
chances there are that other devs will add calls to these methods in their code.

[fdoray, 2016-09-26 17:23:20]

The CQ bit was unchecked by fdoray@chromium.org

[fdoray, 2016-09-26 17:26:00]

https://codereview.chromium.org/2361203002/diff/60001/chrome/utility/importer...
File chrome/utility/importer/firefox_importer_unittest_utils_mac.cc (right):

https://codereview.chromium.org/2361203002/diff/60001/chrome/utility/importer...
chrome/utility/importer/firefox_importer_unittest_utils_mac.cc:194: return true;
On 2016/09/26 15:11:32, jochen (slow) wrote:
> just make the method void()

Done.

[fdoray, 2016-09-26 17:26:04]

The CQ bit was checked by fdoray@chromium.org

[fdoray, 2016-09-26 17:26:04]

The patchset sent to the CQ was uploaded after l-g-t-m from jochen@chromium.org,
ryansturm@chromium.org, tommi@chromium.org, xiyuan@chromium.org
Link to the patchset: https://codereview.chromium.org/2361203002/#ps80001
(title: "CR jochen #6")

[commit-bot: I haz the power, 2016-09-26 17:26:48]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-26 18:53:25]

Description was changed from

==========
Remove calls to MessageLoop::Run/RunUntilIdle.

These methods are deprecated. RunLoop should be used instead.

BUG=616447
TBR=eugenebut@chromium.org
==========

to

==========
Remove calls to MessageLoop::Run/RunUntilIdle.

These methods are deprecated. RunLoop should be used instead.

BUG=616447
TBR=eugenebut@chromium.org
==========

[commit-bot: I haz the power, 2016-09-26 18:53:26]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-09-26 18:55:24]

Description was changed from

==========
Remove calls to MessageLoop::Run/RunUntilIdle.

These methods are deprecated. RunLoop should be used instead.

BUG=616447
TBR=eugenebut@chromium.org
==========

to

==========
Remove calls to MessageLoop::Run/RunUntilIdle.

These methods are deprecated. RunLoop should be used instead.

BUG=616447
TBR=eugenebut@chromium.org

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

[commit-bot: I haz the power, 2016-09-26 18:55:26]

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

[Eugene But (OOO till 7-30), 2016-09-26 22:01:58]

ios/web lgtm