DevTools: Make item info follow the mouse on the flame chart.

DevTools: Make item info follow the mouse on the flame chart.

Add info to the network part of the timeline flamechart.

BUG=528489
Committed: https://crrev.com/708e3609f90afdae12e3cbe70c245dd0409c7ba7
Cr-Commit-Position: refs/heads/master@{#361213}

[alph, 2015-11-21 04:10:46]

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

[alph, 2015-11-23 18:20:08]

Description was changed from

==========
DevTools: Add tracking tooltip to flame chart.

Add info to the network part of the timeline flamechart.

BUG=528489
==========

to

==========
DevTools: Make item info follow the mouse on the flame chart.

Add info to the network part of the timeline flamechart.

BUG=528489
==========

[caseq, 2015-11-23 18:38:12]

lgtm

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

https://codereview.chromium.org/1463393003/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js:953:
if (request.requestMethod)
Let's only care for the requests for which we captured willSendRequest and hence
have method and url? You'd be able to use String.format() then.

[alph, 2015-11-23 19:54:46]

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

https://codereview.chromium.org/1463393003/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/timeline/TimelineFlameChart.js:953:
if (request.requestMethod)
On 2015/11/23 18:38:12, caseq wrote:
> Let's only care for the requests for which we captured willSendRequest and
hence
> have method and url? You'd be able to use String.format() then.

Done.

[alph, 2015-11-23 19:55:23]

The CQ bit was checked by alph@chromium.org

[alph, 2015-11-23 19:55:23]

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

[commit-bot: I haz the power, 2015-11-23 19:56:33]

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

[commit-bot: I haz the power, 2015-11-23 23:14:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-23 23:15:36]

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