CpuProfile: rename HSourcePosition to SourcePosition and move it to compiler.*

CpuProfile: rename HSourcePosition to SourcePosition and move it to compiler.*

Fix CompilationInfo::TraceInlinedFunction argument.
Fix leaked CodeTracer in Isolate

BUG=452067
LOG=n

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

[loislo, 2015-02-17 09:58:52]

loislo@chromium.org changed reviewers:
+ alph@chromium.org, svenpanne@chromium.org, yurys@chromium.org

[loislo, 2015-02-17 09:58:52]

PTAL

It is almost mechanical patch

[alph, 2015-02-17 10:19:50]

lgtm

[Sven Panne, 2015-02-17 11:08:03]

LGTM with a nit and a comment.

https://codereview.chromium.org/928343003/diff/1/src/compiler.h
File src/compiler.h (right):

https://codereview.chromium.org/928343003/diff/1/src/compiler.h#newcode82
src/compiler.h:82: int value_;
What I meant in previous comments is that "int" is problematic here, effectively
reducing the portable field sizes. Consequently, the types of position and
inlining id are wrong, too, they should be unsigned. But then again we have the
problem with the kNoPosition hack, which stuffs in a negative value.

This should be handled in a separate CL.

https://codereview.chromium.org/928343003/diff/1/src/compiler.h#newcode463
src/compiler.h:463: SourcePosition raw_position);
s/raw_position/position/

[loislo, 2015-02-17 12:02:49]

New patchsets have been uploaded after l-g-t-m from
alph@chromium.org,svenpanne@chromium.org

[loislo, 2015-02-17 12:02:57]

The CQ bit was checked by loislo@chromium.org

[commit-bot: I haz the power, 2015-02-17 12:03:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/928343003/20001

[commit-bot: I haz the power, 2015-02-17 12:26:10]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-17 12:26:26]

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