New issue
Advanced search Search tips

Issue 828688 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner: ----
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug
Proj-XR



Sign in to add a comment

Update layout is slow

Project Member Reported by vollick@chromium.org, Apr 4 2018

Issue description

This is also taking quite a bit of the frame budget and it's likely that little is changing frame-to-frame.

 
Components: Internals>VR
Labels: Proj-VR
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 9 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0c10c3c2b5261d746fc4455fe0ffe134cdfb38b6

commit 0c10c3c2b5261d746fc4455fe0ffe134cdfb38b6
Author: Ian Vollick <vollick@chromium.org>
Date: Mon Apr 09 15:47:09 2018

[vr] Prune UI tree walks

This CL cuts unnecessary work from our tree walks where we can.

Bug:  828684 , 828688 , 828687 
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I3706deec5adeae6b4c4d186d45187854e79a840f
Reviewed-on: https://chromium-review.googlesource.com/1000430
Reviewed-by: Christopher Grant <cjgrant@chromium.org>
Commit-Queue: Ian Vollick <vollick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549185}
[modify] https://crrev.com/0c10c3c2b5261d746fc4455fe0ffe134cdfb38b6/chrome/browser/vr/elements/content_element_unittest.cc
[modify] https://crrev.com/0c10c3c2b5261d746fc4455fe0ffe134cdfb38b6/chrome/browser/vr/elements/linear_layout.cc
[modify] https://crrev.com/0c10c3c2b5261d746fc4455fe0ffe134cdfb38b6/chrome/browser/vr/elements/ui_element.cc
[modify] https://crrev.com/0c10c3c2b5261d746fc4455fe0ffe134cdfb38b6/chrome/browser/vr/elements/ui_element.h
[modify] https://crrev.com/0c10c3c2b5261d746fc4455fe0ffe134cdfb38b6/chrome/browser/vr/elements/ui_element_unittest.cc
[modify] https://crrev.com/0c10c3c2b5261d746fc4455fe0ffe134cdfb38b6/chrome/browser/vr/ui_scene.cc
[modify] https://crrev.com/0c10c3c2b5261d746fc4455fe0ffe134cdfb38b6/chrome/browser/vr/ui_scene_creator.cc
[modify] https://crrev.com/0c10c3c2b5261d746fc4455fe0ffe134cdfb38b6/chrome/browser/vr/ui_unittest.cc

Status: Fixed (was: Available)
As per discussion with vollick@, declaring this complete.
Labels: M-67 Test-Complete
Components: Internals>XR

Sign in to add a comment