New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 699125 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Android MediaRouter only (left Chro...
Closed: Mar 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

2.5% regression in system_health.memory_mobile at 454246:454272

Project Member Reported by kraynov@chromium.org, Mar 7 2017

Issue description

See the link to graphs below.
 
All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=699125

Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?keys=agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgIDglNidqQoM


Bot(s) for this bug's original alert(s):

android-webview-nexus5X
Cc: zqzh...@chromium.org
Owner: zqzh...@chromium.org

=== 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!
Cc: mlamouri@chromium.org
Status: Assigned (was: Untriaged)
Should be the same cause as:  https://crbug.com/697980 
Status: WontFix (was: Assigned)
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