Issue metadata
Sign in to add a comment
|
2.5% regression in system_health.memory_mobile at 454246:454272 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Mar 7 2017
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/8985765507114778128
,
Mar 7 2017
=== Auto-CCing suspected CL author zqzhang@chromium.org === Hi zqzhang@chromium.org, the bisect results pointed to your CL, please take a look at the results. === BISECT JOB RESULTS === Perf regression found with culprit Suspected Commit Author : zqzhang Commit : abc082487c58110e72e8d8e9ff6fa0b7abc9dd12 Date : Thu Mar 02 16:07:14 2017 Subject: [Blink>Media] Add heuristic for dominant video detection for Android Bisect Details Configuration: android_webview_arm64_aosp_perf_bisect Benchmark : system_health.memory_mobile Metric : memory:webview:all_processes:reported_by_os:system_memory:proportional_resident_size_avg/load_tools/load_tools_weather Change : 1.82% | 157790962.667 -> 161877649.143 Revision Result N chromium@454245 157790963 +- 2658899 6 good chromium@454259 158109256 +- 2777899 9 good chromium@454266 157857693 +- 2406010 9 good chromium@454269 158940598 +- 3810435 14 good chromium@454271 159634943 +- 6060971 14 good chromium@454272 161877649 +- 7497523 14 bad <-- Please refer to the following doc on diagnosing memory regressions: https://chromium.googlesource.com/chromium/src/+/master/docs/memory-infra/memory_benchmarks.md To Run This Test src/tools/perf/run_benchmark -v --browser=android-webview --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=load.tools.weather system_health.memory_mobile Debug Info https://chromeperf.appspot.com/buildbucket_job_status/8985765507114778128 Is this bisect wrong? https://chromeperf.appspot.com/bad_bisect?try_job_id=6451634835554304 | O O | Visit http://www.chromium.org/developers/speed-infra/perf-bug-faq | X | for more information addressing perf regression bugs. For feedback, | / \ | file a bug with component Speed>Bisection. Thank you!
,
Mar 7 2017
Should be the same cause as: https://crbug.com/697980
,
Mar 8 2017
This is wontfix as of #697980. I'm adding a flag to disable this feature in case the client side doesn't need fullscreen detection, which is tracked in issue #699565. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by kraynov@chromium.org
, Mar 7 2017