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

Side by Side Diff: Source/devtools/front_end/timeline/TimelineModel.js

Issue 336183005: Remove title() from TimelineModel.Record (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 6 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 | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2012 Google Inc. All rights reserved. 2 * Copyright (C) 2012 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 353 matching lines...) Expand 10 before | Expand all | Expand 10 after
364 * @return {!Array.<!WebInspector.TimelineModel.Record>} 364 * @return {!Array.<!WebInspector.TimelineModel.Record>}
365 */ 365 */
366 children: function() { }, 366 children: function() { },
367 367
368 /** 368 /**
369 * @return {!WebInspector.TimelineCategory} 369 * @return {!WebInspector.TimelineCategory}
370 */ 370 */
371 category: function() { }, 371 category: function() { },
372 372
373 /** 373 /**
374 * @return {string}
375 */
376 title: function() { },
377
378 /**
379 * @return {number} 374 * @return {number}
380 */ 375 */
381 startTime: function() { }, 376 startTime: function() { },
382 377
383 /** 378 /**
384 * @return {string|undefined} 379 * @return {string|undefined}
385 */ 380 */
386 thread: function() { }, 381 thread: function() { },
387 382
388 /** 383 /**
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
428 setUserObject: function(key, value) { }, 423 setUserObject: function(key, value) { },
429 424
430 /** 425 /**
431 * @return {!Object.<string, number>} 426 * @return {!Object.<string, number>}
432 */ 427 */
433 aggregatedStats: function() { }, 428 aggregatedStats: function() { },
434 429
435 /** 430 /**
436 * @return {?Array.<string>} 431 * @return {?Array.<string>}
437 */ 432 */
438 warnings: function() { }, 433 warnings: function() { }
439
440 /**
441 * @param {!RegExp} regExp
442 * @return {boolean}
443 */
444 testContentMatching: function(regExp) { }
445 } 434 }
446 435
447 /** 436 /**
448 * @constructor 437 * @constructor
449 */ 438 */
450 WebInspector.TimelineModel.Filter = function() 439 WebInspector.TimelineModel.Filter = function()
451 { 440 {
452 /** @type {!WebInspector.TimelineModel} */ 441 /** @type {!WebInspector.TimelineModel} */
453 this._model; 442 this._model;
454 } 443 }
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
499 function recordTimestampComparator(a, b) 488 function recordTimestampComparator(a, b)
500 { 489 {
501 // Never return 0, as the merge function will squash identical entri es. 490 // Never return 0, as the merge function will squash identical entri es.
502 return a.startTime() < b.startTime() ? -1 : 1; 491 return a.startTime() < b.startTime() ? -1 : 1;
503 } 492 }
504 var result = this._backgroundRecordsBuffer.mergeOrdered(records, recordT imestampComparator); 493 var result = this._backgroundRecordsBuffer.mergeOrdered(records, recordT imestampComparator);
505 this._backgroundRecordsBuffer = []; 494 this._backgroundRecordsBuffer = [];
506 return result; 495 return result;
507 } 496 }
508 } 497 }
OLDNEW
« no previous file with comments | « Source/devtools/front_end/timeline/TimelineFlameChart.js ('k') | Source/devtools/front_end/timeline/TimelineModelImpl.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698