Don't attempt to walk the native stack by default.

Don't attempt to walk the native stack by default.

This was unintentially enabled by default as a consequence of
https://codereview.chromium.org/1104323003/

BUG=23368
R=johnmccutchan@google.com

Committed: https://code.google.com/p/dart/source/detail?r=45666

[Cutch, 2015-05-08 18:12:33]

On 2015/05/08 18:01:53, koda wrote:

The define in the gypis (on all configurations) should be renamed to:

NATIVE_CODE_HAS_FRAME_POINTERS (or something like this).

The flag code at the top of profiler.cc should read:

#if defined(USING_SIMULATOR)
DEFINE_FLAG(bool, profile_vm, true,
            "Always collect native stack traces.");
#else
DEFINE_FLAG(bool, profile_vm, false,
            "Always collect native stack traces.");
#endif

[koda, 2015-05-08 22:05:35]

On 2015/05/08 18:12:33, Cutch wrote:
> On 2015/05/08 18:01:53, koda wrote:
> 
> The define in the gypis (on all configurations) should be renamed to:
> 
> NATIVE_CODE_HAS_FRAME_POINTERS (or something like this).
> 
> The flag code at the top of profiler.cc should read:
> 
> #if defined(USING_SIMULATOR)
> DEFINE_FLAG(bool, profile_vm, true,
>             "Always collect native stack traces.");
> #else
> DEFINE_FLAG(bool, profile_vm, false,
>             "Always collect native stack traces.");
> #endif

Done. PTAL

[Cutch, 2015-05-08 22:06:06]

lgtm

[koda, 2015-05-08 22:34:25]

Committed patchset #2 (id:20001) manually as r45666 (presubmit successful).