cc: Clear the |layer_id_to_scroll_offset_map_| during ScrollTree deserialization .

cc: Clear the |layer_id_to_scroll_offset_map_| during ScrollTree deserialization.

This should be cleared when a new property tree is created in the
LayerTreeHost during deserialization but the ScrollTree copy constructor
retains this map.

BUG=603271
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/d0b61b1bed0afc451d7bb724ed9f27ec5e7205bc
Cr-Commit-Position: refs/heads/master@{#387743}

[Khushal, 2016-04-15 19:24:33]

Description was changed from

==========
cc: Clear the |layer_id_to_scroll_offset_map_| during ScrollTree .

This should be cleared when a new property tree is created in the
LayerTreeHost during deserialization but the ScrollTree copy constructor
retains this map.

BUG=603271
==========

to

==========
cc: Clear the |layer_id_to_scroll_offset_map_| during ScrollTree .

This should be cleared when a new property tree is created in the
LayerTreeHost during deserialization but the ScrollTree copy constructor
retains this map.

BUG=603271
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[Khushal, 2016-04-15 19:24:50]

Description was changed from

==========
cc: Clear the |layer_id_to_scroll_offset_map_| during ScrollTree .

This should be cleared when a new property tree is created in the
LayerTreeHost during deserialization but the ScrollTree copy constructor
retains this map.

BUG=603271
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Clear the |layer_id_to_scroll_offset_map_| during ScrollTree .

This should be cleared when a new property tree is created in the
LayerTreeHost during deserialization but the ScrollTree copy constructor
retains this map.

BUG=603271
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[Khushal, 2016-04-15 19:25:03]

Description was changed from

==========
cc: Clear the |layer_id_to_scroll_offset_map_| during ScrollTree .

This should be cleared when a new property tree is created in the
LayerTreeHost during deserialization but the ScrollTree copy constructor
retains this map.

BUG=603271
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Clear the |layer_id_to_scroll_offset_map_| during ScrollTree
deserialization.

This should be cleared when a new property tree is created in the
LayerTreeHost during deserialization but the ScrollTree copy constructor
retains this map.

BUG=603271
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[Khushal, 2016-04-15 19:25:15]

khushalsagar@chromium.org changed reviewers:
+ vmpstr@chromium.org

[vmpstr, 2016-04-15 20:34:47]

test?

[Khushal, 2016-04-15 21:35:40]

On 2016/04/15 20:34:47, vmpstr wrote:
> test?

Done.

[vmpstr, 2016-04-15 21:39:16]

lgtm

https://codereview.chromium.org/1886973005/diff/20001/cc/trees/property_tree_...
File cc/trees/property_tree_unittest.cc (right):

https://codereview.chromium.org/1886973005/diff/20001/cc/trees/property_tree_...
cc/trees/property_tree_unittest.cc:282: 
nit: can you add an EXPECT_EQ(original, result) here as well to make sure you're
still testing the previous test?

[Khushal, 2016-04-15 21:52:39]

Thanks!

https://codereview.chromium.org/1886973005/diff/20001/cc/trees/property_tree_...
File cc/trees/property_tree_unittest.cc (right):

https://codereview.chromium.org/1886973005/diff/20001/cc/trees/property_tree_...
cc/trees/property_tree_unittest.cc:282: 
On 2016/04/15 21:39:15, vmpstr wrote:
> nit: can you add an EXPECT_EQ(original, result) here as well to make sure
you're
> still testing the previous test?

Done.

[Khushal, 2016-04-15 21:52:43]

The CQ bit was checked by khushalsagar@chromium.org

[Khushal, 2016-04-15 21:52:44]

The patchset sent to the CQ was uploaded after l-g-t-m from vmpstr@chromium.org
Link to the patchset: https://codereview.chromium.org/1886973005/#ps40001
(title: "Add back previous test expectation.")

[commit-bot: I haz the power, 2016-04-15 21:53:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1886973005/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1886973005/40001

[commit-bot: I haz the power, 2016-04-15 23:20:35]

Description was changed from

==========
cc: Clear the |layer_id_to_scroll_offset_map_| during ScrollTree
deserialization.

This should be cleared when a new property tree is created in the
LayerTreeHost during deserialization but the ScrollTree copy constructor
retains this map.

BUG=603271
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Clear the |layer_id_to_scroll_offset_map_| during ScrollTree
deserialization.

This should be cleared when a new property tree is created in the
LayerTreeHost during deserialization but the ScrollTree copy constructor
retains this map.

BUG=603271
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[commit-bot: I haz the power, 2016-04-15 23:20:36]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-04-15 23:21:47]

Description was changed from

==========
cc: Clear the |layer_id_to_scroll_offset_map_| during ScrollTree
deserialization.

This should be cleared when a new property tree is created in the
LayerTreeHost during deserialization but the ScrollTree copy constructor
retains this map.

BUG=603271
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Clear the |layer_id_to_scroll_offset_map_| during ScrollTree
deserialization.

This should be cleared when a new property tree is created in the
LayerTreeHost during deserialization but the ScrollTree copy constructor
retains this map.

BUG=603271
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/d0b61b1bed0afc451d7bb724ed9f27ec5e7205bc
Cr-Commit-Position: refs/heads/master@{#387743}
==========

[commit-bot: I haz the power, 2016-04-15 23:21:47]

Patchset 3 (id:??) landed as
https://crrev.com/d0b61b1bed0afc451d7bb724ed9f27ec5e7205bc
Cr-Commit-Position: refs/heads/master@{#387743}