DevTools: Skip native frames on timeline JS profiler.

DevTools: Skip native frames on timeline JS profiler.

Skip native frames on timeline tracing-based JS profiler.
It is made to align them with the stacks collected with capture
option. These do not include native functions.

BUG=406277
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=197746

[alph, 2015-06-24 13:21:07]

alph@chromium.org changed reviewers:
+ yurys@chromium.org

[yurys, 2015-06-24 13:57:45]

lgtm

https://codereview.chromium.org/1207693003/diff/1/Source/devtools/front_end/t...
File Source/devtools/front_end/timeline/TimelineJSProfile.js (left):

https://codereview.chromium.org/1207693003/diff/1/Source/devtools/front_end/t...
Source/devtools/front_end/timeline/TimelineJSProfile.js:255:
console.error("Entry at address " + oldAddressHex + " not found");
Why was this removed? Is there a valid case when we don't know the old address?

https://codereview.chromium.org/1207693003/diff/1/Source/devtools/front_end/t...
File Source/devtools/front_end/timeline/TimelineJSProfile.js (right):

https://codereview.chromium.org/1207693003/diff/1/Source/devtools/front_end/t...
Source/devtools/front_end/timeline/TimelineJSProfile.js:438: return
!entry.isNative ? entry : null;
entry.isNative ? null : entry;

[alph, 2015-06-24 14:03:39]

https://codereview.chromium.org/1207693003/diff/1/Source/devtools/front_end/t...
File Source/devtools/front_end/timeline/TimelineJSProfile.js (left):

https://codereview.chromium.org/1207693003/diff/1/Source/devtools/front_end/t...
Source/devtools/front_end/timeline/TimelineJSProfile.js:255:
console.error("Entry at address " + oldAddressHex + " not found");
On 2015/06/24 13:57:44, yurys wrote:
> Why was this removed? Is there a valid case when we don't know the old
address?

Reverted.

https://codereview.chromium.org/1207693003/diff/1/Source/devtools/front_end/t...
File Source/devtools/front_end/timeline/TimelineJSProfile.js (right):

https://codereview.chromium.org/1207693003/diff/1/Source/devtools/front_end/t...
Source/devtools/front_end/timeline/TimelineJSProfile.js:438: return
!entry.isNative ? entry : null;
On 2015/06/24 13:57:44, yurys wrote:
> entry.isNative ? null : entry;

Done.

[alph, 2015-06-24 14:03:47]

The CQ bit was checked by alph@chromium.org

[alph, 2015-06-24 14:03:47]

The patchset sent to the CQ was uploaded after l-g-t-m from yurys@chromium.org
Link to the patchset: https://codereview.chromium.org/1207693003/#ps20001
(title: "addressing comments.")

[commit-bot: I haz the power, 2015-06-24 14:04:08]

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

[commit-bot: I haz the power, 2015-06-24 15:26:37]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=197746