Do not upload thread load to UMA when throttling is disabled by command-line flag. |
||||
Issue descriptionThrottling can be disabled by a command line flag for testing, which can skew results for testing. We should disable reporting when we see this flag.
,
Feb 13 2017
,
Feb 14 2017
Your change meets the bar and is auto-approved for M57. Please go ahead and merge the CL to branch 2987 manually. Please contact milestone owner if you have questions. Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Feb 14 2017
Could you please merge your CL into M57 branch 2987 before 4 PM PT today, Tuesday (02/14/17). Thank you.
,
Feb 16 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2dfac077f46cae7ed8ec36fbf143d22fcecebf9a commit 2dfac077f46cae7ed8ec36fbf143d22fcecebf9a Author: Alexander Timin <altimin@chromium.org> Date: Thu Feb 16 12:41:39 2017 [scheduler] Don't report thread load to UMA when throttling is disabled. BUG= 691655 Review-Url: https://codereview.chromium.org/2697643002 Cr-Commit-Position: refs/heads/master@{#450076} (cherry picked from commit 815ec31ab7487998e1bc713a1fc7124041405dca) Review-Url: https://codereview.chromium.org/2700773002 . Cr-Commit-Position: refs/branch-heads/2987@{#545} Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943} [modify] https://crrev.com/2dfac077f46cae7ed8ec36fbf143d22fcecebf9a/third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.cc |
||||
►
Sign in to add a comment |
||||
Comment 1 by bugdroid1@chromium.org
, Feb 13 2017