Remove sampling trace events from the binding layer

Remove sampling trace events from the binding layer

This CL decreases the binary size by 600 KB by removing the sampling trace events
from the binding layer. As far as I know, the sampling profiler is not widely used recently,
so it wouldn't be a big deal to remove them. Also if we really want to have
the sampling events in the binding layer, we should add the events to the V8 side
(c.f., now V8 supports tracing) and thus avoid bloating the binary size by adding
the events to each C++ callback of DOM attribute/method/constant etc.

BUG=

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

[haraken, 2016-02-08 05:15:47]

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

[haraken, 2016-02-08 05:15:48]

PTAL

[Yuki, 2016-02-08 05:47:55]

LGTM

Now then, we don't always need Xxx{Getter,Setter,Method}Callback function in
addition to Xxx{Getter,Setter,Method}.  We can remove one indirection call.

[haraken, 2016-02-08 05:50:45]

> Now then, we don't always need Xxx{Getter,Setter,Method}Callback function in
> addition to Xxx{Getter,Setter,Method}.  We can remove one indirection call.

The indirection has been useful to hook all DOM callbacks. One example is the
tracing macro. Another example would be various local experiments. However, if
it contributes to the binary size reduction, it's worth removing (I hope that
compilers are smart enough to inline those function calls).

[haraken, 2016-02-08 05:50:48]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-02-08 05:51:00]

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

[commit-bot: I haz the power, 2016-02-08 08:58:32]

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

[commit-bot: I haz the power, 2016-02-08 08:58:33]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[haraken, 2016-02-08 09:00:02]

The CQ bit was checked by haraken@chromium.org

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

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

[commit-bot: I haz the power, 2016-02-08 12:01:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-08 12:02:22]

Description was changed from

==========
Remove sampling trace events from the binding layer

This CL decreases the binary size by 600 KB by removing the sampling trace
events
from the binding layer. As far as I know, the sampling profiler is not widely
used recently,
so it wouldn't be a big deal to remove them. Also if we really want to have
the sampling events in the binding layer, we should add the events to the V8
side
(c.f., now V8 supports tracing) and thus avoid bloating the binary size by
adding
the events to each C++ callback of DOM attribute/method/constant etc.

BUG=
==========

to

==========
Remove sampling trace events from the binding layer

This CL decreases the binary size by 600 KB by removing the sampling trace
events
from the binding layer. As far as I know, the sampling profiler is not widely
used recently,
so it wouldn't be a big deal to remove them. Also if we really want to have
the sampling events in the binding layer, we should add the events to the V8
side
(c.f., now V8 supports tracing) and thus avoid bloating the binary size by
adding
the events to each C++ callback of DOM attribute/method/constant etc.

BUG=

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

[commit-bot: I haz the power, 2016-02-08 12:02:23]

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