New issue
Advanced search Search tips

Issue 821382 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Mar 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

3.4% regression in smoothness.gpu_rasterization.tough_filters_cases at 539204:539302

Project Member Reported by hjd@google.com, Mar 13 2018

Issue description

See the link to graphs below.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Mar 13 2018

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

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


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

android-one
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Mar 13 2018

Cc: kenrb@chromium.org enne@chromium.org ericrk@chromium.org
Owner: ericrk@chromium.org
Status: Assigned (was: Untriaged)
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/1484eeee440000

Revert "Use glUnpremultiplyAndDitherCopyCHROMIUM in GPU Raster" by ericrk@chromium.org
https://chromium.googlesource.com/chromium/src/+/bc529af85b41acf66ed90e53929a246c10f998bd

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

Comment 4 by ericrk@chromium.org, Mar 16 2018

Status: Fixed (was: Assigned)
We're making a conscious decision with this CL to improve visual quality at a slight performance cost. 3.4% (on one test case) seems acceptable. We'll assess more if we see larger perf hits or more wide spread ones.

Comment 5 by ericrk@chromium.org, Mar 16 2018

Status: WontFix (was: Fixed)

Sign in to add a comment