Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1348)

Issue 2853053002: DevTools: clicking in console messages should not jump to bottom (Closed)

Created:
3 years, 7 months ago by luoe
Modified:
3 years, 7 months ago
Reviewers:
kozy
CC:
chromium-reviews
Target Ref:
refs/branch-heads/3071
Project:
chromium
Visibility:
Public.

Description

DevTools: clicking in console messages should not jump to bottom ConsolePrompt's moveCaretToEndOfPrompt() works by setting the CodeMirror editor's selection to the end. After a recent CodeMirror roll, doing so now scrolls the editor into view. This change in behavior led to an error where expanding an object in console would jump the viewport to bottom. This CL only focuses the prompt without moving the cursor, in some cases. NOTRY=true NOPRESUBMIT=true BUG=706128 Review-Url: https://codereview.chromium.org/2840663002 Cr-Commit-Position: refs/heads/master@{#468009} (cherry picked from commit c5213f9bd6b6e6e95f40852b56ef62f1307c94ee) Review-Url: https://codereview.chromium.org/2853053002 Cr-Commit-Position: refs/branch-heads/3071@{#331} Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641} Committed: https://chromium.googlesource.com/chromium/src/+/0eefdccfe7b90d093875ebda910a72fbb182b6b8

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+133 lines, -7 lines) Patch
A third_party/WebKit/LayoutTests/inspector/console/console-focus.html View 1 chunk +97 lines, -0 lines 0 comments Download
A third_party/WebKit/LayoutTests/inspector/console/console-focus-expected.txt View 1 chunk +27 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/devtools/front_end/console/ConsoleView.js View 2 chunks +9 lines, -7 lines 0 comments Download

Messages

Total messages: 9 (5 generated)
luoe
This is a merge to M59, no conflicts. Needs a committer's lgtm to land. Please ...
3 years, 7 months ago (2017-05-01 17:57:30 UTC) #3
kozy
lgtm
3 years, 7 months ago (2017-05-01 18:38:05 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/2853053002/1
3 years, 7 months ago (2017-05-01 18:39:01 UTC) #6
commit-bot: I haz the power
3 years, 7 months ago (2017-05-01 18:40:18 UTC) #9
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/0eefdccfe7b90d093875ebda910a...

Powered by Google App Engine
This is Rietveld 408576698