Move CallbackBase::polymorphic_invoke_ into BindStateBase

Move CallbackBase::polymorphic_invoke_ into BindStateBase

Each Callback instance had its own polymorphic_invoke, that doubles the
size of Callback, and its initialization gains 40kB of the stripped
binary size of Chrome on Linux.

This CL moves polymorphic_invoke_ into BindStateBase to share it among
Callback.

Committed: https://crrev.com/1886c27824eda740f5b50f858ff987ed557b6166
Cr-Commit-Position: refs/heads/master@{#417197}

[tzik, 2016-09-06 06:15:05]

The CQ bit was checked by tzik@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-06 06:15:23]

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

[tzik, 2016-09-06 06:20:47]

Description was changed from

==========
Move CallbackBase::polymorphic_invoke_ to BindStateBase
==========

to

==========
Move CallbackBase::polymorphic_invoke_ into BindStateBase

Each Callback instance had its own polymorphic_invoke, that doubles the
size of Callback, and its initialization gains 32kB of the stripped
binary size of Chrome on Linux.

This CL moves polymorphic_invoke_ into BindStateBase to share it among
Callback.
==========

[tzik, 2016-09-06 06:20:47]

tzik@chromium.org changed reviewers:
+ dcheng@chromium.org, yutak@chromium.org

[tzik, 2016-09-06 07:38:45]

PTAL

[commit-bot: I haz the power, 2016-09-06 08:11:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-06 08:11:47]

Dry run: This issue passed the CQ dry run.

[tzik, 2016-09-06 12:20:01]

Description was changed from

==========
Move CallbackBase::polymorphic_invoke_ into BindStateBase

Each Callback instance had its own polymorphic_invoke, that doubles the
size of Callback, and its initialization gains 32kB of the stripped
binary size of Chrome on Linux.

This CL moves polymorphic_invoke_ into BindStateBase to share it among
Callback.
==========

to

==========
Move CallbackBase::polymorphic_invoke_ into BindStateBase

Each Callback instance had its own polymorphic_invoke, that doubles the
size of Callback, and its initialization gains 40kB of the stripped
binary size of Chrome on Linux.

This CL moves polymorphic_invoke_ into BindStateBase to share it among
Callback.
==========

[tzik, 2016-09-06 14:19:52]

The CQ bit was checked by tzik@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-06 14:20:08]

Dry run: 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-09-06 16:25:55]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-06 16:25:55]

Dry run: This issue passed the CQ dry run.

[dcheng, 2016-09-06 21:43:10]

lgtm

[Yuta Kitamura, 2016-09-07 08:01:45]

lgtm

[tzik, 2016-09-07 14:40:45]

The CQ bit was checked by tzik@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-07 14:40:59]

Dry run: 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-09-07 18:42:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-07 18:42:52]

Dry run: Exceeded global retry quota

[tzik, 2016-09-08 03:46:30]

The CQ bit was checked by tzik@chromium.org

[tzik, 2016-09-08 03:46:31]

The patchset sent to the CQ was uploaded after l-g-t-m from dcheng@chromium.org,
yutak@chromium.org
Link to the patchset: https://codereview.chromium.org/2317563002/#ps80001
(title: "rebase")

[commit-bot: I haz the power, 2016-09-08 03:46:52]

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-09-08 05:45:56]

Description was changed from

==========
Move CallbackBase::polymorphic_invoke_ into BindStateBase

Each Callback instance had its own polymorphic_invoke, that doubles the
size of Callback, and its initialization gains 40kB of the stripped
binary size of Chrome on Linux.

This CL moves polymorphic_invoke_ into BindStateBase to share it among
Callback.
==========

to

==========
Move CallbackBase::polymorphic_invoke_ into BindStateBase

Each Callback instance had its own polymorphic_invoke, that doubles the
size of Callback, and its initialization gains 40kB of the stripped
binary size of Chrome on Linux.

This CL moves polymorphic_invoke_ into BindStateBase to share it among
Callback.
==========

[commit-bot: I haz the power, 2016-09-08 05:45:57]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-09-08 05:48:58]

Description was changed from

==========
Move CallbackBase::polymorphic_invoke_ into BindStateBase

Each Callback instance had its own polymorphic_invoke, that doubles the
size of Callback, and its initialization gains 40kB of the stripped
binary size of Chrome on Linux.

This CL moves polymorphic_invoke_ into BindStateBase to share it among
Callback.
==========

to

==========
Move CallbackBase::polymorphic_invoke_ into BindStateBase

Each Callback instance had its own polymorphic_invoke, that doubles the
size of Callback, and its initialization gains 40kB of the stripped
binary size of Chrome on Linux.

This CL moves polymorphic_invoke_ into BindStateBase to share it among
Callback.

Committed: https://crrev.com/1886c27824eda740f5b50f858ff987ed557b6166
Cr-Commit-Position: refs/heads/master@{#417197}
==========

[commit-bot: I haz the power, 2016-09-08 05:48:59]

Patchset 5 (id:??) landed as
https://crrev.com/1886c27824eda740f5b50f858ff987ed557b6166
Cr-Commit-Position: refs/heads/master@{#417197}