headless: Register event handlers on demand

headless: Register event handlers on demand

Change the DevTools C++ bindings to only register event handlers when
the user adds an observer for the events. This allows dead code
elimination to cut about 176 KiB from the Headless Shell application.

BUG=612904
Committed: https://crrev.com/1e5ba5ad472920050c6abae57cc43225bdb5ce1b
Cr-Commit-Position: refs/heads/master@{#429857}

[Sami, 2016-11-03 18:50:39]

Description was changed from

==========
headless: Register event handlers on demand

Change the DevTools C++ bindings to only register event handlers when
the user adds an observer for the events. This allows dead code
elimination to cut about XX KiB from the Headless Shell application.

BUG=612904
==========

to

==========
headless: Register event handlers on demand

Change the DevTools C++ bindings to only register event handlers when
the user adds an observer for the events. This allows dead code
elimination to cut about 176 KiB from the Headless Shell application.

BUG=612904
==========

[Sami, 2016-11-03 18:50:39]

skyostil@chromium.org changed reviewers:
+ altimin@chromium.org

[altimin, 2016-11-04 11:10:47]

lgtm, thanks!

[Sami, 2016-11-04 11:12:16]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2016-11-04 11:12:39]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-04 11:22:20]

Description was changed from

==========
headless: Register event handlers on demand

Change the DevTools C++ bindings to only register event handlers when
the user adds an observer for the events. This allows dead code
elimination to cut about 176 KiB from the Headless Shell application.

BUG=612904
==========

to

==========
headless: Register event handlers on demand

Change the DevTools C++ bindings to only register event handlers when
the user adds an observer for the events. This allows dead code
elimination to cut about 176 KiB from the Headless Shell application.

BUG=612904
==========

[commit-bot: I haz the power, 2016-11-04 11:22:20]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-04 11:24:25]

Description was changed from

==========
headless: Register event handlers on demand

Change the DevTools C++ bindings to only register event handlers when
the user adds an observer for the events. This allows dead code
elimination to cut about 176 KiB from the Headless Shell application.

BUG=612904
==========

to

==========
headless: Register event handlers on demand

Change the DevTools C++ bindings to only register event handlers when
the user adds an observer for the events. This allows dead code
elimination to cut about 176 KiB from the Headless Shell application.

BUG=612904

Committed: https://crrev.com/1e5ba5ad472920050c6abae57cc43225bdb5ce1b
Cr-Commit-Position: refs/heads/master@{#429857}
==========

[commit-bot: I haz the power, 2016-11-04 11:24:26]

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