Issue metadata
Sign in to add a comment
|
37.5%-38.6% regression in memory.top_10_mobile at 421375:421438 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Sep 29 2016
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/9000180908725867136
,
Sep 30 2016
===== 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 |
|||||||||||||||||||||
Comment 1 by pmeenan@chromium.org
, Sep 29 2016