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

Issue 610301 link

Starred by 0 users

Issue metadata

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



Sign in to add a comment

2.5% regression in page_cycler.basic_oopif at 392003:392010

Project Member Reported by alexclarke@chromium.org, May 9 2016

Issue description

This might just be noise.  Lets see if the bots find anything.
 
All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=610301

Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?keys=agxzfmNocm9tZXBlcmZyFAsSB0Fub21hbHkYgICgtJe-uAoM


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

android-nexus5
Mergedinto: 609940
Status: Duplicate (was: Assigned)

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


===== SUSPECTED CL(s) =====
Subject : Re-enable complex text path on android
Author  : eae
Commit description:
  
Trying to re-enable the complex text path on Android now that hb_font_t
is cached with the user data container instead of the hb_face_t object.

This should result in lower memory usage for the complex text code path,
we'll watch the memory bots though and revert as needed if it regresses.

BUG=577306
R=drott@chromium.org

Review-Url: https://codereview.chromium.org/1949053002
Cr-Commit-Position: refs/heads/master@{#392010}
Commit  : 9af72a88ea5b39c2c36c99f8b091c5feae82ef7d
Date    : Fri May 06 05:51:07 2016


===== TESTED REVISIONS =====
Revision         Mean     Std Dev  N  Good?
chromium@392002  7167.68  40.4358  6  good
chromium@392006  7196.59  25.9105  5  good
chromium@392008  7219.46  17.0826  5  good
chromium@392009  7197.51  45.1293  5  good
chromium@392010  7373.21  29.6327  5  bad    <--

Bisect job ran on: android_nexus5_perf_bisect
Bug ID: 610301

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: cold_times/page_load_time
Relative Change: 3.21%
Score: 99.9

Buildbot stdio: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_nexus5_perf_bisect/builds/3674
Job details: https://chromeperf.appspot.com/buildbucket_job_status/9013134269156299888


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

| 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!

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


===== SUSPECTED CL(s) =====
Subject : Re-enable complex text path on android
Author  : eae
Commit description:
  
Trying to re-enable the complex text path on Android now that hb_font_t
is cached with the user data container instead of the hb_face_t object.

This should result in lower memory usage for the complex text code path,
we'll watch the memory bots though and revert as needed if it regresses.

BUG=577306
R=drott@chromium.org

Review-Url: https://codereview.chromium.org/1949053002
Cr-Commit-Position: refs/heads/master@{#392010}
Commit  : 9af72a88ea5b39c2c36c99f8b091c5feae82ef7d
Date    : Fri May 06 05:51:07 2016


===== TESTED REVISIONS =====
Revision         Mean     Std Dev  N  Good?
chromium@392000  7197.39  23.5869  6  good
chromium@392005  7217.16  27.2512  5  good
chromium@392008  7200.26  38.1187  5  good
chromium@392009  7228.41  41.8551  5  good
chromium@392010  7372.69  12.7627  5  bad    <--
chromium@392020  7342.01  26.3576  5  bad
chromium@392040  7345.57  21.0216  5  bad

Bisect job ran on: android_nexus5_perf_bisect
Bug ID: 610301

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: cold_times/page_load_time
Relative Change: 2.04%
Score: 99.9

Buildbot stdio: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_nexus5_perf_bisect/builds/3675
Job details: https://chromeperf.appspot.com/buildbucket_job_status/9013134246039372256


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

| 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