Clean-up ProfileSyncServiceBookmarkTest

Clean-up ProfileSyncServiceBookmarkTest

This CL removes some unused #includes and puts all test fixture data members
into the private section.

This is both to make the following componentisation easier, and to address
proactively requests like
https://codereview.chromium.org/1646553002/diff/160001/chrome/browser/sync/abstract_profile_sync_service_test.h#newcode53
before they are made in the componentisation CL.

BUG=586642
Committed: https://crrev.com/227ce81f692e7656828d0ba72e3fad5d3d99ad3b
Cr-Commit-Position: refs/heads/master@{#376490}

[vabr (Chromium), 2016-02-17 16:05:57]

vabr@chromium.org changed reviewers:
+ pavely@chromium.org

[vabr (Chromium), 2016-02-17 16:05:57]

Hello Pavel,

PTAL.

Thanks!
Vaclav

[pavely, 2016-02-17 19:15:16]

lgtm

[vabr (Chromium), 2016-02-19 09:41:20]

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

[commit-bot: I haz the power, 2016-02-19 09:43:40]

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

[commit-bot: I haz the power, 2016-02-19 09:56:09]

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

[commit-bot: I haz the power, 2016-02-19 09:56:10]

Dry run: 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...)

[vabr (Chromium), 2016-02-19 13:07:41]

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

[commit-bot: I haz the power, 2016-02-19 13:10:36]

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

[commit-bot: I haz the power, 2016-02-19 13:26:46]

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

[commit-bot: I haz the power, 2016-02-19 13:26:47]

Dry run: 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...)

[vabr (Chromium), 2016-02-19 15:55:02]

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

[commit-bot: I haz the power, 2016-02-19 15:58:16]

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

[commit-bot: I haz the power, 2016-02-19 16:23:36]

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

[commit-bot: I haz the power, 2016-02-19 16:23:37]

Dry run: 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...)

[vabr (Chromium), 2016-02-19 17:47:33]

The CQ bit was checked by vabr@chromium.org

[vabr (Chromium), 2016-02-19 17:47:34]

The patchset sent to the CQ was uploaded after l-g-t-m from pavely@chromium.org
Link to the patchset: https://codereview.chromium.org/1706853002/#ps40001
(title: "Just rebased")

[commit-bot: I haz the power, 2016-02-19 17:49:54]

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

[commit-bot: I haz the power, 2016-02-19 18:11:32]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-02-19 18:12:45]

Description was changed from

==========
Clean-up ProfileSyncServiceBookmarkTest

This CL removes some unused #includes and puts all test fixture data members
into the private section.

This is both to make the following componentisation easier, and to address
proactively requests like
https://codereview.chromium.org/1646553002/diff/160001/chrome/browser/sync/ab...
before they are made in the componentisation CL.

BUG=586642
==========

to

==========
Clean-up ProfileSyncServiceBookmarkTest

This CL removes some unused #includes and puts all test fixture data members
into the private section.

This is both to make the following componentisation easier, and to address
proactively requests like
https://codereview.chromium.org/1646553002/diff/160001/chrome/browser/sync/ab...
before they are made in the componentisation CL.

BUG=586642

Committed: https://crrev.com/227ce81f692e7656828d0ba72e3fad5d3d99ad3b
Cr-Commit-Position: refs/heads/master@{#376490}
==========

[commit-bot: I haz the power, 2016-02-19 18:12:46]

Patchset 3 (id:??) landed as
https://crrev.com/227ce81f692e7656828d0ba72e3fad5d3d99ad3b
Cr-Commit-Position: refs/heads/master@{#376490}