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

Issue 602330 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

2.8% regression in thread_times.simple_mobile_sites at 385511:385608

Project Member Reported by rsch...@chromium.org, Apr 11 2016

Issue description

See the link to graphs below.
 
All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=602330

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


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

android-nexus5
This looks recovered in some places. Sending more bisects.
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Apr 29 2016

Mergedinto: 601793
Status: Duplicate (was: Assigned)

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


===== SUSPECTED CL(s) =====
Subject : Revert of Enable GPU Rasterization for content with any author defined viewport. (patchset #2 id:20001 of https://codereview.chromium.org/1817583003/ )
Author  : vmiura
Commit description:
  
Reason for revert:
Reverting until we can debug memory health regressions on WebView.

BUG= 596881 

Original issue's description:
> Enable GPU Rasterization for content with any author defined viewport.
>
> BUG= 591179 
>
> Committed: https://crrev.com/b338ee0b3125007b94854e44ed7b6df820c72a75
> Cr-Commit-Position: refs/heads/master@{#382394}

TBR=aelias@chromium.org,pdr@chromium.org,chrishtr@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 591179 

Review URL: https://codereview.chromium.org/1868563003

Cr-Commit-Position: refs/heads/master@{#385547}
Commit  : e707452627a8ace7951e49421be443fe544aac71
Date    : Wed Apr 06 20:29:10 2016


===== TESTED REVISIONS =====
Revision         Mean     Std Dev    N  Good?
chromium@385532  2.65956  0.0217891  5  good
chromium@385541  2.64224  0.0204438  5  good
chromium@385545  2.63663  0.0205674  5  good
chromium@385546  2.61896  0.0197764  5  good
chromium@385547  3.04968  0.0253837  5  bad    <--
chromium@385549  3.05509  0.0399876  5  bad
chromium@385565  3.08176  0.0319514  5  bad
chromium@385598  3.0947   0.040431   5  bad

Bisect job ran on: android_s5_perf_bisect
Bug ID: 602330

Test Command: src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --also-run-disabled-tests thread_times.simple_mobile_sites
Test Metric: thread_GPU_cpu_time_per_frame/http___m.nytimes.com_
Relative Change: 16.36%
Score: 99.9

Buildbot stdio: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_s5_perf_bisect/builds/631
Job details: https://chromeperf.appspot.com/buildbucket_job_status/9014084853519875904


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

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

Comment 4 by 42576172...@developer.gserviceaccount.com, Apr 29 2016


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


===== SUSPECTED CL(s) =====
Subject : Revert of Enable GPU Rasterization for content with any author defined viewport. (patchset #2 id:20001 of https://codereview.chromium.org/1817583003/ )
Author  : vmiura
Commit description:
  
Reason for revert:
Reverting until we can debug memory health regressions on WebView.

BUG= 596881 

Original issue's description:
> Enable GPU Rasterization for content with any author defined viewport.
>
> BUG= 591179 
>
> Committed: https://crrev.com/b338ee0b3125007b94854e44ed7b6df820c72a75
> Cr-Commit-Position: refs/heads/master@{#382394}

TBR=aelias@chromium.org,pdr@chromium.org,chrishtr@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 591179 

Review URL: https://codereview.chromium.org/1868563003

Cr-Commit-Position: refs/heads/master@{#385547}
Commit  : e707452627a8ace7951e49421be443fe544aac71
Date    : Wed Apr 06 20:29:10 2016


===== TESTED REVISIONS =====
Revision         Mean     Std Dev    N  Good?
chromium@385532  3.25029  0.0212104  5  good
chromium@385541  3.27481  0.0111358  5  good
chromium@385545  3.25852  0.0240216  5  good
chromium@385546  3.27504  0.029222   5  good
chromium@385547  3.77556  0.183191   5  bad    <--
chromium@385549  3.61782  0.22391    5  bad
chromium@385565  3.79033  0.216252   5  bad
chromium@385598  3.87055  0.050217   5  bad

Bisect job ran on: android_s5_perf_bisect
Bug ID: 602330

Test Command: src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --also-run-disabled-tests thread_times.simple_mobile_sites
Test Metric: thread_renderer_compositor_cpu_time_per_frame/thread_renderer_compositor_cpu_time_per_frame
Relative Change: 19.08%
Score: 99.5

Buildbot stdio: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_s5_perf_bisect/builds/632
Job details: https://chromeperf.appspot.com/buildbucket_job_status/9014079622146837888


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

| 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