New issue
Advanced search Search tips

Issue 826818 link

Starred by 1 user
Status: WontFix
Owner:
bsalo...@google.com
Closed: Apr 2018
Cc:
robertph...@google.com
npm@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

48.6%-157.2% regression in system_health.memory_mobile at 545749:545869

Project Member Reported by npm@chromium.org, Mar 28 2018 
See the link to graphs below.
Project Member

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

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

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


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

android-nexus5
android-nexus5X
android-nexus6
android-nexus7v2
chromium-rel-mac11-air
chromium-rel-mac11-pro
chromium-rel-win7-dual
chromium-rel-win7-x64-dual
chromium-rel-win8-dual
linux-release
Project Member

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

Cc: robertph...@google.com
Status: Assigned (was: Untriaged)
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/12fa527b440000

Revert "Use new Skia GPU read/write pixel code paths" by robertphillips@google.com
https://chromium.googlesource.com/chromium/src/+/56044665258df106f9c5dd353ed9662cab28aefd

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

Comment 4 by sullivan@chromium.org, Apr 5 2018

Owner: robertphillips@chromium.org

Comment 5 by robertphillips@chromium.org, Apr 5 2018

Owner: bsalo...@google.com

Comment 6 by robertphillips@chromium.org, Apr 5 2018 

We will be trying to reland "Use new Skia GPU read/write pixel code paths" hopefully with some performance optimizations.

Comment 7 by bsalo...@google.com, Apr 5 2018

Status: WontFix (was: Assigned)
Realistically if do the the performance optimizations we'll also lose the memory benefit.
►

Sign in to add a comment