DevTools: Slim down timeline overview height

DevTools: Slim down timeline overview height

Review-Url: https://codereview.chromium.org/2644373004
Cr-Commit-Position: refs/heads/master@{#445269}
Committed: https://chromium.googlesource.com/chromium/src/+/799d1b122128aa137ee7c4a66f7e65eaf39b0485

[alph, 2017-01-20 21:34:46]

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

[caseq, 2017-01-20 23:07:02]

can we have a screenshot for this one?

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

https://codereview.chromium.org/2644373004/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/timeline/TimelineEventOverview.js:170:
requests.forEach(drawRequest);
let's make changes to path more explicit?

https://codereview.chromium.org/2644373004/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/timeline/TimelineEventOverview.js:204:
const style = Timeline.TimelineUIUtils.networkCategoryColor(category);
inline?

https://codereview.chromium.org/2644373004/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/timeline/TimelineEventOverview.js:210:
path.style = style;
why do we change band path's style each time?

[alph, 2017-01-21 01:42:33]

all done. ptal

[alph, 2017-01-21 01:46:06]

ss:
https://drive.google.com/a/google.com/file/d/0ByLjXIa7NKi9M1RzemRMV045WjQ/vie...

[pfeldman, 2017-01-21 02:17:47]

lgtm

[alph, 2017-01-21 02:18:19]

The CQ bit was checked by alph@chromium.org

[commit-bot: I haz the power, 2017-01-21 02:18:45]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-21 04:07:52]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484965099468960,
"parent_rev": "a07de206ea3a4b029574556e10066340f5c7d6ee", "commit_rev":
"799d1b122128aa137ee7c4a66f7e65eaf39b0485"}

[commit-bot: I haz the power, 2017-01-21 04:08:23]

Description was changed from

==========
DevTools: Slim down timeline overview height
==========

to

==========
DevTools: Slim down timeline overview height

Review-Url: https://codereview.chromium.org/2644373004
Cr-Commit-Position: refs/heads/master@{#445269}
Committed:
https://chromium.googlesource.com/chromium/src/+/799d1b122128aa137ee7c4a66f7e...
==========

[commit-bot: I haz the power, 2017-01-21 04:08:24]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/799d1b122128aa137ee7c4a66f7e...