Rename is_visible to is_reflectable; use is_debuggable instead of is_reflectable

Rename is_visible to is_reflectable; use is_debuggable instead of is_reflectable to decide whether to filter a frame from stack traces.

Merge verbose_stacktrace flag into show_hidden_frames.

Don't mark native functions as non-debuggable (breakpoints don't actually work there now but nothing goes wrong).

BUG=

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

[rmacnak, 2015-01-22 22:55:02]

rmacnak@google.com changed reviewers:
+ iposva@google.com

[Ivan Posva, 2015-01-23 19:37:07]

LGTMwC -ip

https://codereview.chromium.org/868453002/diff/100001/runtime/vm/debugger.cc
File runtime/vm/debugger.cc (right):

https://codereview.chromium.org/868453002/diff/100001/runtime/vm/debugger.cc#...
runtime/vm/debugger.cc:908: if (FLAG_show_invisible_frames ||
frame->function().is_stacktrace_visible()) {
Shouldn't we just use is_debuggable() here instead of adding the alias
is_stacktrace_visible?

https://codereview.chromium.org/868453002/diff/100001/runtime/vm/object.h
File runtime/vm/object.h (right):

https://codereview.chromium.org/868453002/diff/100001/runtime/vm/object.h#new...
runtime/vm/object.h:2190: 
Thanks! This explanation was needed.

[rmacnak, 2015-01-23 21:16:17]

https://codereview.chromium.org/868453002/diff/100001/runtime/vm/debugger.cc
File runtime/vm/debugger.cc (right):

https://codereview.chromium.org/868453002/diff/100001/runtime/vm/debugger.cc#...
runtime/vm/debugger.cc:908: if (FLAG_show_invisible_frames ||
frame->function().is_stacktrace_visible()) {
On 2015/01/23 19:37:07, Ivan Posva wrote:
> Shouldn't we just use is_debuggable() here instead of adding the alias
> is_stacktrace_visible?

Yeah, flattened these out.

[rmacnak, 2015-01-23 21:16:50]

Committed patchset #7 (id:120001) manually as 43121 (presubmit successful).