Clean up ifdefs in base stack sampling profiler unit test.

Clean up ifdefs in base stack sampling profiler unit test.

This keeps the property of compiling the code on all platforms,
while removing the need to have a ton of individual per-test ifdefs.

For tests that are later disabled for other reasons, this can result
in DISABLED_ prefix appearing two times, but that doesn't seem too
problematic and the code cleanup benefits outweigh that.

BUG=none

Review-Url: https://codereview.chromium.org/2947023002
Cr-Commit-Position: refs/heads/master@{#480949}
Committed: https://chromium.googlesource.com/chromium/src/+/a933334960a47bbb9dab17f17f1341c80ed4749d

[Alexei Svitkine (slow), 2017-06-20 19:47:03]

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

[Alexei Svitkine (slow), 2017-06-20 19:47:10]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2017-06-20 19:47:44]

Dry run: CQ is trying da patch.

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

[Alexei Svitkine (slow), 2017-06-20 19:47:53]

asvitkine@chromium.org changed reviewers:
+ wittman@chromium.org

[Alexei Svitkine (slow), 2017-06-20 19:47:53]

Still working on getting my other CL for periodic sampling cleaned up for
review, but in the meanwhile, here's a cleanup CL to a unit test file.

[commit-bot: I haz the power, 2017-06-20 20:10:09]

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

[commit-bot: I haz the power, 2017-06-20 20:10:09]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Alexei Svitkine (slow), 2017-06-20 20:16:29]

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

[commit-bot: I haz the power, 2017-06-20 20:17:10]

Dry run: CQ is trying da patch.

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

[Mike Wittman, 2017-06-20 20:28:13]

lgtm, thanks!

[commit-bot: I haz the power, 2017-06-20 20:37:01]

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

[commit-bot: I haz the power, 2017-06-20 20:37:01]

Dry run: This issue passed the CQ dry run.

[Alexei Svitkine (slow), 2017-06-20 20:38:04]

The CQ bit was checked by asvitkine@chromium.org

[commit-bot: I haz the power, 2017-06-20 20:38:58]

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-06-20 20:46:49]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1497991084187390,
"parent_rev": "cfbb107861720c74448855869af827e844728054", "commit_rev":
"a933334960a47bbb9dab17f17f1341c80ed4749d"}

[commit-bot: I haz the power, 2017-06-20 20:47:04]

Description was changed from

==========
Clean up ifdefs in base stack sampling profiler unit test.

This keeps the property of compiling the code on all platforms,
while removing the need to have a ton of individual per-test ifdefs.

For tests that are later disabled for other reasons, this can result
in DISABLED_ prefix appearing two times, but that doesn't seem too
problematic and the code cleanup benefits outweigh that.

BUG=none
==========

to

==========
Clean up ifdefs in base stack sampling profiler unit test.

This keeps the property of compiling the code on all platforms,
while removing the need to have a ton of individual per-test ifdefs.

For tests that are later disabled for other reasons, this can result
in DISABLED_ prefix appearing two times, but that doesn't seem too
problematic and the code cleanup benefits outweigh that.

BUG=none

Review-Url: https://codereview.chromium.org/2947023002
Cr-Commit-Position: refs/heads/master@{#480949}
Committed:
https://chromium.googlesource.com/chromium/src/+/a933334960a47bbb9dab17f17f13...
==========

[commit-bot: I haz the power, 2017-06-20 20:47:05]

Committed patchset #2 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/a933334960a47bbb9dab17f17f13...