• Alexei Svitkine's avatar
    Add full system profiles in .pma file as part of metrics init. · 87dac882
    Alexei Svitkine authored
    Prior to this change, only an incomplete system profile, notably
    missing field trials and uma_default_state fields, was stored
    at this point. So unclean shut downs after this point in time,
    but before the initial metrics log was saved would not include
    that info. This change fixes that by running the code that
    populates all the system profile fields.
    
    Since we're now calling ProvideSystemProfileMetrics() twice, the
    logic in FieldTrialsProvider was no longer correct in tracking
    the log creation time. Instead, this CL updates the API to have
    that be passed in.
    
    Additionally, Chrome OS an Screen Info providers needed updates since
    their dependencies are not yet available this early, so they are not
    able to provide all the information this early.
    
    Finally, webview unit tests needed updating to init mojo as they are
    now exercising the provider code which has a Mojo dependency.
    
    Bug: 992538
    TBR: sky@chromium.org
    Change-Id: Iae54398065470e390b70507d5b3ac09d5a983c9f
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1746909
    Commit-Queue: Alexei Svitkine <asvitkine@chromium.org>
    Reviewed-by: default avatarRichard Coles <torne@chromium.org>
    Reviewed-by: default avatarRobert Kaplow <rkaplow@chromium.org>
    Reviewed-by: default avatarBrian White <bcwhite@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#687000}
    87dac882
metrics_provider.cc 2.22 KB