Replace SyncTest::SetUpInProcessBrowserTestFixture with SyncTest::SetUpOnMainThread.

Replace SyncTest::SetUpInProcessBrowserTestFixture with SyncTest::SetUpOnMainThread.

The former was added a long time ago in https://codereview.chromium.org/214047 before we had SetUpOnMainThread. We don't need SetUpInProcessBrowserTestFixture anymore after the core browser test harness has been split apart. Since SetUpInProcessBrowserTestFixture was added for sync, removing it here first and then I'll remove the other usages later.

BUG=713847
Review-Url: https://codereview.chromium.org/2835503003
Cr-Commit-Position: refs/heads/master@{#466151}
Committed: https://chromium.googlesource.com/chromium/src/+/4621c2490b9b9cc5cba1d57a57c6009687190eef

[jam, 2017-04-20 19:25:20]

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

[commit-bot: I haz the power, 2017-04-20 19:26:12]

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

[jam, 2017-04-20 20:01:53]

Description was changed from

==========
test sync

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

to

==========
Replace SyncTest::SetUpInProcessBrowserTestFixture with
SyncTest::SetUpOnMainThread.

The former was added a long time ago in https://codereview.chromium.org/214047
before we had SetUpOnMainThread. We don't need SetUpInProcessBrowserTestFixture
anymore after the core browser test harness has been split apart. Since
SetUpInProcessBrowserTestFixture was added for sync, removing it here first and
then I'll remove the other usages later.

BUG=713847
==========

[jam, 2017-04-20 20:01:54]

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

[commit-bot: I haz the power, 2017-04-20 20:22:28]

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

[commit-bot: I haz the power, 2017-04-20 20:22:29]

Dry run: This issue passed the CQ dry run.

[Nicolas Zea, 2017-04-20 21:18:32]

lgtm

[jam, 2017-04-20 21:47:49]

The CQ bit was checked by jam@chromium.org

[commit-bot: I haz the power, 2017-04-20 21:48:26]

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, 2017-04-20 21:52:39]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492724869323630, "parent_rev":
"6a4bbc741c93bb68c97825a8bf759636b1820c74", "commit_rev":
"4621c2490b9b9cc5cba1d57a57c6009687190eef"}

[commit-bot: I haz the power, 2017-04-20 21:52:45]

Description was changed from

==========
Replace SyncTest::SetUpInProcessBrowserTestFixture with
SyncTest::SetUpOnMainThread.

The former was added a long time ago in https://codereview.chromium.org/214047
before we had SetUpOnMainThread. We don't need SetUpInProcessBrowserTestFixture
anymore after the core browser test harness has been split apart. Since
SetUpInProcessBrowserTestFixture was added for sync, removing it here first and
then I'll remove the other usages later.

BUG=713847
==========

to

==========
Replace SyncTest::SetUpInProcessBrowserTestFixture with
SyncTest::SetUpOnMainThread.

The former was added a long time ago in https://codereview.chromium.org/214047
before we had SetUpOnMainThread. We don't need SetUpInProcessBrowserTestFixture
anymore after the core browser test harness has been split apart. Since
SetUpInProcessBrowserTestFixture was added for sync, removing it here first and
then I'll remove the other usages later.

BUG=713847

Review-Url: https://codereview.chromium.org/2835503003
Cr-Commit-Position: refs/heads/master@{#466151}
Committed:
https://chromium.googlesource.com/chromium/src/+/4621c2490b9b9cc5cba1d57a57c6...
==========

[commit-bot: I haz the power, 2017-04-20 21:52:46]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/4621c2490b9b9cc5cba1d57a57c6...