Move API for field trial params to base from variations.

Move API for field trial params to base from variations.

Now that the underlying implementation also works in sub-processes,
this CL moves the APIs to base (naming the field trial params), so
that they can be easily accessed by all Chrome sub-processes.

Marks the existing variation params APIs as deprecated. Existing
callers can be migrates via future CLs and these can then be
removed.

Unit tests also moved to base.

BUG=681160
Review-Url: https://codereview.chromium.org/2667553002
Cr-Commit-Position: refs/heads/master@{#447108}
Committed: https://chromium.googlesource.com/chromium/src/+/79ab08c0137f5485173e8de5cd28201774ce1a0f

[Alexei Svitkine (slow), 2017-01-30 07:49:49]

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

[Alexei Svitkine (slow), 2017-01-30 07:49:58]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2017-01-30 07:50:02]

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-01-30 07:56:51]

Patchset #1 (id:20001) has been deleted

[Alexei Svitkine (slow), 2017-01-30 07:56:58]

Patchset #3 (id:80001) has been deleted

[Alexei Svitkine (slow), 2017-01-30 07:57:04]

Patchset #2 (id:60001) has been deleted

[Alexei Svitkine (slow), 2017-01-30 07:57:08]

Patchset #1 (id:40001) has been deleted

[Alexei Svitkine (slow), 2017-01-30 07:57:14]

Patchset #1 (id:100001) has been deleted

[Alexei Svitkine (slow), 2017-01-30 07:59:22]

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

[Alexei Svitkine (slow), 2017-01-30 15:39:04]

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

[commit-bot: I haz the power, 2017-01-30 15:39:34]

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, 2017-01-30 16:52:17]

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

[commit-bot: I haz the power, 2017-01-30 16:52:18]

Dry run: This issue passed the CQ dry run.

[rkaplow, 2017-01-30 22:09:11]

lgtm

[Alexei Svitkine (slow), 2017-01-30 22:09:29]

The CQ bit was checked by asvitkine@chromium.org

[commit-bot: I haz the power, 2017-01-30 22:10:01]

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-01-30 22:21:34]

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

[commit-bot: I haz the power, 2017-01-30 22:21:35]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Alexei Svitkine (slow), 2017-01-30 22:23:55]

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

[Alexei Svitkine (slow), 2017-01-30 22:23:56]

+gab for base/ owners

[Alexei Svitkine (slow), 2017-01-30 22:24:29]

gab: just need an owners approval for base/BUILD.gn since I'm owner of
base/metrics

Thanks!

[gab, 2017-01-30 23:08:59]

On 2017/01/30 22:24:29, Alexei Svitkine (very slow) wrote:
> gab: just need an owners approval for base/BUILD.gn since I'm owner of
> base/metrics
> 
> Thanks!

LGTM but note that it's fine to TBR trivial (add/remove files) GN changes.

[Alexei Svitkine (slow), 2017-01-30 23:21:19]

The CQ bit was checked by asvitkine@chromium.org

[commit-bot: I haz the power, 2017-01-30 23:21:54]

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-01-30 23:27:14]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1485818479322460,
"parent_rev": "3809eb577f75f4ad4ff2998a10f4e9448b14c298", "commit_rev":
"79ab08c0137f5485173e8de5cd28201774ce1a0f"}

[commit-bot: I haz the power, 2017-01-30 23:27:45]

Description was changed from

==========
Move API for field trial params to base from variations.

Now that the underlying implementation also works in sub-processes,
this CL moves the APIs to base (naming the field trial params), so
that they can be easily accessed by all Chrome sub-processes.

Marks the existing variation params APIs as deprecated. Existing
callers can be migrates via future CLs and these can then be
removed.

Unit tests also moved to base.

BUG=681160
==========

to

==========
Move API for field trial params to base from variations.

Now that the underlying implementation also works in sub-processes,
this CL moves the APIs to base (naming the field trial params), so
that they can be easily accessed by all Chrome sub-processes.

Marks the existing variation params APIs as deprecated. Existing
callers can be migrates via future CLs and these can then be
removed.

Unit tests also moved to base.

BUG=681160

Review-Url: https://codereview.chromium.org/2667553002
Cr-Commit-Position: refs/heads/master@{#447108}
Committed:
https://chromium.googlesource.com/chromium/src/+/79ab08c0137f5485173e8de5cd28...
==========

[commit-bot: I haz the power, 2017-01-30 23:27:46]

Committed patchset #1 (id:120001) as
https://chromium.googlesource.com/chromium/src/+/79ab08c0137f5485173e8de5cd28...