Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright 2014 The Chromium Authors. All rights reserved. | 1 // Copyright 2014 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 /** | 4 /** |
| 5 * @implements {Sources.SourcesView.EditorAction} | 5 * @implements {Sources.SourcesView.EditorAction} |
| 6 * @unrestricted | 6 * @unrestricted |
| 7 */ | 7 */ |
| 8 Sources.InplaceFormatterEditorAction = class { | 8 Sources.InplaceFormatterEditorAction = class { |
| 9 /** | 9 /** |
| 10 * @param {!Common.Event} event | 10 * @param {!Common.Event} event |
| (...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 92 function innerCallback(formattedContent, formatterMapping) { | 92 function innerCallback(formattedContent, formatterMapping) { |
| 93 if (uiSourceCode.workingCopy() === formattedContent) | 93 if (uiSourceCode.workingCopy() === formattedContent) |
| 94 return; | 94 return; |
| 95 var sourceFrame = this._sourcesView.viewForFile(uiSourceCode); | 95 var sourceFrame = this._sourcesView.viewForFile(uiSourceCode); |
| 96 var start = [0, 0]; | 96 var start = [0, 0]; |
| 97 if (sourceFrame) { | 97 if (sourceFrame) { |
| 98 var selection = sourceFrame.selection(); | 98 var selection = sourceFrame.selection(); |
| 99 start = formatterMapping.originalToFormatted(selection.startLine, select ion.startColumn); | 99 start = formatterMapping.originalToFormatted(selection.startLine, select ion.startColumn); |
| 100 } | 100 } |
| 101 uiSourceCode.setWorkingCopy(formattedContent); | 101 uiSourceCode.setWorkingCopy(formattedContent); |
| 102 this._formatDecorations(uiSourceCode, formatterMapping); | |
| 103 | |
| 102 this._sourcesView.showSourceLocation(uiSourceCode, start[0], start[1]); | 104 this._sourcesView.showSourceLocation(uiSourceCode, start[0], start[1]); |
| 103 } | 105 } |
| 104 } | 106 } |
| 107 | |
| 108 /** | |
| 109 * @param {!Workspace.UISourceCode} uiSourceCode | |
| 110 * @param {!Sources.FormatterSourceMapping} sourceMapping | |
| 111 */ | |
| 112 _formatDecorations(uiSourceCode, sourceMapping) { | |
| 113 var decorations = uiSourceCode.allDecorations(); | |
| 114 if (!decorations.length) | |
| 115 return; | |
| 116 | |
| 117 uiSourceCode.removeAllDecorations(); | |
| 118 | |
| 119 for (var decoration of decorations) { | |
| 120 var range = decoration.range(); | |
| 121 var startLocation = new Array(2); | |
|
lushnikov
2016/11/17 22:50:43
drop these
| |
| 122 var endLocation = new Array(2); | |
| 123 | |
| 124 startLocation = sourceMapping.originalToFormatted(range.startLine,range.st artColumn); | |
|
lushnikov
2016/11/17 22:50:43
why create arrays beforehand? just:
var startLoca
| |
| 125 | |
| 126 endLocation = sourceMapping.originalToFormatted(range.endLine,range.endCol umn); | |
|
lushnikov
2016/11/17 22:50:43
nit: space is required between function arguments
| |
| 127 | |
| 128 uiSourceCode.addDecoration( | |
| 129 new Common.TextRange(startLocation[0], startLocation[1], | |
| 130 endLocation[0], endLocation[1]), | |
| 131 /** @type {string} */ (decoration.type()), | |
| 132 decoration.data()); | |
| 133 } | |
| 134 } | |
| 105 }; | 135 }; |
| OLD | NEW |