GitLab

The DCHECK was introduced long ago in
https://chromiumcodereview.appspot.com/11522009 where it may have been
necessary, but in today's implementation the code does not rely on
ThreadTaskRunnerHandle.

The DCHECK is a proxy for verifying that a global field trial list
exists, which can be checked more directly via the value returned by
FieldTrialList::AddObserver().

The patch is motivated by DCHECK failures observed recently on ios
after the codepath has been triggered from the sync thread, due to
https://chromium-review.googlesource.com/c/chromium/src/+/1827368.

Bug: 1011088
Change-Id: I075c28f86962c88b9331290c1b57f2da8a3af755
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1844992
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721564}