Issue metadata
Sign in to add a comment
|
1.8%-1605.8% regression in memory.top_10_mobile at 421340:421438 |
||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Sep 29 2016
Started bisect job https://chromeperf.appspot.com/buildbucket_job_status/9000182292111098192
,
Sep 29 2016
,
Sep 30 2016
=== Auto-CCing suspected CL author sievers@chromium.org === Hi sievers@chromium.org, the bisect results pointed to your CL below as possibly causing a regression. Please have a look at this info and see whether your CL be related. ===== 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@421387 141394 182.91 5 good chromium@421389 141394 182.91 5 good chromium@421390 2410578 182.91 5 bad <-- chromium@421393 2410578 182.91 5 bad chromium@421399 2410578 182.91 5 bad chromium@421410 2410578 182.91 5 bad Bisect job ran on: android_nexus5X_perf_bisect Bug ID: 651430 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/background/after_http_m_intl_taobao_com_group_purchase_html Relative Change: 1604.87% Score: 99.9 Buildbot stdio: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_nexus5X_perf_bisect/builds/705 Job details: https://chromeperf.appspot.com/buildbucket_job_status/9000182292111098192 Not what you expected? We'll investigate and get back to you! https://chromeperf.appspot.com/bad_bisect?try_job_id=6120656637263872 | 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