Revert of [tracing] Turn StackFrame into struct.

Revert of [tracing] Turn StackFrame into struct. (patchset #5 id:80001 of https://codereview.chromium.org/1891543003/ )

Reason for revert:
Broke build on Windows x64 (warning was treated as error): heap_profiler_allocation_context.cc(58): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data

Original issue's description:
> [tracing] Turn StackFrame into struct.
>
> This change turns StackFrame (aka const char*) into a struct and
> introduces 'type' field which controls how stack frame is formatted
> when it's written to trace file. As an example, thread name, which
> previously was just a string like any other function name, is now
> formatted as '[Thread: %s]'.
>
> More stack frame types will be added in the future, for example
> native allocation tracing will add 'program counter' type.
>
> BUG=602701
TBR=primiano@chromium.org,ssid@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=602701
Committed: https://crrev.com/7e308e6f283b287ad0e2229e8d8adb865a7542b6
Cr-Commit-Position: refs/heads/master@{#388558}

[Dmitry Skiba, 2016-04-20 19:55:45]

Created Revert of [tracing] Turn StackFrame into struct.

[commit-bot: I haz the power, 2016-04-20 19:56:05]

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

[commit-bot: I haz the power, 2016-04-20 19:56:05]

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

[commit-bot: I haz the power, 2016-04-20 19:56:06]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Primiano Tucci (use gerrit), 2016-04-20 19:58:43]

lgtm

[Primiano Tucci (use gerrit), 2016-04-20 19:59:17]

The CQ bit was checked by primiano@chromium.org

[commit-bot: I haz the power, 2016-04-20 19:59:32]

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

[commit-bot: I haz the power, 2016-04-20 20:00:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 19:25:46]

Description was changed from

==========
Revert of [tracing] Turn StackFrame into struct. (patchset #5 id:80001 of
https://codereview.chromium.org/1891543003/ )

Reason for revert:
Broke build on Windows x64 (warning was treated as error):
heap_profiler_allocation_context.cc(58): warning C4267: 'argument': conversion
from 'size_t' to 'int', possible loss of data

Original issue's description:
> [tracing] Turn StackFrame into struct.
> 
> This change turns StackFrame (aka const char*) into a struct and
> introduces 'type' field which controls how stack frame is formatted
> when it's written to trace file. As an example, thread name, which
> previously was just a string like any other function name, is now
> formatted as '[Thread: %s]'.
> 
> More stack frame types will be added in the future, for example
> native allocation tracing will add 'program counter' type.
> 
> BUG=602701

TBR=primiano@chromium.org,ssid@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=602701
==========

to

==========
Revert of [tracing] Turn StackFrame into struct. (patchset #5 id:80001 of
https://codereview.chromium.org/1891543003/ )

Reason for revert:
Broke build on Windows x64 (warning was treated as error):
heap_profiler_allocation_context.cc(58): warning C4267: 'argument': conversion
from 'size_t' to 'int', possible loss of data

Original issue's description:
> [tracing] Turn StackFrame into struct.
>
> This change turns StackFrame (aka const char*) into a struct and
> introduces 'type' field which controls how stack frame is formatted
> when it's written to trace file. As an example, thread name, which
> previously was just a string like any other function name, is now
> formatted as '[Thread: %s]'.
>
> More stack frame types will be added in the future, for example
> native allocation tracing will add 'program counter' type.
>
> BUG=602701

TBR=primiano@chromium.org,ssid@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=602701

Committed: https://crrev.com/7e308e6f283b287ad0e2229e8d8adb865a7542b6
Cr-Commit-Position: refs/heads/master@{#388558}
==========

[commit-bot: I haz the power, 2016-04-22 19:25:47]

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