• Alexei Svitkine's avatar
    Fix --force-fieldtrials= changing group of unrelated trials. · 047cb6b7
    Alexei Svitkine authored
    The command-line flag would turn off UMA, so as not to bias server-
    side data. But this had the consequence of changing how all field
    trials are randomized (causing the low entropy source to be used),
    which would cause unintended side-effects and in particular make it
    very hard to effectively debug trial behavior.
    
    This change updates the logic so that specifying that flag does not
    change the state of UMA consent, but instead changes the state of
    UMA reporting. This way, field trials are still randomized the
    same way, but UMA data is not reported.
    
    Includes a unit test.
    
    Bug: 932273
    Change-Id: I4216bb23a9784d963f30d0e2516e1e4d31784ab7
    Reviewed-on: https://chromium-review.googlesource.com/c/1479632Reviewed-by: default avatarJesse Doherty <jwd@chromium.org>
    Commit-Queue: Alexei Svitkine <asvitkine@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#634885}
    047cb6b7
chrome_metrics_service_accessor.h 6.12 KB