Reland of Added unit test for stack traces in telemetry crashes.

Reland of Added unit test for stack traces in telemetry crashes. (patchset #1 id:1 of https://codereview.chromium.org/1580313002/ )

Reason for revert:
Try again disabling 10.6

Original issue's description:
> Revert of Added unit test for stack traces in telemetry crashes. (patchset #17 id:320001 of https://codereview.chromium.org/1498633003/ )
>
> Reason for revert:
> Test fails on some bots.
>
> Original issue's description:
> > Added unit test for stack traces in telemetry crashes.
> >
> > This CL also fixes an issue on Mac where telemetry does not
> > know how to locate chromium_framework for the symbols.
> >
> > R=nednguyen@google.com
> > BUG=563716
> > TEST=ran unit test locally
> >
> > Committed: https://crrev.com/a20e03ee6906c50bdfaf8fb34fe684b4a89db852
> > Cr-Commit-Position: refs/heads/master@{#368997}
>
> TBR=nednguyen@google.com
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=563716
>
> Committed: https://crrev.com/f0cb0bf2dd7f677423318d2318902311e635538d
> Cr-Commit-Position: refs/heads/master@{#369056}

R=nednguyen@google.com
BUG=563716
Committed: https://crrev.com/5f0df0518d4befa6b0c597d0ccdcfa8e50367d00
Cr-Commit-Position: refs/heads/master@{#369404}

[David Yen, 2016-01-13 22:01:12]

Created Reland of Added unit test for stack traces in telemetry crashes.

[David Yen, 2016-01-13 22:05:29]

Description was changed from

==========
Reland of Added unit test for stack traces in telemetry crashes. (patchset #1
id:1 of https://codereview.chromium.org/1580313002/ )

Reason for revert:
Try again disabling 10.6

Original issue's description:
> Revert of Added unit test for stack traces in telemetry crashes. (patchset #17
id:320001 of https://codereview.chromium.org/1498633003/ )
> 
> Reason for revert:
> Test fails on some bots.
> 
> Original issue's description:
> > Added unit test for stack traces in telemetry crashes.
> >
> > This CL also fixes an issue on Mac where telemetry does not
> > know how to locate chromium_framework for the symbols.
> >
> > R=nednguyen@google.com
> > BUG=563716
> > TEST=ran unit test locally
> >
> > Committed: https://crrev.com/a20e03ee6906c50bdfaf8fb34fe684b4a89db852
> > Cr-Commit-Position: refs/heads/master@{#368997}
> 
> TBR=nednguyen@google.com
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=563716
> 
> Committed: https://crrev.com/f0cb0bf2dd7f677423318d2318902311e635538d
> Cr-Commit-Position: refs/heads/master@{#369056}

TBR=nednguyen@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=563716
==========

to

==========
Reland of Added unit test for stack traces in telemetry crashes. (patchset #1
id:1 of https://codereview.chromium.org/1580313002/ )

Reason for revert:
Try again disabling 10.6

Original issue's description:
> Revert of Added unit test for stack traces in telemetry crashes. (patchset #17
id:320001 of https://codereview.chromium.org/1498633003/ )
> 
> Reason for revert:
> Test fails on some bots.
> 
> Original issue's description:
> > Added unit test for stack traces in telemetry crashes.
> >
> > This CL also fixes an issue on Mac where telemetry does not
> > know how to locate chromium_framework for the symbols.
> >
> > R=nednguyen@google.com
> > BUG=563716
> > TEST=ran unit test locally
> >
> > Committed: https://crrev.com/a20e03ee6906c50bdfaf8fb34fe684b4a89db852
> > Cr-Commit-Position: refs/heads/master@{#368997}
> 
> TBR=nednguyen@google.com
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=563716
> 
> Committed: https://crrev.com/f0cb0bf2dd7f677423318d2318902311e635538d
> Cr-Commit-Position: refs/heads/master@{#369056}

R=nednguyen@google.com
BUG=563716
==========

[nednguyen, 2016-01-13 22:07:37]

lgtm

[nednguyen, 2016-01-13 22:08:18]

https://codereview.chromium.org/1581143002/diff/90001/tools/perf/core/stacktr...
File tools/perf/core/stacktrace_unittest.py (right):

https://codereview.chromium.org/1581143002/diff/90001/tools/perf/core/stacktr...
tools/perf/core/stacktrace_unittest.py:13: @decorators.Disabled('snowleopard')
nit: add comment why we disable this on snowleopard

[David Yen, 2016-01-13 22:16:24]

https://codereview.chromium.org/1581143002/diff/90001/tools/perf/core/stacktr...
File tools/perf/core/stacktrace_unittest.py (right):

https://codereview.chromium.org/1581143002/diff/90001/tools/perf/core/stacktr...
tools/perf/core/stacktrace_unittest.py:13: @decorators.Disabled('snowleopard')
On 2016/01/13 22:08:18, nednguyen wrote:
> nit: add comment why we disable this on snowleopard

Done.

[David Yen, 2016-01-14 02:19:20]

On 2016/01/13 22:16:24, David Yen wrote:
>
https://codereview.chromium.org/1581143002/diff/90001/tools/perf/core/stacktr...
> File tools/perf/core/stacktrace_unittest.py (right):
> 
>
https://codereview.chromium.org/1581143002/diff/90001/tools/perf/core/stacktr...
> tools/perf/core/stacktrace_unittest.py:13: @decorators.Disabled('snowleopard')
> On 2016/01/13 22:08:18, nednguyen wrote:
> > nit: add comment why we disable this on snowleopard
> 
> Done.

Looks like it skipped on 10.6 properly, I'll check the CQ.

[David Yen, 2016-01-14 02:19:24]

The CQ bit was checked by dyen@chromium.org

[David Yen, 2016-01-14 02:19:24]

The patchset sent to the CQ was uploaded after l-g-t-m from nednguyen@google.com
Link to the patchset: https://codereview.chromium.org/1581143002/#ps110001
(title: "added comment")

[commit-bot: I haz the power, 2016-01-14 02:23:36]

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

[commit-bot: I haz the power, 2016-01-14 04:30:23]

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

[commit-bot: I haz the power, 2016-01-14 04:30:24]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on tryserver.chromium.android (JOB_TIMED_OUT, no
build URL)
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_TIMED_OUT,
no build URL)
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_TIMED_OUT,
no build URL)
  android_clang_dbg_recipe on tryserver.chromium.android (JOB_TIMED_OUT, no
build URL)
  android_compile_dbg on tryserver.chromium.android (JOB_TIMED_OUT, no build
URL)
  cast_shell_android on tryserver.chromium.android (JOB_TIMED_OUT, no build URL)

[nednguyen, 2016-01-14 13:45:57]

The CQ bit was checked by nednguyen@google.com

[commit-bot: I haz the power, 2016-01-14 13:46:15]

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

[commit-bot: I haz the power, 2016-01-14 13:50:59]

Description was changed from

==========
Reland of Added unit test for stack traces in telemetry crashes. (patchset #1
id:1 of https://codereview.chromium.org/1580313002/ )

Reason for revert:
Try again disabling 10.6

Original issue's description:
> Revert of Added unit test for stack traces in telemetry crashes. (patchset #17
id:320001 of https://codereview.chromium.org/1498633003/ )
> 
> Reason for revert:
> Test fails on some bots.
> 
> Original issue's description:
> > Added unit test for stack traces in telemetry crashes.
> >
> > This CL also fixes an issue on Mac where telemetry does not
> > know how to locate chromium_framework for the symbols.
> >
> > R=nednguyen@google.com
> > BUG=563716
> > TEST=ran unit test locally
> >
> > Committed: https://crrev.com/a20e03ee6906c50bdfaf8fb34fe684b4a89db852
> > Cr-Commit-Position: refs/heads/master@{#368997}
> 
> TBR=nednguyen@google.com
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=563716
> 
> Committed: https://crrev.com/f0cb0bf2dd7f677423318d2318902311e635538d
> Cr-Commit-Position: refs/heads/master@{#369056}

R=nednguyen@google.com
BUG=563716
==========

to

==========
Reland of Added unit test for stack traces in telemetry crashes. (patchset #1
id:1 of https://codereview.chromium.org/1580313002/ )

Reason for revert:
Try again disabling 10.6

Original issue's description:
> Revert of Added unit test for stack traces in telemetry crashes. (patchset #17
id:320001 of https://codereview.chromium.org/1498633003/ )
> 
> Reason for revert:
> Test fails on some bots.
> 
> Original issue's description:
> > Added unit test for stack traces in telemetry crashes.
> >
> > This CL also fixes an issue on Mac where telemetry does not
> > know how to locate chromium_framework for the symbols.
> >
> > R=nednguyen@google.com
> > BUG=563716
> > TEST=ran unit test locally
> >
> > Committed: https://crrev.com/a20e03ee6906c50bdfaf8fb34fe684b4a89db852
> > Cr-Commit-Position: refs/heads/master@{#368997}
> 
> TBR=nednguyen@google.com
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=563716
> 
> Committed: https://crrev.com/f0cb0bf2dd7f677423318d2318902311e635538d
> Cr-Commit-Position: refs/heads/master@{#369056}

R=nednguyen@google.com
BUG=563716
==========

[commit-bot: I haz the power, 2016-01-14 13:51:00]

Committed patchset #3 (id:110001)

[commit-bot: I haz the power, 2016-01-14 13:52:40]

Description was changed from

==========
Reland of Added unit test for stack traces in telemetry crashes. (patchset #1
id:1 of https://codereview.chromium.org/1580313002/ )

Reason for revert:
Try again disabling 10.6

Original issue's description:
> Revert of Added unit test for stack traces in telemetry crashes. (patchset #17
id:320001 of https://codereview.chromium.org/1498633003/ )
> 
> Reason for revert:
> Test fails on some bots.
> 
> Original issue's description:
> > Added unit test for stack traces in telemetry crashes.
> >
> > This CL also fixes an issue on Mac where telemetry does not
> > know how to locate chromium_framework for the symbols.
> >
> > R=nednguyen@google.com
> > BUG=563716
> > TEST=ran unit test locally
> >
> > Committed: https://crrev.com/a20e03ee6906c50bdfaf8fb34fe684b4a89db852
> > Cr-Commit-Position: refs/heads/master@{#368997}
> 
> TBR=nednguyen@google.com
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=563716
> 
> Committed: https://crrev.com/f0cb0bf2dd7f677423318d2318902311e635538d
> Cr-Commit-Position: refs/heads/master@{#369056}

R=nednguyen@google.com
BUG=563716
==========

to

==========
Reland of Added unit test for stack traces in telemetry crashes. (patchset #1
id:1 of https://codereview.chromium.org/1580313002/ )

Reason for revert:
Try again disabling 10.6

Original issue's description:
> Revert of Added unit test for stack traces in telemetry crashes. (patchset #17
id:320001 of https://codereview.chromium.org/1498633003/ )
>
> Reason for revert:
> Test fails on some bots.
>
> Original issue's description:
> > Added unit test for stack traces in telemetry crashes.
> >
> > This CL also fixes an issue on Mac where telemetry does not
> > know how to locate chromium_framework for the symbols.
> >
> > R=nednguyen@google.com
> > BUG=563716
> > TEST=ran unit test locally
> >
> > Committed: https://crrev.com/a20e03ee6906c50bdfaf8fb34fe684b4a89db852
> > Cr-Commit-Position: refs/heads/master@{#368997}
>
> TBR=nednguyen@google.com
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=563716
>
> Committed: https://crrev.com/f0cb0bf2dd7f677423318d2318902311e635538d
> Cr-Commit-Position: refs/heads/master@{#369056}

R=nednguyen@google.com
BUG=563716

Committed: https://crrev.com/5f0df0518d4befa6b0c597d0ccdcfa8e50367d00
Cr-Commit-Position: refs/heads/master@{#369404}
==========

[commit-bot: I haz the power, 2016-01-14 13:52:42]

Patchset 3 (id:??) landed as
https://crrev.com/5f0df0518d4befa6b0c597d0ccdcfa8e50367d00
Cr-Commit-Position: refs/heads/master@{#369404}