Revert of Include file names in trace_turbo output

Revert of Include file names in trace_turbo output (patchset #3 id:40001 of https://codereview.chromium.org/2083863004/ )

Reason for revert:
Many build bots are failing with a message of the form:

Missing or invalid v8 JSON file: /tmp/tmp2qcEUy_swarming/0/output.json

Can be relanded once we understand why these failures are occuring.

Original issue's description:
> Include file names in trace_turbo output
>
> The trace turbo output will overwrite itself when functions in different
> files share the same name. Output files now have the form
> `turbo-<function_name>:<opt_file_name>-<opt_phase>.suffix`.
>
> R=ofrobots@google.com
> BUG=
>
> Committed: https://crrev.com/a53b9bf02f31e5647c37e0392afa19f74df1a3ba
> Cr-Commit-Position: refs/heads/master@{#37199}

TBR=ofrobots@google.com,bmeurer@chromium.org,danno@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=

Committed: https://crrev.com/97c2bc362f234bd58515a0faf6af23b4f8ad183a
Cr-Commit-Position: refs/heads/master@{#37204}

[mattloring, 2016-06-22 20:46:13]

The CQ bit was checked by mattloring@google.com

[mattloring, 2016-06-22 20:46:13]

Created Revert of Include file names in trace_turbo output

[commit-bot: I haz the power, 2016-06-22 20:46:25]

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

[commit-bot: I haz the power, 2016-06-22 20:46:26]

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

[commit-bot: I haz the power, 2016-06-22 20:46:27]

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.
Committers are members of the group "project-v8-committers".
Note that this has nothing to do with OWNERS files.

[mattloring, 2016-06-22 20:53:57]

The error messages from the build bots were of the form:

Missing or invalid v8 JSON file: /tmp/tmp2qcEUy_swarming/0/output.json

The change being reverted replaces '/' in the output filename with '_'. This
transformation should only be applied to file names inserted into the json file
name as opposed to the full file path.

[mattloring, 2016-06-22 22:17:39]

mattloring@google.com changed reviewers:
+ machenbach@google.com

[mattloring, 2016-06-22 22:17:40]

What steps should I have taken to avoid this issue before landing?

[ofrobots, 2016-06-22 22:20:24]

On 2016/06/22 22:17:40, mattloring wrote:
> What steps should I have taken to avoid this issue before landing?

LGTM.

[ofrobots, 2016-06-22 22:20:45]

The CQ bit was checked by ofrobots@google.com

[commit-bot: I haz the power, 2016-06-22 22:20:54]

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

[commit-bot: I haz the power, 2016-06-22 22:21:10]

Description was changed from

==========
Revert of Include file names in trace_turbo output (patchset #3 id:40001 of
https://codereview.chromium.org/2083863004/ )

Reason for revert:
Many build bots are failing with a message of the form:

Missing or invalid v8 JSON file: /tmp/tmp2qcEUy_swarming/0/output.json

Can be relanded once we understand why these failures are occuring.

Original issue's description:
> Include file names in trace_turbo output
> 
> The trace turbo output will overwrite itself when functions in different
> files share the same name. Output files now have the form
> `turbo-<function_name>:<opt_file_name>-<opt_phase>.suffix`.
> 
> R=ofrobots@google.com
> BUG=
> 
> Committed: https://crrev.com/a53b9bf02f31e5647c37e0392afa19f74df1a3ba
> Cr-Commit-Position: refs/heads/master@{#37199}

TBR=ofrobots@google.com,bmeurer@chromium.org,danno@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=
==========

to

==========
Revert of Include file names in trace_turbo output (patchset #3 id:40001 of
https://codereview.chromium.org/2083863004/ )

Reason for revert:
Many build bots are failing with a message of the form:

Missing or invalid v8 JSON file: /tmp/tmp2qcEUy_swarming/0/output.json

Can be relanded once we understand why these failures are occuring.

Original issue's description:
> Include file names in trace_turbo output
> 
> The trace turbo output will overwrite itself when functions in different
> files share the same name. Output files now have the form
> `turbo-<function_name>:<opt_file_name>-<opt_phase>.suffix`.
> 
> R=ofrobots@google.com
> BUG=
> 
> Committed: https://crrev.com/a53b9bf02f31e5647c37e0392afa19f74df1a3ba
> Cr-Commit-Position: refs/heads/master@{#37199}

TBR=ofrobots@google.com,bmeurer@chromium.org,danno@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=
==========

[commit-bot: I haz the power, 2016-06-22 22:21:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-22 22:24:01]

Description was changed from

==========
Revert of Include file names in trace_turbo output (patchset #3 id:40001 of
https://codereview.chromium.org/2083863004/ )

Reason for revert:
Many build bots are failing with a message of the form:

Missing or invalid v8 JSON file: /tmp/tmp2qcEUy_swarming/0/output.json

Can be relanded once we understand why these failures are occuring.

Original issue's description:
> Include file names in trace_turbo output
> 
> The trace turbo output will overwrite itself when functions in different
> files share the same name. Output files now have the form
> `turbo-<function_name>:<opt_file_name>-<opt_phase>.suffix`.
> 
> R=ofrobots@google.com
> BUG=
> 
> Committed: https://crrev.com/a53b9bf02f31e5647c37e0392afa19f74df1a3ba
> Cr-Commit-Position: refs/heads/master@{#37199}

TBR=ofrobots@google.com,bmeurer@chromium.org,danno@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=
==========

to

==========
Revert of Include file names in trace_turbo output (patchset #3 id:40001 of
https://codereview.chromium.org/2083863004/ )

Reason for revert:
Many build bots are failing with a message of the form:

Missing or invalid v8 JSON file: /tmp/tmp2qcEUy_swarming/0/output.json

Can be relanded once we understand why these failures are occuring.

Original issue's description:
> Include file names in trace_turbo output
>
> The trace turbo output will overwrite itself when functions in different
> files share the same name. Output files now have the form
> `turbo-<function_name>:<opt_file_name>-<opt_phase>.suffix`.
>
> R=ofrobots@google.com
> BUG=
>
> Committed: https://crrev.com/a53b9bf02f31e5647c37e0392afa19f74df1a3ba
> Cr-Commit-Position: refs/heads/master@{#37199}

TBR=ofrobots@google.com,bmeurer@chromium.org,danno@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=

Committed: https://crrev.com/97c2bc362f234bd58515a0faf6af23b4f8ad183a
Cr-Commit-Position: refs/heads/master@{#37204}
==========

[commit-bot: I haz the power, 2016-06-22 22:24:02]

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

[mattloring, 2016-06-22 22:59:04]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2081323007/ by mattloring@google.com.

The reason for reverting is: Ready to test fix and reland..