Chromium Code Reviews

Issue 2465373002: [Devtools] Fix network timeline canvas selection splitview (Closed)

Created:
4 years, 1 month ago by allada
Modified:
4 years, 1 month ago
Reviewers:
dgozman
CC:
chromium-reviews, caseq+blink_chromium.org, lushnikov+blink_chromium.org, pfeldman+blink_chromium.org, apavlov+blink_chromium.org, devtools-reviews_chromium.org, blink-reviews, pfeldman, kozyatinskiy+blink_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[Devtools] Fix network timeline canvas selection splitview This patch fixes a regression caused when a user selects a request in the new network canvas timeline it would show the canvas to the left instead of the name column. R=dgozman BUG=653738 Committed: https://crrev.com/4025c2cba57945b8b51ef021e2915906f3c71b0d Committed: https://crrev.com/a8ce2ab11cbe520868896bfc52ca72e3e25e2165 Cr-Original-Commit-Position: refs/heads/master@{#429092} Cr-Commit-Position: refs/heads/master@{#429150}

Patch Set 1 #

Patch Set 2 : changes #

Unified diffs Side-by-side diffs Stats (+1 line, -1 line)
M third_party/WebKit/Source/devtools/front_end/network/NetworkLogViewColumns.js View 1 chunk +1 line, -1 line 0 comments

Messages

Total messages: 21 (12 generated)
allada
PTL
4 years, 1 month ago (2016-11-01 17:51:13 UTC) #2
dgozman
lgtm
4 years, 1 month ago (2016-11-01 18:58:13 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2465373002/1
4 years, 1 month ago (2016-11-01 20:24:53 UTC) #8
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 1 month ago (2016-11-01 20:30:25 UTC) #10
pfeldman
A revert of this CL (patchset #1 id:1) has been created in https://codereview.chromium.org/2472523002/ by pfeldman@chromium.org. ...
4 years, 1 month ago (2016-11-01 20:31:38 UTC) #11
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/4025c2cba57945b8b51ef021e2915906f3c71b0d Cr-Commit-Position: refs/heads/master@{#429092}
4 years, 1 month ago (2016-11-01 20:36:06 UTC) #13
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2465373002/20001
4 years, 1 month ago (2016-11-01 21:33:50 UTC) #17
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years, 1 month ago (2016-11-01 23:19:09 UTC) #19
commit-bot: I haz the power
4 years, 1 month ago (2016-11-01 23:22:16 UTC) #21
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/a8ce2ab11cbe520868896bfc52ca72e3e25e2165
Cr-Commit-Position: refs/heads/master@{#429150}

Powered by Google App Engine