Remove calls to deprecated MessageLoop methods in site_instance_impl_unittest.cc

Remove calls to deprecated MessageLoop methods in site_instance_impl_unittest.cc

This CL replaces calls to MessageLoop::RunUntilIdle with
RunLoop().RunUntilIdle() in site_instance_impl_unittest.cc.

Note that there is no reason to call RunUntilIdle() multiple times
in a row. After a call returns, the MessageLoop is idle. It stays idle
until a task is posted or a timer expires. There is no reason to call
RunUntilIdle() before that.

BUG=616447
Committed: https://crrev.com/e46618d151ea43f2a7cc135f25cea6a173f67a58
Cr-Commit-Position: refs/heads/master@{#410447}

[fdoray, 2016-08-05 16:03:18]

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

[commit-bot: I haz the power, 2016-08-05 16:03:37]

Dry run: 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-08-05 16:12:15]

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

[commit-bot: I haz the power, 2016-08-05 16:12:17]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[fdoray, 2016-08-05 18:20:25]

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

[commit-bot: I haz the power, 2016-08-05 18:20:49]

Dry run: 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-08-05 19:08:41]

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

[commit-bot: I haz the power, 2016-08-05 19:08:43]

Dry run: This issue passed the CQ dry run.

[fdoray, 2016-08-06 01:09:56]

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

[fdoray, 2016-08-06 01:09:56]

jam@: PTAL

[jam, 2016-08-08 19:49:03]

The CQ bit was checked by jam@chromium.org

[jam, 2016-08-08 19:49:03]

lgtm

[commit-bot: I haz the power, 2016-08-08 19:49:57]

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-08-08 20:42:18]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-08 20:44:52]

Description was changed from

==========
Remove calls to deprecated MessageLoop methods in site_instance_impl_unittest.cc

This CL replaces calls to MessageLoop::RunUntilIdle with
RunLoop().RunUntilIdle() in site_instance_impl_unittest.cc.

Note that there is no reason to call RunUntilIdle() multiple times
in a row. After a call returns, the MessageLoop is idle. It stays idle
until a task is posted or a timer expires. There is no reason to call
RunUntilIdle() before that.

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

to

==========
Remove calls to deprecated MessageLoop methods in site_instance_impl_unittest.cc

This CL replaces calls to MessageLoop::RunUntilIdle with
RunLoop().RunUntilIdle() in site_instance_impl_unittest.cc.

Note that there is no reason to call RunUntilIdle() multiple times
in a row. After a call returns, the MessageLoop is idle. It stays idle
until a task is posted or a timer expires. There is no reason to call
RunUntilIdle() before that.

BUG=616447

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

[commit-bot: I haz the power, 2016-08-08 20:44:53]

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