DevTools: fix front-end compilation problems after #399131

DevTools: fix front-end compilation problems after #399131

BUG=614935
Committed: https://crrev.com/7c6861df88e8e16dd2be24270ab8836a6923e115
Cr-Commit-Position: refs/heads/master@{#400650}

[caseq, 2016-06-17 09:38:54]

caseq@chromium.org changed reviewers:
+ alph@chromium.org, paulirish@chromium.org

[alph, 2016-06-17 12:08:07]

lgtm

https://codereview.chromium.org/2072173003/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/timeline/TimelineTreeView.js
(right):

https://codereview.chromium.org/2072173003/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/timeline/TimelineTreeView.js:616:
_createAggregator: function()
Annotate plz

https://codereview.chromium.org/2072173003/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/timeline/TimelineUIUtils.js
(right):

https://codereview.chromium.org/2072173003/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/timeline/TimelineUIUtils.js:316:
return frame.scriptId !== "0" && (!frame.url || !frame.url.startsWith("native
"));
nit: !(frame.url && frame.url.startsWith("native "))

[paulirish, 2016-06-17 12:38:55]

lgtm

appreciate you taking care of this.

[caseq, 2016-06-17 13:09:23]

The CQ bit was checked by caseq@chromium.org

[caseq, 2016-06-17 13:09:24]

The patchset sent to the CQ was uploaded after l-g-t-m from
paulirish@chromium.org, alph@chromium.org
Link to the patchset: https://codereview.chromium.org/2072173003/#ps20001
(title: "review comments addressed")

[commit-bot: I haz the power, 2016-06-17 13:09:41]

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

[commit-bot: I haz the power, 2016-06-17 14:21:37]

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

[commit-bot: I haz the power, 2016-06-17 14:21:38]

Failed to apply the patch.

[caseq, 2016-06-19 09:29:27]

Description was changed from

==========
DevTools: fix front-end compilcation problems after #399131

BUG=614935
==========

to

==========
DevTools: fix front-end compilation problems after #399131

BUG=614935
==========

[caseq, 2016-06-20 09:35:14]

Patchset #3 (id:40001) has been deleted

[caseq, 2016-06-20 09:35:22]

The CQ bit was checked by caseq@chromium.org

[caseq, 2016-06-20 09:35:23]

The patchset sent to the CQ was uploaded after l-g-t-m from
paulirish@chromium.org, alph@chromium.org
Link to the patchset: https://codereview.chromium.org/2072173003/#ps20001
(title: "review comments addressed")

[commit-bot: I haz the power, 2016-06-20 09:35:40]

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

[commit-bot: I haz the power, 2016-06-20 11:12:15]

Description was changed from

==========
DevTools: fix front-end compilation problems after #399131

BUG=614935
==========

to

==========
DevTools: fix front-end compilation problems after #399131

BUG=614935
==========

[commit-bot: I haz the power, 2016-06-20 11:12:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-20 11:14:08]

Description was changed from

==========
DevTools: fix front-end compilation problems after #399131

BUG=614935
==========

to

==========
DevTools: fix front-end compilation problems after #399131

BUG=614935

Committed: https://crrev.com/7c6861df88e8e16dd2be24270ab8836a6923e115
Cr-Commit-Position: refs/heads/master@{#400650}
==========

[commit-bot: I haz the power, 2016-06-20 11:14:09]

Patchset 2 (id:??) landed as
https://crrev.com/7c6861df88e8e16dd2be24270ab8836a6923e115
Cr-Commit-Position: refs/heads/master@{#400650}