Commit 84b61dd5 authored by Peter Mayo's avatar Peter Mayo Committed by Commit Bot

Clean up some comments in paint_artifact_compositor.

Make them use the same variable name as the code they describe.

Bug: None
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: Id8547edbcc39b591865165bab440165e4d05e7e1
Reviewed-on: https://chromium-review.googlesource.com/1169697
Commit-Queue: Peter Mayo <petermayo@chromium.org>
Reviewed-by: default avatarPeter Mayo <petermayo@chromium.org>
Reviewed-by: default avatarPhilip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583140}
parent ac183c99
...@@ -547,14 +547,14 @@ void PaintArtifactCompositor::LayerizeGroup( ...@@ -547,14 +547,14 @@ void PaintArtifactCompositor::LayerizeGroup(
: subgroup_layer.property_tree_state.Clip(), : subgroup_layer.property_tree_state.Clip(),
&current_group)); &current_group));
} }
// At this point pendingLayers.back() is the either a layer from a // At this point pending_layers.back() is the either a layer from a
// "decomposited" subgroup or a layer created from a chunk we just // "decomposited" subgroup or a layer created from a chunk we just
// processed. Now determine whether it could be merged into a previous // processed. Now determine whether it could be merged into a previous
// layer. // layer.
const PendingLayer& new_layer = pending_layers.back(); const PendingLayer& new_layer = pending_layers.back();
DCHECK(!new_layer.requires_own_layer); DCHECK(!new_layer.requires_own_layer);
DCHECK_EQ(&current_group, new_layer.property_tree_state.Effect()); DCHECK_EQ(&current_group, new_layer.property_tree_state.Effect());
// This iterates pendingLayers[firstLayerInCurrentGroup:-1] in reverse. // This iterates pending_layers[first_layer_in_current_group:-1] in reverse.
for (size_t candidate_index = pending_layers.size() - 1; for (size_t candidate_index = pending_layers.size() - 1;
candidate_index-- > first_layer_in_current_group;) { candidate_index-- > first_layer_in_current_group;) {
PendingLayer& candidate_layer = pending_layers[candidate_index]; PendingLayer& candidate_layer = pending_layers[candidate_index];
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment