Remove V8.External traces due to its high overhead

Remove V8.External traces due to its high overhead

The Tracing split CL https://codereview.chromium.org/1707563002 mostly moved the location of
the TRACE call, but it added 2 very high frequency calls related to tracking V8.External.

In most benchmark and devices the added overhead is negligible except on N6 where it gets amplified.
This CL removes those 2 calls until more efficient tracing or a different technique is used.

BUG=588137
LOG=N

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

[fmeawad, 2016-03-03 23:37:12]

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

[commit-bot: I haz the power, 2016-03-03 23:37:20]

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

[fmeawad, 2016-03-03 23:37:39]

fmeawad@chromium.org changed reviewers:
+ jochen@chromium.org

[fmeawad, 2016-03-03 23:37:39]

PTAL

[fmeawad, 2016-03-03 23:39:59]

Description was changed from

==========
Remove V8.External traces due to its high overhead

The Tracing split CL https://codereview.chromium.org/1707563002 mostly moved the
location of
the TRACE call, but it added 2 very high frequency calls related to tracking
V8.External.

In most benchmark and devices the added overhead is negligible except on N6
where it gets amplified.
This CL removes those 2 calls until more efficient tracing or a different
technique is used.

BUG=588137
==========

to

==========
Remove V8.External traces due to its high overhead

The Tracing split CL https://codereview.chromium.org/1707563002 mostly moved the
location of
the TRACE call, but it added 2 very high frequency calls related to tracking
V8.External.

In most benchmark and devices the added overhead is negligible except on N6
where it gets amplified.
This CL removes those 2 calls until more efficient tracing or a different
technique is used.

BUG=588137
LOG=N
==========

[fmeawad, 2016-03-03 23:40:07]

The CQ bit was unchecked by fmeawad@chromium.org

[fmeawad, 2016-03-03 23:40:12]

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

[commit-bot: I haz the power, 2016-03-03 23:40:23]

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

[commit-bot: I haz the power, 2016-03-03 23:57:33]

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

[commit-bot: I haz the power, 2016-03-03 23:57:34]

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2016-03-04 12:51:05]

lgtm

[fmeawad, 2016-03-04 14:50:30]

The CQ bit was checked by fmeawad@chromium.org

[commit-bot: I haz the power, 2016-03-04 14:50:40]

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

[commit-bot: I haz the power, 2016-03-04 14:52:58]

Description was changed from

==========
Remove V8.External traces due to its high overhead

The Tracing split CL https://codereview.chromium.org/1707563002 mostly moved the
location of
the TRACE call, but it added 2 very high frequency calls related to tracking
V8.External.

In most benchmark and devices the added overhead is negligible except on N6
where it gets amplified.
This CL removes those 2 calls until more efficient tracing or a different
technique is used.

BUG=588137
LOG=N
==========

to

==========
Remove V8.External traces due to its high overhead

The Tracing split CL https://codereview.chromium.org/1707563002 mostly moved the
location of
the TRACE call, but it added 2 very high frequency calls related to tracking
V8.External.

In most benchmark and devices the added overhead is negligible except on N6
where it gets amplified.
This CL removes those 2 calls until more efficient tracing or a different
technique is used.

BUG=588137
LOG=N
==========

[commit-bot: I haz the power, 2016-03-04 14:52:59]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-04 14:54:25]

Description was changed from

==========
Remove V8.External traces due to its high overhead

The Tracing split CL https://codereview.chromium.org/1707563002 mostly moved the
location of
the TRACE call, but it added 2 very high frequency calls related to tracking
V8.External.

In most benchmark and devices the added overhead is negligible except on N6
where it gets amplified.
This CL removes those 2 calls until more efficient tracing or a different
technique is used.

BUG=588137
LOG=N
==========

to

==========
Remove V8.External traces due to its high overhead

The Tracing split CL https://codereview.chromium.org/1707563002 mostly moved the
location of
the TRACE call, but it added 2 very high frequency calls related to tracking
V8.External.

In most benchmark and devices the added overhead is negligible except on N6
where it gets amplified.
This CL removes those 2 calls until more efficient tracing or a different
technique is used.

BUG=588137
LOG=N

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

[commit-bot: I haz the power, 2016-03-04 14:54:26]

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