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

Issue 611936 link

Starred by 1 user
Status: Fixed
Owner:
█ jaydasika@chromium.org
Last visit > 30 days ago
Closed: May 2017
Cc:
weiliangc@chromium.org
█ vollick@chromium.org
ajuma@chromium.org
sunxd@chromium.org
█ jaydasika@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Feature



Sign in to add a comment

Add a property id to tree index map to property trees

Project Member Reported by jaydasika@chromium.org, May 14 2016 
For now, the property id will the owning Layer's id. This map should be used whenever we want to get property tree node from a layer id without de-referencing Layer/LayerImpl.
Project Member

Comment 1 by bugdroid1@chromium.org, May 14 2016 

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

commit fb0572612ff4b8288fbf821faf9632fabe4c55b8
Author: jaydasika <jaydasika@chromium.org>
Date: Sat May 14 03:12:28 2016

cc : Fix bug in LayerToPropertyTreeIndices map

This is a bug that went unnoticed when I wrote
https://codereview.chromium.org/1975623002/

BUG= 611936 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/1983543002
Cr-Commit-Position: refs/heads/master@{#393734}

[modify] https://crrev.com/fb0572612ff4b8288fbf821faf9632fabe4c55b8/cc/trees/layer_tree_host_unittest.cc
[modify] https://crrev.com/fb0572612ff4b8288fbf821faf9632fabe4c55b8/cc/trees/property_tree_builder.cc

Comment 2 by jaydasika@chromium.org, May 13 2017

Status: Fixed (was: Started)
►

Sign in to add a comment