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

Issue 804306 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

3.7%-7.9% regression in smoothness.tough_animation_cases at 529527:529720

Project Member Reported by pmeenan@chromium.org, Jan 22 2018

Issue description

See the link to graphs below.
 
Project Member

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

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

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


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

android-nexus6
android-nexus7v2
android-webview-nexus6
Project Member

Comment 2 by 42576172...@developer.gserviceaccount.com, Jan 22 2018

๐Ÿ“ Pinpoint job started.
https://pinpoint-dot-chromeperf.appspot.com/job/12ffa2e8840000
Project Member

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

๐Ÿ“ Pinpoint job started.
https://pinpoint-dot-chromeperf.appspot.com/job/12922b04840000
Kicked off another bisect on one of the really clear regressions.
Project Member

Comment 5 by 42576172...@developer.gserviceaccount.com, Jan 23 2018

Cc: kbr@chromium.org weiliangc@chromium.org cco3@chromium.org enne@chromium.org scheib@chromium.org
Owner: scheib@chromium.org
Status: Assigned (was: Untriaged)
๐Ÿ“ Found significant differences after each of 2 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/12922b04840000

viz: Clamp tex coords of non-AA tile quads
By enne@chromium.org ยท Wed Jan 17 04:47:09 2018
chromium @ 97414c7ab5cfc16a9e672de768754279e4ca5720

bluetooth web platform test readme section for generator.py
By scheib@chromium.org ยท Wed Jan 17 04:48:48 2018
chromium @ 28bedacad094f1a59db6c9632ce79c20ad12ea6e

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

Comment 6 by enne@chromium.org, Jan 23 2018

Owner: enne@chromium.org
This is clearly mine and not Vince's.  However, this is a correctness fix and there's not a lot of alternatives here.
Project Member

Comment 7 by 42576172...@developer.gserviceaccount.com, Jan 23 2018

๐Ÿ˜ฟ Pinpoint job stopped with an error.
https://pinpoint-dot-chromeperf.appspot.com/job/12ffa2e8840000
Components: Internals>GPU>Metrics

Comment 9 by enne@chromium.org, Feb 7 2018

Status: WontFix (was: Assigned)
ChromiumPerf/android-nexus7v2/smoothness.tough_animation_cases / frame_times has recovered, but the other two on nexus 6 have not.

This was a correctness fix however, and there's really not anything else I can do here.  I did a little bit of investigation on the original bug to see if I could find some more cases to turn off this clamping, but nothing panned out.

Sign in to add a comment