[Devtools] Fixed undefined type in Network Waterfall for Other types

[Devtools] Fixed undefined type in Network Waterfall for Other types

This patch fixes issue where the network waterfall cannot resolve the
type color of Other if the mime-type is not in the list of known types.
(ie: when it resolves to default type "other")

R=dgozman
BUG=678392
Review-Url: https://codereview.chromium.org/2609863005
Cr-Commit-Position: refs/heads/master@{#442459}
Committed: https://chromium.googlesource.com/chromium/src/+/aae7548bfd243de6452e770e42884ead54093485

[allada, 2017-01-04 22:30:39]

PTL

[allada, 2017-01-04 22:32:31]

Description was changed from

==========
[Devtools] Fixed undefined type in Network Waterfall for Other types

This patch fixes issue where the network waterfall cannot resolve the
type color of Other if the mime-type is not in the list of known types.
(ie: when it resolves to default type "other")

R=dgozman
BUG=None
==========

to

==========
[Devtools] Fixed undefined type in Network Waterfall for Other types

This patch fixes issue where the network waterfall cannot resolve the
type color of Other if the mime-type is not in the list of known types.
(ie: when it resolves to default type "other")

R=dgozman
BUG=678392
==========

[dgozman, 2017-01-09 23:59:51]

lgtm

https://codereview.chromium.org/2609863005/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js
(right):

https://codereview.chromium.org/2609863005/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js:840: var
groupKey = request.connectionId;
Unrelated?

[allada, 2017-01-10 00:13:56]

The CQ bit was checked by allada@chromium.org

[allada, 2017-01-10 00:13:57]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/2609863005/#ps20001
(title: "[Devtools] Fixed undefined type in Network Waterfall for Other types")

[allada, 2017-01-10 00:14:06]

done.

[commit-bot: I haz the power, 2017-01-10 00:15:21]

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-10 03:03:41]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484007236761900,
"parent_rev": "23a69173a82474bd2981f5f75766e37f75ebbfb0", "commit_rev":
"aae7548bfd243de6452e770e42884ead54093485"}

[commit-bot: I haz the power, 2017-01-10 03:04:10]

Description was changed from

==========
[Devtools] Fixed undefined type in Network Waterfall for Other types

This patch fixes issue where the network waterfall cannot resolve the
type color of Other if the mime-type is not in the list of known types.
(ie: when it resolves to default type "other")

R=dgozman
BUG=678392
==========

to

==========
[Devtools] Fixed undefined type in Network Waterfall for Other types

This patch fixes issue where the network waterfall cannot resolve the
type color of Other if the mime-type is not in the list of known types.
(ie: when it resolves to default type "other")

R=dgozman
BUG=678392

Review-Url: https://codereview.chromium.org/2609863005
Cr-Commit-Position: refs/heads/master@{#442459}
Committed:
https://chromium.googlesource.com/chromium/src/+/aae7548bfd243de6452e770e4288...
==========

[commit-bot: I haz the power, 2017-01-10 03:04:11]

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