GitLab

FieldTrials should be recorded through sync when an event logger client
registers a dependency, their dependent event is recorded, and the
trial is finalized. All trials are recorded whenever the FieldTrial
event is emitted, which should be on a change or period, but no more.
Feature [and params] exist but may never actually change.

Because FieldTrials (and other imminent types) are not tied to
navigations, reworked the ShouldRecordEvent() check to be more
permissive, but also enforce presence (or lack) of navigation id.

Bug: 719038
Change-Id: Ib62a18949dde41d194935347768b3fef0d2623d6
Reviewed-on: https://chromium-review.googlesource.com/764408
Commit-Queue: Sky Malice <skym@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Reviewed-by: Pavel Yatsuk <pavely@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521052}