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

Unified Diff: third_party/WebKit/Source/devtools/front_end/ui/DOMSyntaxHighlighter.js

Issue 2466123002: DevTools: reformat front-end code to match chromium style. (Closed)
Patch Set: all done Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/devtools/front_end/ui/DOMSyntaxHighlighter.js
diff --git a/third_party/WebKit/Source/devtools/front_end/ui/DOMSyntaxHighlighter.js b/third_party/WebKit/Source/devtools/front_end/ui/DOMSyntaxHighlighter.js
index 71d6ec1c229fd5f69783ffd790fa890747789239..b2ba6a198fde68a2d6e0f42613e9bf64cc2e7495 100644
--- a/third_party/WebKit/Source/devtools/front_end/ui/DOMSyntaxHighlighter.js
+++ b/third_party/WebKit/Source/devtools/front_end/ui/DOMSyntaxHighlighter.js
@@ -27,85 +27,81 @@
* (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
-
/**
- * @constructor
- * @param {string} mimeType
- * @param {boolean} stripExtraWhitespace
+ * @unrestricted
*/
-WebInspector.DOMSyntaxHighlighter = function(mimeType, stripExtraWhitespace)
-{
+WebInspector.DOMSyntaxHighlighter = class {
+ /**
+ * @param {string} mimeType
+ * @param {boolean} stripExtraWhitespace
+ */
+ constructor(mimeType, stripExtraWhitespace) {
this._mimeType = mimeType;
this._stripExtraWhitespace = stripExtraWhitespace;
-};
+ }
-WebInspector.DOMSyntaxHighlighter.prototype = {
- /**
- * @param {string} content
- * @param {string} className
- * @return {!Element}
- */
- createSpan: function(content, className)
- {
- var span = createElement("span");
- span.className = "cm-" + className;
- if (this._stripExtraWhitespace && className !== "whitespace")
- content = content.replace(/^[\n\r]*/, "").replace(/\s*$/, "");
- span.createTextChild(content);
- return span;
- },
+ /**
+ * @param {string} content
+ * @param {string} className
+ * @return {!Element}
+ */
+ createSpan(content, className) {
+ var span = createElement('span');
+ span.className = 'cm-' + className;
+ if (this._stripExtraWhitespace && className !== 'whitespace')
+ content = content.replace(/^[\n\r]*/, '').replace(/\s*$/, '');
+ span.createTextChild(content);
+ return span;
+ }
- /**
- * @param {!Element} node
- * @return {!Promise.<undefined>}
- */
- syntaxHighlightNode: function(node)
- {
- var lines = node.textContent.split("\n");
- var plainTextStart;
- var line;
+ /**
+ * @param {!Element} node
+ * @return {!Promise.<undefined>}
+ */
+ syntaxHighlightNode(node) {
+ var lines = node.textContent.split('\n');
+ var plainTextStart;
+ var line;
- return self.runtime.extension(WebInspector.TokenizerFactory).instance().then(processTokens.bind(this));
+ return self.runtime.extension(WebInspector.TokenizerFactory).instance().then(processTokens.bind(this));
- /**
- * @param {!WebInspector.TokenizerFactory} tokenizerFactory
- * @this {WebInspector.DOMSyntaxHighlighter}
- */
- function processTokens(tokenizerFactory)
- {
- node.removeChildren();
- var tokenize = tokenizerFactory.createTokenizer(this._mimeType);
- for (var i = 0; i < lines.length; ++i) {
- line = lines[i];
- plainTextStart = 0;
- tokenize(line, processToken.bind(this));
- if (plainTextStart < line.length) {
- var plainText = line.substring(plainTextStart, line.length);
- node.createTextChild(plainText);
- }
- if (i < lines.length - 1)
- node.createTextChild("\n");
- }
+ /**
+ * @param {!WebInspector.TokenizerFactory} tokenizerFactory
+ * @this {WebInspector.DOMSyntaxHighlighter}
+ */
+ function processTokens(tokenizerFactory) {
+ node.removeChildren();
+ var tokenize = tokenizerFactory.createTokenizer(this._mimeType);
+ for (var i = 0; i < lines.length; ++i) {
+ line = lines[i];
+ plainTextStart = 0;
+ tokenize(line, processToken.bind(this));
+ if (plainTextStart < line.length) {
+ var plainText = line.substring(plainTextStart, line.length);
+ node.createTextChild(plainText);
}
+ if (i < lines.length - 1)
+ node.createTextChild('\n');
+ }
+ }
- /**
- * @param {string} token
- * @param {?string} tokenType
- * @param {number} column
- * @param {number} newColumn
- * @this {WebInspector.DOMSyntaxHighlighter}
- */
- function processToken(token, tokenType, column, newColumn)
- {
- if (!tokenType)
- return;
+ /**
+ * @param {string} token
+ * @param {?string} tokenType
+ * @param {number} column
+ * @param {number} newColumn
+ * @this {WebInspector.DOMSyntaxHighlighter}
+ */
+ function processToken(token, tokenType, column, newColumn) {
+ if (!tokenType)
+ return;
- if (column > plainTextStart) {
- var plainText = line.substring(plainTextStart, column);
- node.createTextChild(plainText);
- }
- node.appendChild(this.createSpan(token, tokenType));
- plainTextStart = newColumn;
- }
+ if (column > plainTextStart) {
+ var plainText = line.substring(plainTextStart, column);
+ node.createTextChild(plainText);
+ }
+ node.appendChild(this.createSpan(token, tokenType));
+ plainTextStart = newColumn;
}
+ }
};

Powered by Google App Engine
This is Rietveld 408576698