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

Issue 699116 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression

Blocked on:
issue 699476



Sign in to add a comment

56.3% regression in system_health.memory_mobile at 454708:454753

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=699116

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


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

android-webview-nexus5X
Cc: jfernan...@igalia.com
Owner: jfernan...@igalia.com

=== Auto-CCing suspected CL author jfernandez@igalia.com ===

Hi jfernandez@igalia.com, 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 : jfernandez
  Commit : a90dba1f2dd094e51229dd2e5e059c6b53cee93a
  Date   : Sat Mar 04 01:36:34 2017
  Subject: [table] Stretching tables when needed due to self-alignment properties

Bisect Details
  Configuration: android_webview_arm64_aosp_perf_bisect
  Benchmark    : system_health.memory_mobile
  Metric       : memory:webview:all_processes:reported_by_chrome:cc:effective_size_avg/load_tools/load_tools_weather
  Change       : 58.13% | 17328296.0 -> 27401725.3333

Revision             Result                   N
chromium@454707      17328296 +- 0.0          6      good
chromium@454730      17328296 +- 0.0          6      good
chromium@454742      17328296 +- 0.0          6      good
chromium@454743      27401725 +- 1779821      6      bad       <--
chromium@454744      27077763 +- 5404.2       6      bad
chromium@454745      27076776 +- 0.0          6      bad
chromium@454748      27076776 +- 0.0          6      bad
chromium@454753      27401725 +- 1779821      6      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/8985767546331003712

Is this bisect wrong?
  https://chromeperf.appspot.com/bad_bisect?try_job_id=5782027959271424


| 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!
I'll take a look ASAP.
Sorry, replied with my personal account. I'll take a look ASAP.
https://crrev.com/2528253003 has been reverted. Could anybody verify the regression is solved now ?
Blockedon: 699476
In any case, I think we should merge https://crrev.com/2740063003 in M59 as soon as possible.
Status: Fixed (was: Untriaged)
This issue should be FIXED now, as the patch causing the regression as been reverted.

Sign in to add a comment