Chromium Code Reviews| Index: cc/trees/layer_tree_impl.cc |
| diff --git a/cc/trees/layer_tree_impl.cc b/cc/trees/layer_tree_impl.cc |
| index 8080c57afb3446bd07f2ac00ff77b4758ea965c5..78dd4cc6f552e54575a551529b20827f0e9a3595 100644 |
| --- a/cc/trees/layer_tree_impl.cc |
| +++ b/cc/trees/layer_tree_impl.cc |
| @@ -380,7 +380,7 @@ void LayerTreeImpl::PushPropertiesTo(LayerTreeImpl* target_tree) { |
| // To maintain the current scrolling node we need to use element ids which |
| // are stable across the property tree update in SetPropertyTrees. |
| - ElementId scrolling_element_id; |
| + ElementId scrolling_element_id = 0; |
|
wkorman
2017/05/05 20:06:10
kInvalidElementId here?
chrishtr
2017/05/05 20:46:09
Done.
|
| if (ScrollNode* scrolling_node = target_tree->CurrentlyScrollingNode()) |
| scrolling_element_id = scrolling_node->element_id; |
| @@ -536,8 +536,8 @@ void LayerTreeImpl::AddToElementMap(LayerImpl* layer) { |
| return; |
| TRACE_EVENT2(TRACE_DISABLED_BY_DEFAULT("compositor-worker"), |
| - "LayerTreeImpl::AddToElementMap", "element", |
| - element_id.AsValue().release(), "layer_id", layer->id()); |
| + "LayerTreeImpl::AddToElementMap", "element", element_id, |
| + "layer_id", layer->id()); |
| #if DCHECK_IS_ON() |
| LayerImpl* existing_layer = LayerByElementId(element_id); |
| @@ -564,8 +564,7 @@ void LayerTreeImpl::RemoveFromElementMap(LayerImpl* layer) { |
| TRACE_EVENT2(TRACE_DISABLED_BY_DEFAULT("compositor-worker"), |
| "LayerTreeImpl::RemoveFromElementMap", "element", |
| - layer->element_id().AsValue().release(), "layer_id", |
| - layer->id()); |
| + layer->element_id(), "layer_id", layer->id()); |
| layer_tree_host_impl_->mutator_host()->UnregisterElement( |
| layer->element_id(), |
| @@ -985,7 +984,7 @@ void LayerTreeImpl::ClearViewportLayers() { |
| // For unit tests, we use the layer's id as its element id. |
| static void SetElementIdForTesting(LayerImpl* layer) { |
| - layer->SetElementId(LayerIdToElementIdForTesting(layer->id())); |
| + layer->SetElementId(layer->id()); |
| } |
| void LayerTreeImpl::SetElementIdsForTesting() { |