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

Side by Side Diff: third_party/WebKit/Source/devtools/front_end/sources/ScriptFormatter.js

Issue 1907263002: DevTools: Fix all outstanding JavaScript style issues. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove yoda disable. fix throw parens Created 4 years, 8 months 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2011 Google Inc. All rights reserved. 2 * Copyright (C) 2011 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
78 78
79 /** 79 /**
80 * @constructor 80 * @constructor
81 * @implements {WebInspector.Formatter} 81 * @implements {WebInspector.Formatter}
82 * @param {string} mimeType 82 * @param {string} mimeType
83 * @param {string} content 83 * @param {string} content
84 * @param {function(string, !WebInspector.FormatterSourceMapping)} callback 84 * @param {function(string, !WebInspector.FormatterSourceMapping)} callback
85 */ 85 */
86 WebInspector.ScriptFormatter = function(mimeType, content, callback) 86 WebInspector.ScriptFormatter = function(mimeType, content, callback)
87 { 87 {
88 content = content.replace(/\r\n?|[\n\u2028\u2029]/g, "\n").replace(/^\uFEFF/ , ''); 88 content = content.replace(/\r\n?|[\n\u2028\u2029]/g, "\n").replace(/^\uFEFF/ , "");
89 this._callback = callback; 89 this._callback = callback;
90 this._originalContent = content; 90 this._originalContent = content;
91 91
92 this._worker = new WorkerRuntime.Worker("formatter_worker"); 92 this._worker = new WorkerRuntime.Worker("formatter_worker");
93 this._worker.onmessage = this._didFormatContent.bind(this); 93 this._worker.onmessage = this._didFormatContent.bind(this);
94 94
95 var parameters = { 95 var parameters = {
96 mimeType: mimeType, 96 mimeType: mimeType,
97 content: content, 97 content: content,
98 indentString: WebInspector.moduleSetting("textEditorIndent").get() 98 indentString: WebInspector.moduleSetting("textEditorIndent").get()
(...skipping 137 matching lines...) Expand 10 before | Expand all | Expand 10 after
236 */ 236 */
237 _convertPosition: function(positions1, positions2, position) 237 _convertPosition: function(positions1, positions2, position)
238 { 238 {
239 var index = positions1.upperBound(position) - 1; 239 var index = positions1.upperBound(position) - 1;
240 var convertedPosition = positions2[index] + position - positions1[index] ; 240 var convertedPosition = positions2[index] + position - positions1[index] ;
241 if (index < positions2.length - 1 && convertedPosition > positions2[inde x + 1]) 241 if (index < positions2.length - 1 && convertedPosition > positions2[inde x + 1])
242 convertedPosition = positions2[index + 1]; 242 convertedPosition = positions2[index + 1];
243 return convertedPosition; 243 return convertedPosition;
244 } 244 }
245 } 245 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698