[DevTools] Improve UX of timeline column sort in Network panel.

[DevTools] Improve UX of timeline column sort in Network panel.

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

[dgozman, 2015-05-27 13:03:34]

dgozman@chromium.org changed reviewers:
+ paulirish@chromium.org, pfeldman@chromium.org

[dgozman, 2015-05-27 13:03:35]

Take a look please.

[paulirish, 2015-05-27 15:54:17]

Some wording tweaks

[paulirish, 2015-05-27 15:54:29]

https://codereview.chromium.org/1157263002/diff/1/Source/devtools/front_end/n...
File Source/devtools/front_end/network/NetworkLogView.js (right):

https://codereview.chromium.org/1157263002/diff/1/Source/devtools/front_end/n...
Source/devtools/front_end/network/NetworkLogView.js:512: option.label =
WebInspector.UIString("Timeline \u2013 Response Time");
Response Start Time

https://codereview.chromium.org/1157263002/diff/1/Source/devtools/front_end/n...
Source/devtools/front_end/network/NetworkLogView.js:524: option.label =
WebInspector.UIString("Timeline \u2013 Duration");
Total Duration

[dgozman, 2015-05-28 11:05:08]

Pavel, ping.

[pfeldman, 2015-05-29 12:42:49]

lgtm

[dgozman, 2015-05-29 12:58:32]

https://codereview.chromium.org/1157263002/diff/1/Source/devtools/front_end/n...
File Source/devtools/front_end/network/NetworkLogView.js (right):

https://codereview.chromium.org/1157263002/diff/1/Source/devtools/front_end/n...
Source/devtools/front_end/network/NetworkLogView.js:512: option.label =
WebInspector.UIString("Timeline \u2013 Response Time");
On 2015/05/27 15:54:29, paulirish wrote:
> Response Start Time

This one is too similar to Start Time. I'd rather leave as is.

https://codereview.chromium.org/1157263002/diff/1/Source/devtools/front_end/n...
Source/devtools/front_end/network/NetworkLogView.js:524: option.label =
WebInspector.UIString("Timeline \u2013 Duration");
On 2015/05/27 15:54:29, paulirish wrote:
> Total Duration

Done.

[dgozman, 2015-05-29 12:58:35]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2015-05-29 12:58:35]

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

[commit-bot: I haz the power, 2015-05-29 12:58:50]

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

[commit-bot: I haz the power, 2015-05-29 14:08:00]

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