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

Issue 816521 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

9.3% regression in performance_browser_tests at 536044:536534

Project Member Reported by tdres...@chromium.org, Feb 26 2018

Issue description

See the link to graphs below.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Feb 26 2018

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

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


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

chromium-rel-mac11-pro
Kicking off a few more bisects.
Project Member

Comment 6 by 42576172...@developer.gserviceaccount.com, Mar 19 2018

Cc: x...@chromium.org roc...@chromium.org m...@chromium.org
Owner: m...@chromium.org
Status: Assigned (was: Untriaged)
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/11f2505e440000

InterprocessFramePool rewritten to also work with ChromeOS sandbox. by miu@chromium.org
https://chromium.googlesource.com/chromium/src/+/aa3b9baaee9e212ae25390143d51a46920a2e380

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

Comment 7 by m...@chromium.org, Mar 20 2018

Status: WontFix (was: Assigned)
No regression because the 0.4 ms shifted from one "stage" in the pipeline to another: We see a 0.4 ms drop in "send_to_renderer" at the same time the 0.4 ms increase in "encode" happens. Also, "total_latency," which encompasses all stages has remained unchanged.

In fact, it was a performance improvement because another stage, "capture_duration," saw a 0.75 ms improvement! So, the new buffer pool is more performant overall.
Project Member

Comment 8 by 42576172...@developer.gserviceaccount.com, Mar 22 2018

Status: Assigned (was: WontFix)
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/15e205be440000

InterprocessFramePool rewritten to also work with ChromeOS sandbox. by miu@chromium.org
https://chromium.googlesource.com/chromium/src/+/aa3b9baaee9e212ae25390143d51a46920a2e380

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

Comment 9 by m...@chromium.org, Mar 23 2018

Status: WontFix (was: Assigned)
I meant it last time. :)

Sign in to add a comment