cc: Make property tree destructor and clear non-virtual.

cc: Make property tree destructor and clear non-virtual.

These are never invoked through a base class pointer, so there is no apparent
need for virtual dispatch here.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

[jbroman, 2016-05-03 00:04:20]

Description was changed from

==========
cc: Make property tree destructor and clear non-virtual.

These are never invoked through a base class pointer, so there is no apparent
need for virtual dispatch here.
==========

to

==========
cc: Make property tree destructor and clear non-virtual.

These are never invoked through a base class pointer, so there is no apparent
need for virtual dispatch here.
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[jbroman, 2016-05-03 13:38:20]

Description was changed from

==========
cc: Make property tree destructor and clear non-virtual.

These are never invoked through a base class pointer, so there is no apparent
need for virtual dispatch here.
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Make property tree destructor and clear non-virtual.

These are never invoked through a base class pointer, so there is no apparent
need for virtual dispatch here.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[jbroman, 2016-05-03 13:52:28]

jbroman@chromium.org changed reviewers:
+ ajuma@chromium.org

[ajuma, 2016-05-03 14:08:23]

lgtm, thanks for cleaning this up.

[jbroman, 2016-05-03 14:08:36]

The CQ bit was checked by jbroman@chromium.org

[commit-bot: I haz the power, 2016-05-03 14:08:57]

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

[commit-bot: I haz the power, 2016-05-03 14:44:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-03 14:45:28]

Description was changed from

==========
cc: Make property tree destructor and clear non-virtual.

These are never invoked through a base class pointer, so there is no apparent
need for virtual dispatch here.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Make property tree destructor and clear non-virtual.

These are never invoked through a base class pointer, so there is no apparent
need for virtual dispatch here.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

[commit-bot: I haz the power, 2016-05-03 14:45:29]

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

[esprehn, 2016-05-04 02:54:04]

If they're final the compiler will just do this for you, using virtual means you
get correctness when someone adds the call later. :)