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

Unified Diff: third_party/WebKit/Source/devtools/front_end/sources/FilePathScoreFunction.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/sources/FilePathScoreFunction.js
diff --git a/third_party/WebKit/Source/devtools/front_end/sources/FilePathScoreFunction.js b/third_party/WebKit/Source/devtools/front_end/sources/FilePathScoreFunction.js
index 9a35f6b5120e57efa4bd85efcd585de7bba3972c..c1be4aa53ab749b6d07f3ca8669a21f8067ea55e 100644
--- a/third_party/WebKit/Source/devtools/front_end/sources/FilePathScoreFunction.js
+++ b/third_party/WebKit/Source/devtools/front_end/sources/FilePathScoreFunction.js
@@ -27,165 +27,158 @@
* (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} query
+ * @unrestricted
*/
-WebInspector.FilePathScoreFunction = function(query)
-{
+WebInspector.FilePathScoreFunction = class {
+ /**
+ * @param {string} query
+ */
+ constructor(query) {
this._query = query;
this._queryUpperCase = query.toUpperCase();
this._score = null;
this._sequence = null;
- this._dataUpperCase = "";
+ this._dataUpperCase = '';
this._fileNameIndex = 0;
-};
+ }
-WebInspector.FilePathScoreFunction.prototype = {
- /**
- * @param {string} data
- * @param {?Array<number>} matchIndexes
- * @return {number}
- */
- score: function(data, matchIndexes)
- {
- if (!data || !this._query)
- return 0;
- var n = this._query.length;
- var m = data.length;
- if (!this._score || this._score.length < n * m) {
- this._score = new Int32Array(n * m * 2);
- this._sequence = new Int32Array(n * m * 2);
- }
- var score = this._score;
- var sequence = /** @type {!Int32Array} */ (this._sequence);
- this._dataUpperCase = data.toUpperCase();
- this._fileNameIndex = data.lastIndexOf("/");
- for (var i = 0; i < n; ++i) {
- for (var j = 0; j < m; ++j) {
- var skipCharScore = j === 0 ? 0 : score[i * m + j - 1];
- var prevCharScore = i === 0 || j === 0 ? 0 : score[(i - 1) * m + j - 1];
- var consecutiveMatch = i === 0 || j === 0 ? 0 : sequence[(i - 1) * m + j - 1];
- var pickCharScore = this._match(this._query, data, i, j, consecutiveMatch);
- if (pickCharScore && prevCharScore + pickCharScore >= skipCharScore) {
- sequence[i * m + j] = consecutiveMatch + 1;
- score[i * m + j] = (prevCharScore + pickCharScore);
- } else {
- sequence[i * m + j] = 0;
- score[i * m + j] = skipCharScore;
- }
- }
+ /**
+ * @param {string} data
+ * @param {?Array<number>} matchIndexes
+ * @return {number}
+ */
+ score(data, matchIndexes) {
+ if (!data || !this._query)
+ return 0;
+ var n = this._query.length;
+ var m = data.length;
+ if (!this._score || this._score.length < n * m) {
+ this._score = new Int32Array(n * m * 2);
+ this._sequence = new Int32Array(n * m * 2);
+ }
+ var score = this._score;
+ var sequence = /** @type {!Int32Array} */ (this._sequence);
+ this._dataUpperCase = data.toUpperCase();
+ this._fileNameIndex = data.lastIndexOf('/');
+ for (var i = 0; i < n; ++i) {
+ for (var j = 0; j < m; ++j) {
+ var skipCharScore = j === 0 ? 0 : score[i * m + j - 1];
+ var prevCharScore = i === 0 || j === 0 ? 0 : score[(i - 1) * m + j - 1];
+ var consecutiveMatch = i === 0 || j === 0 ? 0 : sequence[(i - 1) * m + j - 1];
+ var pickCharScore = this._match(this._query, data, i, j, consecutiveMatch);
+ if (pickCharScore && prevCharScore + pickCharScore >= skipCharScore) {
+ sequence[i * m + j] = consecutiveMatch + 1;
+ score[i * m + j] = (prevCharScore + pickCharScore);
+ } else {
+ sequence[i * m + j] = 0;
+ score[i * m + j] = skipCharScore;
}
- if (matchIndexes)
- this._restoreMatchIndexes(sequence, n, m, matchIndexes);
- return score[n * m - 1];
- },
+ }
+ }
+ if (matchIndexes)
+ this._restoreMatchIndexes(sequence, n, m, matchIndexes);
+ return score[n * m - 1];
+ }
- /**
- * @param {string} data
- * @param {number} j
- * @return {boolean}
- */
- _testWordStart: function(data, j)
- {
- var prevChar = data.charAt(j - 1);
- return j === 0 || prevChar === "_" || prevChar === "-" || prevChar === "/" ||
- (data[j - 1] !== this._dataUpperCase[j - 1] && data[j] === this._dataUpperCase[j]);
- },
+ /**
+ * @param {string} data
+ * @param {number} j
+ * @return {boolean}
+ */
+ _testWordStart(data, j) {
+ var prevChar = data.charAt(j - 1);
+ return j === 0 || prevChar === '_' || prevChar === '-' || prevChar === '/' ||
+ (data[j - 1] !== this._dataUpperCase[j - 1] && data[j] === this._dataUpperCase[j]);
+ }
- /**
- * @param {!Int32Array} sequence
- * @param {number} n
- * @param {number} m
- * @param {!Array<number>} out
- */
- _restoreMatchIndexes: function(sequence, n, m, out)
- {
- var i = n - 1, j = m - 1;
- while (i >= 0 && j >= 0) {
- switch (sequence[i * m + j]) {
- case 0:
- --j;
- break;
- default:
- out.push(j);
- --i;
- --j;
- break;
- }
- }
- out.reverse();
- },
+ /**
+ * @param {!Int32Array} sequence
+ * @param {number} n
+ * @param {number} m
+ * @param {!Array<number>} out
+ */
+ _restoreMatchIndexes(sequence, n, m, out) {
+ var i = n - 1, j = m - 1;
+ while (i >= 0 && j >= 0) {
+ switch (sequence[i * m + j]) {
+ case 0:
+ --j;
+ break;
+ default:
+ out.push(j);
+ --i;
+ --j;
+ break;
+ }
+ }
+ out.reverse();
+ }
- /**
- * @param {string} query
- * @param {string} data
- * @param {number} i
- * @param {number} j
- * @return {number}
- */
- _singleCharScore: function(query, data, i, j)
- {
- var isWordStart = this._testWordStart(data, j);
- var isFileName = j > this._fileNameIndex;
- var isPathTokenStart = j === 0 || data[j - 1] === "/";
- var isCapsMatch = query[i] === data[j] && query[i] === this._queryUpperCase[i];
- var score = 10;
- if (isPathTokenStart)
- score += 4;
- if (isWordStart)
- score += 2;
- if (isCapsMatch)
- score += 6;
- if (isFileName)
- score += 4;
- // promote the case of making the whole match in the filename
- if (j === this._fileNameIndex + 1 && i === 0)
- score += 5;
- if (isFileName && isWordStart)
- score += 3;
- return score;
- },
+ /**
+ * @param {string} query
+ * @param {string} data
+ * @param {number} i
+ * @param {number} j
+ * @return {number}
+ */
+ _singleCharScore(query, data, i, j) {
+ var isWordStart = this._testWordStart(data, j);
+ var isFileName = j > this._fileNameIndex;
+ var isPathTokenStart = j === 0 || data[j - 1] === '/';
+ var isCapsMatch = query[i] === data[j] && query[i] === this._queryUpperCase[i];
+ var score = 10;
+ if (isPathTokenStart)
+ score += 4;
+ if (isWordStart)
+ score += 2;
+ if (isCapsMatch)
+ score += 6;
+ if (isFileName)
+ score += 4;
+ // promote the case of making the whole match in the filename
+ if (j === this._fileNameIndex + 1 && i === 0)
+ score += 5;
+ if (isFileName && isWordStart)
+ score += 3;
+ return score;
+ }
- /**
- * @param {string} query
- * @param {string} data
- * @param {number} i
- * @param {number} j
- * @param {number} sequenceLength
- * @return {number}
- */
- _sequenceCharScore: function(query, data, i, j, sequenceLength)
- {
- var isFileName = j > this._fileNameIndex;
- var isPathTokenStart = j === 0 || data[j - 1] === "/";
- var score = 10;
- if (isFileName)
- score += 4;
- if (isPathTokenStart)
- score += 5;
- score += sequenceLength * 4;
- return score;
- },
+ /**
+ * @param {string} query
+ * @param {string} data
+ * @param {number} i
+ * @param {number} j
+ * @param {number} sequenceLength
+ * @return {number}
+ */
+ _sequenceCharScore(query, data, i, j, sequenceLength) {
+ var isFileName = j > this._fileNameIndex;
+ var isPathTokenStart = j === 0 || data[j - 1] === '/';
+ var score = 10;
+ if (isFileName)
+ score += 4;
+ if (isPathTokenStart)
+ score += 5;
+ score += sequenceLength * 4;
+ return score;
+ }
- /**
- * @param {string} query
- * @param {string} data
- * @param {number} i
- * @param {number} j
- * @param {number} consecutiveMatch
- * @return {number}
- */
- _match: function(query, data, i, j, consecutiveMatch)
- {
- if (this._queryUpperCase[i] !== this._dataUpperCase[j])
- return 0;
+ /**
+ * @param {string} query
+ * @param {string} data
+ * @param {number} i
+ * @param {number} j
+ * @param {number} consecutiveMatch
+ * @return {number}
+ */
+ _match(query, data, i, j, consecutiveMatch) {
+ if (this._queryUpperCase[i] !== this._dataUpperCase[j])
+ return 0;
- if (!consecutiveMatch)
- return this._singleCharScore(query, data, i, j);
- else
- return this._sequenceCharScore(query, data, i, j - consecutiveMatch, consecutiveMatch);
- }
+ if (!consecutiveMatch)
+ return this._singleCharScore(query, data, i, j);
+ else
+ return this._sequenceCharScore(query, data, i, j - consecutiveMatch, consecutiveMatch);
+ }
};
-

Powered by Google App Engine
This is Rietveld 408576698