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

Issue 618658 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Jul 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug-Regression



Sign in to add a comment

1.7%-7.6% regression in page_cycler.top_10_mobile at 398641:398726

Project Member Reported by sullivan@chromium.org, Jun 9 2016

Issue description

This is all on the chromium.perf waterfall, looks like a pretty clear memory regression across a lot of pages.
 
Cc: primiano@chromium.org picksi@chromium.org
+primiano, picksi as a heads up for a potentially big memory regression.
Cc: vmp...@chromium.org
Owner: vmp...@chromium.org

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

Hi vmpstr@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 : cc: Separate raster and decode prepaint regions.
Author  : vmpstr
Commit description:
  
This patch separates raster and decode distances to be
independent of each other. Specifically, any tile in prepaint
that is further than 1000 screen space pixels from the
viewport will not be rasterized. Instead, we will pull out
the images on those tiles and predecode those instead.

R=enne, ericrk
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2007163002
Cr-Commit-Position: refs/heads/master@{#398677}
Commit  : 70a29c1298e666c465a8e75530575eda1aeaa37c
Date    : Wed Jun 08 21:21:04 2016


===== TESTED REVISIONS =====
Revision         Mean     Std Dev  N  Good?
chromium@398671  47374.3  481.623  6  good
chromium@398675  47396.2  750.634  5  good
chromium@398676  47545.4  425.727  5  good
chromium@398677  50601.2  760.06   5  bad    <--
chromium@398678  50916.6  701.424  5  bad

Bisect job ran on: android_nexus6_perf_bisect
Bug ID: 618658

Test Command: src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --also-run-disabled-tests page_cycler.basic_oopif
Test Metric: vm_private_dirty_final_renderer/vm_private_dirty_final_renderer
Relative Change: 7.30%
Score: 99.9

Buildbot stdio: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_nexus6_perf_bisect/builds/2228
Job details: https://chromeperf.appspot.com/buildbucket_job_status/9010329665741319216


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

| 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!
Cc: ericrk@chromium.org enne@chromium.org
Project Member

Comment 5 by sheriffbot@chromium.org, Jul 6 2016

Labels: -M-53 M-54 MovedFrom-53
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Ping

Comment 7 by enne@chromium.org, Jul 12 2016

Status: WontFix (was: Assigned)
This is wont fix.  On Android, the renderer memory increases a little, but the total memory decreases by a lot more.

https://chromeperf.appspot.com/report?sid=43f2df3c53feebd218dc9e40262320a9332536533c70b941d4d501f2c09b6d36&start_rev=397703&end_rev=399383

Comment 8 by enne@chromium.org, Jul 12 2016

See also: http://crbug.com/624467

Sign in to add a comment