DevTools: Pretty print inconsistent spacing |
|||
Issue descriptionVersions Tested: 55, 52 What steps will reproduce the problem? (1) Make a new snippet (2) Add the code: 1 || null; 1 || 2; (3) Click the pretty print button What is the expected output? 1 || null; 1 || 2; What do you see instead? 1 || null ; 1 || 2; There is an extra space after the null and before the ;. And it is inconsistent with the 1 || 2;
,
Aug 30 2016
,
Sep 2 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/20536ec459a5d475faf25b67566a8728528b0d3b commit 20536ec459a5d475faf25b67566a8728528b0d3b Author: ahmetemiremir <ahmetemiremir@gmail.com> Date: Fri Sep 02 00:26:40 2016 DevTools: Fix pretty print inconsistent spacing BUG= 642414 Review-Url: https://codereview.chromium.org/2300073003 Cr-Commit-Position: refs/heads/master@{#416142} [modify] https://crrev.com/20536ec459a5d475faf25b67566a8728528b0d3b/third_party/WebKit/LayoutTests/inspector/sources/pretty-print-javascript-7-expected.txt [modify] https://crrev.com/20536ec459a5d475faf25b67566a8728528b0d3b/third_party/WebKit/LayoutTests/inspector/sources/pretty-print-javascript-7.html [modify] https://crrev.com/20536ec459a5d475faf25b67566a8728528b0d3b/third_party/WebKit/Source/devtools/front_end/es_tree/AcornTokenizer.js [modify] https://crrev.com/20536ec459a5d475faf25b67566a8728528b0d3b/third_party/WebKit/Source/devtools/front_end/externs.js
,
Dec 13 2016
|
|||
►
Sign in to add a comment |
|||
Comment 1 by einbinder@chromium.org
, Aug 30 2016