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

Issue 834377 link

Starred by 5 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug
Proj-VR
Proj-XR
Proj-XR-VR



Sign in to add a comment

Disable optimized tree walks

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

Issue description

This was causing too many issues close to branch.

 
This landed in https://chromium-review.googlesource.com/c/chromium/src/+/1011353, but I hadn't created a bug for it.
Cc: cma...@chromium.org
Labels: Merge-Request-67
+cmasso@. I'd failed to make a bug for this CL, but it's quite trivial and not having it in the branch is causing pretty significant issues for our UI. I'm not sure if mentioning the CL here will work with the auto-merge-request bot, but could you please take a look when you have a sec?

Comment 3 by cma...@chromium.org, Apr 18 2018

Labels: -Merge-Request-67 Merge-Approved-67
Labels: Merge-Merged
Status: Fixed (was: Started)
Thanks for quick response! This has been merged.
Cc: dbbrooks@chromium.org
 Issue 834347  has been merged into this issue.
Cc: dougman@chromium.org vsupruniuk@google.com
 Issue 834044  has been merged into this issue.
Project Member

Comment 7 by sheriffbot@chromium.org, Apr 23 2018

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Approved-67
Labels: Test-Complete
Status: Verified (was: Fixed)
Verified fix on build 67.0.3396.16.  The optimization have been disabled.
Specifically there was a fade out/in issue introduced by this change, removing this change removes the issues in the current builds.

Sign in to add a comment