binding: Reduces the binary size of binding's generated code.

binding: Reduces the binary size of binding's generated code.

Retires installConditionallyEnabledProperties.  This function has been
unused for months at least, and it's now unlikely we'll need this
mechanism again.

Stops generating unused functions of preparePrototypeAndInterfaceObject.
Even when we don't use them, each function requires an unique address,
i.e. each function consumes 8 bytes.  Uses nullptr instead of generating
unused functions.

This CL is expected to reduce the size by 400k+ bytes.

BUG=621802
Committed: https://crrev.com/9432327e5f7048159fec797972dbb04808664213
Cr-Commit-Position: refs/heads/master@{#415263}

[Yuki, 2016-08-30 07:12:34]

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

[commit-bot: I haz the power, 2016-08-30 07:12:45]

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

[Yuki, 2016-08-30 07:13:38]

Description was changed from

==========
binding: Reduces the binary size of binding's generated code.

Retires installConditionallyEnabledProperties.  This function has been
unused for months at least, and it's now unlikely we'll need this
mechanism again.

Stops generating unused functions of preparePrototypeAndInterfaceObject.
Even when we don't use them, each function requires an unique address,
i.e. each function consumes 8 bytes.  Uses nullptr instead of generating
unused functions.

BUG=621802
==========

to

==========
binding: Reduces the binary size of binding's generated code.

Retires installConditionallyEnabledProperties.  This function has been
unused for months at least, and it's now unlikely we'll need this
mechanism again.

Stops generating unused functions of preparePrototypeAndInterfaceObject.
Even when we don't use them, each function requires an unique address,
i.e. each function consumes 8 bytes.  Uses nullptr instead of generating
unused functions.

This CL is expected to reduce the size by 400k+ bytes.

BUG=621802
==========

[Yuki, 2016-08-30 07:49:15]

yukishiino@chromium.org changed reviewers:
+ haraken@chromium.org

[Yuki, 2016-08-30 07:49:16]

Could you review this CL?

[haraken, 2016-08-30 09:17:26]

Looks nice! LGTM

[Yuki, 2016-08-30 10:05:49]

The CQ bit was unchecked by yukishiino@chromium.org

[Yuki, 2016-08-30 10:05:57]

The CQ bit was checked by yukishiino@chromium.org

[commit-bot: I haz the power, 2016-08-30 10:06:19]

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-08-30 10:25:21]

Description was changed from

==========
binding: Reduces the binary size of binding's generated code.

Retires installConditionallyEnabledProperties.  This function has been
unused for months at least, and it's now unlikely we'll need this
mechanism again.

Stops generating unused functions of preparePrototypeAndInterfaceObject.
Even when we don't use them, each function requires an unique address,
i.e. each function consumes 8 bytes.  Uses nullptr instead of generating
unused functions.

This CL is expected to reduce the size by 400k+ bytes.

BUG=621802
==========

to

==========
binding: Reduces the binary size of binding's generated code.

Retires installConditionallyEnabledProperties.  This function has been
unused for months at least, and it's now unlikely we'll need this
mechanism again.

Stops generating unused functions of preparePrototypeAndInterfaceObject.
Even when we don't use them, each function requires an unique address,
i.e. each function consumes 8 bytes.  Uses nullptr instead of generating
unused functions.

This CL is expected to reduce the size by 400k+ bytes.

BUG=621802
==========

[commit-bot: I haz the power, 2016-08-30 10:25:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-30 10:27:42]

Description was changed from

==========
binding: Reduces the binary size of binding's generated code.

Retires installConditionallyEnabledProperties.  This function has been
unused for months at least, and it's now unlikely we'll need this
mechanism again.

Stops generating unused functions of preparePrototypeAndInterfaceObject.
Even when we don't use them, each function requires an unique address,
i.e. each function consumes 8 bytes.  Uses nullptr instead of generating
unused functions.

This CL is expected to reduce the size by 400k+ bytes.

BUG=621802
==========

to

==========
binding: Reduces the binary size of binding's generated code.

Retires installConditionallyEnabledProperties.  This function has been
unused for months at least, and it's now unlikely we'll need this
mechanism again.

Stops generating unused functions of preparePrototypeAndInterfaceObject.
Even when we don't use them, each function requires an unique address,
i.e. each function consumes 8 bytes.  Uses nullptr instead of generating
unused functions.

This CL is expected to reduce the size by 400k+ bytes.

BUG=621802

Committed: https://crrev.com/9432327e5f7048159fec797972dbb04808664213
Cr-Commit-Position: refs/heads/master@{#415263}
==========

[commit-bot: I haz the power, 2016-08-30 10:27:43]

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

[haraken, 2016-09-05 01:43:08]

Has this change really reduced the binary size? I cannot see any decrease on the
perf bots:

https://chromeperf.appspot.com/report?sid=53f0e5fe1f02dae85645c1f8eafac4a57a7...

[Yuki, 2016-09-05 05:55:29]

On 2016/09/05 01:43:08, haraken wrote:
> Has this change really reduced the binary size? I cannot see any decrease on
the
> perf bots:
> 
>
https://chromeperf.appspot.com/report?sid=53f0e5fe1f02dae85645c1f8eafac4a57a7...

As I commented on the bug, I don't see much improvement, either.

If it's component build, the size of each *.so reduces.  However, if it's not
component build, probably the linker has been doing a very good job so that we
don't observe an improvement.