New issue
Advanced search Search tips

Issue 623976 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

237.3% regression in performance_browser_tests at 402287:402324

Project Member Reported by lanwei@google.com, Jun 28 2016

Issue description

See the link to graphs below.
 

Comment 1 by lanwei@google.com, Jun 28 2016

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

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


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

chromium-rel-win7-x64-dual
Project Member

Comment 2 by sheriffbot@chromium.org, Jun 29 2016

Labels: Hotlist-Google
Project Member

Comment 3 by sheriffbot@chromium.org, Jul 1 2016

Labels: -M-53 M-54 MovedFrom-53
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Trying again with another bisect.
Cc: majidvp@chromium.org enne@chromium.org
The bisect has failed again :(. However this seems to be a regression that follows a previous improvement.

In particular, the improvement happens in range 401756:401809 and regression occurs in 402287:402324. This exactly coincides with landing (401796) and reversion (402294) of "Turn on enable begin frame scheduling by default". 

The patch modifies begin main frame logic which is relevant to this metric that measures FPS. /cc enne@ who authored the patch.

I suggest closing this bug and not sinking more time on it.

Comment 6 by enne@chromium.org, Jul 20 2016

What does -20ms mean on this graph?

Comment 7 by lanwei@chromium.org, Jul 21 2016

alancutter@ is the owner of the test. I do not know what this test is doing. We are just using these benchmarks to measure the Chrome's performance.
Status: Fixed (was: Assigned)
The metric has removed. I think we can close this.
Erm I meant to say: The metric has recovered.

Sign in to add a comment