• Xida Chen's avatar
    [throughput] Ignore the first sample when reporting UKM · 35c14b06
    Xida Chen authored
    Currently we report throughput to UKM at:
      1. 5 seconds after a sequence starts.
      2. Every 500 seconds after that as long as the sequence lasts.
    The problem is that in many cases users stays on a page less than 500
    seconds, and then we would only get the report for the first 5 seconds,
    which can be heavily biased by the page loading time.
    
    This CL is a temp solution to ignore the first 5 seconds, so instead
    we will report throughput at:
      1. The second 5 seconds after a sequence starts.
      2. Every 500 seconds after that as long as the sequence lasts.
    
    Bug: 1040634
    Change-Id: I808b3ff56acce12c3a81e8fca5b7d472f4fcf4a2
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129014
    Commit-Queue: Xida Chen <xidachen@chromium.org>
    Reviewed-by: default avatarRobert Flack <flackr@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#755463}
    35c14b06
throughput_ukm_reporter.cc 2.66 KB