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

Side by Side Diff: third_party/WebKit/Source/devtools/front_end/coverage/CoverageModel.js

Issue 2861393004: DevTools/Coverage: fix decorations in HTML (Closed)
Patch Set: reverted toFixedIfFloating() changes Created 3 years, 7 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
« no previous file with comments | « third_party/WebKit/Source/devtools/front_end/coverage/CoverageListView.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2017 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2017 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 /** @typedef {{startOffset: number, endOffset: number, count: number}} */ 5 /** @typedef {{startOffset: number, endOffset: number, count: number}} */
6 Coverage.RangeUseCount; 6 Coverage.RangeUseCount;
7 7
8 /** @typedef {{end: number, count: (number|undefined)}} */ 8 /** @typedef {{end: number, count: (number|undefined)}} */
9 Coverage.CoverageSegment; 9 Coverage.CoverageSegment;
10 10
(...skipping 347 matching lines...) Expand 10 before | Expand all | Expand 10 after
358 * @param {number} start 358 * @param {number} start
359 * @param {number} end 359 * @param {number} end
360 * @return {boolean} 360 * @return {boolean}
361 */ 361 */
362 usageForRange(start, end) { 362 usageForRange(start, end) {
363 var index = this._segments.upperBound(start, (position, segment) => position - segment.end); 363 var index = this._segments.upperBound(start, (position, segment) => position - segment.end);
364 for (; index < this._segments.length && this._segments[index].end < end; ++i ndex) { 364 for (; index < this._segments.length && this._segments[index].end < end; ++i ndex) {
365 if (this._segments[index].count) 365 if (this._segments[index].count)
366 return true; 366 return true;
367 } 367 }
368 return index < this._segments.length && this._segments[index].count; 368 return index < this._segments.length && !!this._segments[index].count;
369 } 369 }
370 370
371 /** 371 /**
372 * @param {!Array<!Coverage.CoverageSegment>} segmentsA 372 * @param {!Array<!Coverage.CoverageSegment>} segmentsA
373 * @param {!Array<!Coverage.CoverageSegment>} segmentsB 373 * @param {!Array<!Coverage.CoverageSegment>} segmentsB
374 */ 374 */
375 static _mergeCoverage(segmentsA, segmentsB) { 375 static _mergeCoverage(segmentsA, segmentsB) {
376 var result = []; 376 var result = [];
377 377
378 var indexA = 0; 378 var indexA = 0;
(...skipping 26 matching lines...) Expand all
405 this._usedSize = 0; 405 this._usedSize = 0;
406 406
407 var last = 0; 407 var last = 0;
408 for (var segment of this._segments) { 408 for (var segment of this._segments) {
409 if (segment.count) 409 if (segment.count)
410 this._usedSize += segment.end - last; 410 this._usedSize += segment.end - last;
411 last = segment.end; 411 last = segment.end;
412 } 412 }
413 } 413 }
414 }; 414 };
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/devtools/front_end/coverage/CoverageListView.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698