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

Issue 763407 link

Starred by 8 users

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Sep 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

59.9%-124.7% regression in memory.top_10_mobile at 500324:500358

Project Member Reported by rmcilroy@chromium.org, Sep 8 2017

Issue description

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

(For debugging:) Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?sid=51f745e0bc1fb17c86ac2819459fd6b361366c3ec0b66edc82673550e4fae73a


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

android-webview-nexus6
Cc: xunji...@chromium.org
Owner: xunji...@chromium.org
Status: Assigned (was: Untriaged)

=== Auto-CCing suspected CL author xunjieli@chromium.org ===

Hi xunjieli@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 : xunjieli
  Commit : 724bb776bcdea642a5bc9cc20fe14bb0282b4900
  Date   : Wed Sep 06 14:29:20 2017
  Subject: [Telemetry] Use --ignore-certificate-errors-spki-list to bypass cert errors

Bisect Details
  Configuration: android_webview_nexus6_aosp_perf_bisect
  Benchmark    : memory.top_10_mobile
  Metric       : memory:webview:all_processes:reported_by_os:system_memory:java_heap:proportional_resident_size_avg/foreground/http_www_baidu_com_s_word_google
  Change       : 123.20% | 3938304.0 -> 8790357.33333

Revision                                 Result                 N
chromium@500323                          3938304 +- 180666      6      good
chromium@500341                          3944619 +- 164698      6      good
chromium@500344                          3936256 +- 183819      6      good
chromium@500344,catapult@c5a474f50f      4010325 +- 149200      6      good
chromium@500344,catapult@724bb776bc      9461248 +- 143855      6      bad       <--
chromium@500344,catapult@80c47581b1      9368235 +- 111335      6      bad
chromium@500344,catapult@4f611510b9      9304405 +- 169852      6      bad
chromium@500344,catapult@442f46edf7      9287339 +- 200561      6      bad
chromium@500345                          9139883 +- 216685      6      bad
chromium@500346                          9048405 +- 141154      6      bad
chromium@500350                          8932352 +- 201653      6      bad
chromium@500358                          8790357 +- 313380      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 memory.top_10_mobile

More information on addressing performance regressions:
  http://g.co/ChromePerformanceRegressions

Debug information about this bisect:
  https://chromeperf.appspot.com/buildbucket_job_status/8969004992655444176


For feedback, file a bug with component Speed>Bisection
Cc: nedngu...@google.com mariakho...@chromium.org
Labels: Performance-Memory
Status: WontFix (was: Assigned)
This is not a real regression. 
We changed from --ignore-certificate-errors to --ignore-certificate-errors-spki-list, a new startup flag which allows Chrome to bypass cert errors selectively and still exercise disk cache. I see //net MemoryDumpProvider shows a non-zero value for http cache, as expected.

The metrics are consistent after flag change, so I will mark this as WontFix.

+mariakhomenko@ as a FYI.
Project Member

Comment 5 by 42576172...@developer.gserviceaccount.com, Sep 11 2017

 Issue 763865  has been merged into this issue.
Project Member

Comment 6 by 42576172...@developer.gserviceaccount.com, Sep 11 2017

Cc: perezju@chromium.org
 Issue 763880  has been merged into this issue.
Project Member

Comment 7 by 42576172...@developer.gserviceaccount.com, Sep 11 2017

Cc: bmeu...@chromium.org
 Issue 763816  has been merged into this issue.
Project Member

Comment 8 by 42576172...@developer.gserviceaccount.com, Sep 11 2017

 Issue 763859  has been merged into this issue.
Project Member

Comment 9 by 42576172...@developer.gserviceaccount.com, Sep 11 2017

 Issue 763892  has been merged into this issue.
Project Member

Comment 10 by 42576172...@developer.gserviceaccount.com, Sep 11 2017

 Issue 763855  has been merged into this issue.
 Issue 763863  has been merged into this issue.
Project Member

Comment 12 by 42576172...@developer.gserviceaccount.com, Sep 12 2017

Cc: wolenetz@chromium.org
 Issue 764134  has been merged into this issue.
Project Member

Comment 13 by 42576172...@developer.gserviceaccount.com, Sep 12 2017

 Issue 763875  has been merged into this issue.
Project Member

Comment 14 by 42576172...@developer.gserviceaccount.com, Sep 13 2017

Cc: jgruber@chromium.org
 Issue 764619  has been merged into this issue.
Project Member

Comment 15 by 42576172...@developer.gserviceaccount.com, Sep 14 2017

Cc: jupvfranco@google.com
 Issue 765238  has been merged into this issue.
Project Member

Comment 16 by 42576172...@developer.gserviceaccount.com, Sep 14 2017

 Issue 765277  has been merged into this issue.

Comment 17 by torne@chromium.org, Sep 15 2017

Cc: msrchandra@chromium.org sandeepkumars@chromium.org nyerramilli@chromium.org
 Issue 763680  has been merged into this issue.
Project Member

Comment 18 by 42576172...@developer.gserviceaccount.com, Sep 15 2017

 Issue 765238  has been merged into this issue.
Project Member

Comment 19 by 42576172...@developer.gserviceaccount.com, Sep 15 2017

 Issue 765238  has been merged into this issue.
Project Member

Comment 20 by 42576172...@developer.gserviceaccount.com, Sep 28 2017

Cc: hablich@chromium.org
 Issue 769719  has been merged into this issue.

Sign in to add a comment