New issue
Advanced search Search tips

Issue 889166 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Sep 26
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

2.7%-10.2% regression in rendering.mobile/avg_surface_fps at 591987:592071

Project Member Reported by lanwei@chromium.org, Sep 25

Issue description

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

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


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

Android Nexus5 Perf
Android Nexus6 WebView Perf

blink_perf.svg - Benchmark documentation link:
  https://bit.ly/blink-perf-benchmarks

rendering.mobile - Benchmark documentation link:
  https://bit.ly/rendering-benchmarks
Cc: herb@google.com mtkl...@google.com
Owner: herb@google.com
Status: Assigned (was: Untriaged)
📍 Found significant differences after each of 2 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/16657f7b640000

implement SkPixmap::erase() with draws by mtklein@google.com
https://skia.googlesource.com/skia/+/42f29f11817f9829b92dbf52d77f1447fbfc5ed0
1133 → 1148 (+14.76)

Make atomic lists list for bitmaps and paths by herb@google.com
https://skia.googlesource.com/skia/+/6c8d242b14355bf66c9137e9e4d6c7861d22168f
1152 → 1250 (+97.67)

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

Benchmark documentation link:
  https://bit.ly/blink-perf-benchmarks
Status: WontFix (was: Assigned)
"implement SkPixmap::erase() with draws" has been in and out but is out now.
"Make atomic lists list for bitmaps and paths" is a bug fix.

I think this is WAI.

Sign in to add a comment