New issue
Advanced search Search tips

Issue 798611 link

Starred by 1 user
Status: Fixed
Owner:
ericrk@chromium.org
Closed: Jan 2018
Cc:
flackr@chromium.org
kinuko@chromium.org
boliu@chromium.org
ericrk@chromium.org
piman@chromium.org
yigu@chromium.org
chrishtr@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

14.4% regression in thread_times.simple_mobile_sites at 521610:521673

Project Member Reported by ericrk@chromium.org, Jan 3 2018 
See the link to graphs below.
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Jan 3 2018 

All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=798611

(For debugging:) Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?sid=fc4832ce6d5a87358afa72e7f1f26ce2facfb399f223936d2d6dadacb33f3675


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

android-webview-nexus5X
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Jan 3 2018 


=== BISECT JOB RESULTS ===
NO Perf regression found

Bisect Details
  Configuration: android_webview_arm64_aosp_perf_bisect
  Benchmark    : thread_times.simple_mobile_sites
  Metric       : thread_GPU_cpu_time_per_frame/http___m.nytimes.com_

Revision             Result                     N
chromium@521609      0.66908 +- 0.0609783       21      good
chromium@521673      0.663876 +- 0.0735096      21      bad

To Run This Test
  src/tools/perf/run_benchmark -v --browser=android-webview --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=http...m.nytimes.com. thread_times.simple_mobile_sites

More information on addressing performance regressions:
  http://g.co/ChromePerformanceRegressions

Debug information about this bisect:
  https://chromeperf.appspot.com/buildbucket_job_status/8958465853644744912


For feedback, file a bug with component Speed>Bisection

Comment 5 by sullivan@chromium.org, Jan 25 2018 

Bisecting on a wider range.
Project Member

Comment 6 by 42576172...@developer.gserviceaccount.com, Jan 25 2018

Cc: kinuko@chromium.org flackr@chromium.org chrishtr@chromium.org yigu@chromium.org boliu@chromium.org piman@chromium.org
Owner: yigu@chromium.org
Status: Assigned (was: Untriaged)
📍 Found significant differences after each of 2 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/14fa7a5c840000

[PE] Turn on half-width tiles for all platforms and devices.
By chrishtr@chromium.org · Tue Dec 05 01:51:58 2017
chromium @ a3e4477c29d087d3708dbcb4dc5a90985c2b45e3

Round the scroll offset synced back to main thread instead of flooring
By yigu@chromium.org · Tue Dec 05 15:49:54 2017
chromium @ a3ce2d3110a3f9177a41935daadd43ede72db553

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions

Comment 7 by chrishtr@chromium.org, Jan 25 2018

Owner: ericrk@chromium.org

Comment 8 by ericrk@chromium.org, Jan 25 2018

Status: Fixed (was: Assigned)
This was already addressed by https://chromium-review.googlesource.com/858483, and graphs have returned to normal.
►

Sign in to add a comment