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

Issue 651442 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 651430
Owner:
Last visit > 30 days ago
Closed: Sep 2016
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

37.5%-38.6% regression in memory.top_10_mobile at 421375:421438

Project Member Reported by pmeenan@chromium.org, Sep 29 2016

Issue description

See the link to graphs below.
 
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Sep 30 2016

Mergedinto: 651430
Status: Duplicate (was: Assigned)

===== BISECT JOB RESULTS =====
Status: completed


===== SUSPECTED CL(s) =====
Subject : Reland of Android: Trigger tab placeholder update when activity is paused (patchset #1 id:1 of https://codereview.chromium.org/2306623002/ )
Author  : sievers
Commit description:
  
Reason for reland (revert-of-revert):
Now that https://codereview.chromium.org/2336043004/
has landed, we will free GL context-related ashmem after
the readback and when the app goes to the background.

Original issue's description:
> Revert of Android: Trigger tab placeholder update when activity is paused (patchset #3 id:40001 of https://codereview.chromium.org/2186453004/ )
>
> Reason for revert:
> Seeing lots of memory regression since this landed. Bisects point to this CL with high confidence. See  crbug.com/641962 .
>
> Original issue's description:
> > Android: Trigger tab placeholder update when activity is paused
> >
> > This makes sure we have a placeholder bitmap when we resume
> > and don't have a frame from the renderer yet.
> >
> > BUG= 636630 
> >
> > Committed: https://crrev.com/0a1f48d5cb6a5d263f85bc5322912b7f703c40ac
> > Cr-Commit-Position: refs/heads/master@{#414822}
>
> TBR=dtrainor@chromium.org,sievers@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG= 636630 , 641962 
>
> Committed: https://crrev.com/be54d2fa0b7a766b0391de082eebeeee53888397
> Cr-Commit-Position: refs/heads/master@{#415990}

TBR=dtrainor@chromium.org,primiano@chromium.org,mustaq@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 636630 , 641962 

Review-Url: https://codereview.chromium.org/2372393002
Cr-Commit-Position: refs/heads/master@{#421390}
Commit  : 5d34220f9b827cfd514e03ba6215286f2467f29d
Date    : Wed Sep 28 00:01:07 2016


===== TESTED REVISIONS =====
Revision         Mean     Std Dev  N  Good?
chromium@421374  3410657  5135.41  5  good
chromium@421382  3433922  50861.1  5  good
chromium@421386  3468656  62521.2  5  good
chromium@421388  3427860  41929.7  5  good
chromium@421389  3429826  56730.4  5  good
chromium@421390  6037012  79269.4  5  bad    <--
chromium@421406  6090588  34627.1  5  bad
chromium@421438  6134989  30327.2  5  bad

Bisect job ran on: android_nexus9_perf_bisect
Bug ID: 651442

Test Command: src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --also-run-disabled-tests memory.top_10_mobile
Test Metric: memory:chrome:all_processes:reported_by_os:system_memory:ashmem:proportional_resident_size_avg/foreground/https_mobile_twitter_com_justinbieber_skip_interstitial_true
Relative Change: 79.88%
Score: 99.9

Buildbot stdio: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_nexus9_perf_bisect/builds/2137
Job details: https://chromeperf.appspot.com/buildbucket_job_status/9000180908725867136


Not what you expected? We'll investigate and get back to you!
  https://chromeperf.appspot.com/bad_bisect?try_job_id=5013885977886720

| 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 Tests>AutoBisect.  Thank you!

Sign in to add a comment