Add missing virtual destructor to backend callbacks.

Add missing virtual destructor to backend callbacks.

The missing virtual destructors are causing hard to diagnose delete
size mismatch failures if used with tcmalloc, because the superclass
destructors are called instead of the subclass ones. Previously,
they were not necessary because the destructor was declared virtual
through the old BackendCallback base class (which was removed as of
https://codereview.chromium.org/2468923002).

BUG=none
R=kozyatinskiy@chromium.org

Committed: https://chromium.googlesource.com/deps/inspector_protocolebda02bf94a742a2e26e4f818df1fc77517ac44c

[Eric Seckler, 2016-11-04 12:34:53]

eseckler@chromium.org changed reviewers:
+ dgozman@chromium.org, kozyatinskiy@chromium.org

[Eric Seckler, 2016-11-04 12:35:19]

Description was changed from

==========
Add missing virtual constructor to backend callbacks.

The missing destructors are causing hard to diagnose delete size
mismatch failures if used with tcmalloc. Previously, they were not
necessary because the destructor was declared virtual through the old
BackendCallback base class (which was removed as of
https://codereview.chromium.org/2468923002).

BUG=none
==========

to

==========
Add missing virtual destructor to backend callbacks.

The missing destructors are causing hard to diagnose delete size
mismatch failures if used with tcmalloc. Previously, they were not
necessary because the destructor was declared virtual through the old
BackendCallback base class (which was removed as of
https://codereview.chromium.org/2468923002).

BUG=none
==========

[Eric Seckler, 2016-11-04 12:35:37]

On 2016/11/04 12:34:54, Eric Seckler wrote:

Missing _de_structors of course :)

[Eric Seckler, 2016-11-04 12:42:06]

Description was changed from

==========
Add missing virtual destructor to backend callbacks.

The missing destructors are causing hard to diagnose delete size
mismatch failures if used with tcmalloc. Previously, they were not
necessary because the destructor was declared virtual through the old
BackendCallback base class (which was removed as of
https://codereview.chromium.org/2468923002).

BUG=none
==========

to

==========
Add missing virtual destructor to backend callbacks.

The missing virtual destructors are causing hard to diagnose delete
size mismatch failures if used with tcmalloc, because the superclass
destructors are  called instead of the subclass ones. Previously,
they were not necessary because the destructor was declared virtual
through the old BackendCallback base class (which was removed as of
https://codereview.chromium.org/2468923002).

BUG=none
==========

[Eric Seckler, 2016-11-04 12:42:14]

Description was changed from

==========
Add missing virtual destructor to backend callbacks.

The missing virtual destructors are causing hard to diagnose delete
size mismatch failures if used with tcmalloc, because the superclass
destructors are  called instead of the subclass ones. Previously,
they were not necessary because the destructor was declared virtual
through the old BackendCallback base class (which was removed as of
https://codereview.chromium.org/2468923002).

BUG=none
==========

to

==========
Add missing virtual destructor to backend callbacks.

The missing virtual destructors are causing hard to diagnose delete
size mismatch failures if used with tcmalloc, because the superclass
destructors are called instead of the subclass ones. Previously,
they were not necessary because the destructor was declared virtual
through the old BackendCallback base class (which was removed as of
https://codereview.chromium.org/2468923002).

BUG=none
==========

[kozy, 2016-11-04 14:27:56]

lgtm

[Sami, 2016-11-04 14:46:16]

Description was changed from

==========
Add missing virtual destructor to backend callbacks.

The missing virtual destructors are causing hard to diagnose delete
size mismatch failures if used with tcmalloc, because the superclass
destructors are called instead of the subclass ones. Previously,
they were not necessary because the destructor was declared virtual
through the old BackendCallback base class (which was removed as of
https://codereview.chromium.org/2468923002).

BUG=none
==========

to

==========
Add missing virtual destructor to backend callbacks.

The missing virtual destructors are causing hard to diagnose delete
size mismatch failures if used with tcmalloc, because the superclass
destructors are called instead of the subclass ones. Previously,
they were not necessary because the destructor was declared virtual
through the old BackendCallback base class (which was removed as of
https://codereview.chromium.org/2468923002).

BUG=none
R=kozyatinskiy@chromium.org

Committed:
https://chromium.googlesource.com/deps/inspector_protocolebda02bf94a742a2e26e...
==========

[Sami, 2016-11-04 14:46:17]

Committed patchset #2 (id:20001) manually as
ebda02bf94a742a2e26e4f818df1fc77517ac44c (presubmit successful).