New issue
Advanced search Search tips

Issue 832368 link

Starred by 2 users
Status: Available
Owner: ----
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug-Regression
Proj-XR



Sign in to add a comment

1.5%-173.6% regression in xr.browsing.wpr.static at 549885:549964

Project Member Reported by bsheedy@google.com, Apr 12 2018 
Bisect points to https://chromium-review.googlesource.com/999802. Not familiar with the CL, but looks like we traded performance for a larger memory footprint?
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Apr 12 2018 

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

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


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

pixel_xl

Comment 2 by bsheedy@chromium.org, Apr 12 2018

Components: Internals>VR
Labels: Proj-VR VR-Perf

Comment 3 by mthiesse@chromium.org, Apr 13 2018 

Yeah, I expected some tiny memory impact, but not this much. What's confusing to me is why this wasn't a constant absolute delta increase. It really shouldn't depend on the page at all.

Also, the memory impact should be very short-lived when changing the size of the content rect. Maybe we're picking up on the short-lived size increase flakily or something...

Comment 4 by mthiesse@chromium.org, Apr 13 2018

Labels: -Pri-2 -M-67 M-68 OS-Android Pri-1
I think the tradeoff is worth it for 67, and we should try to do better for 68.

Comment 5 by mthiesse@chromium.org, May 24 2018

Labels: -M-68 M-69

Comment 6 by btebbs@chromium.org, Jul 4

Components: Internals>XR

Comment 7 by mthiesse@chromium.org, Jul 20

Labels: -M-69 Target-70

Comment 8 by efoo@chromium.org, Aug 7 

Removing Internals>VR component and assigning to Internals>XR

Comment 9 by efoo@chromium.org, Aug 7

Components: -Internals>VR

Comment 10 by mthiesse@chromium.org, Nov 7

Owner: ----
Status: Available (was: Assigned)
►

Sign in to add a comment