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

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

Issue 340513003: DevTools: Add JSDoc for static methods, fix JSDoc types and induced errors (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebased patch 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) 2014 Google Inc. All rights reserved. 2 * Copyright (C) 2014 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 253 matching lines...) Expand 10 before | Expand all | Expand 10 after
264 * @param {number} entryIndex 264 * @param {number} entryIndex
265 * @return {boolean} 265 * @return {boolean}
266 */ 266 */
267 canJumpToEntry: function(entryIndex) 267 canJumpToEntry: function(entryIndex)
268 { 268 {
269 return false; 269 return false;
270 }, 270 },
271 271
272 /** 272 /**
273 * @param {number} entryIndex 273 * @param {number} entryIndex
274 * @return {!string} 274 * @return {string}
275 */ 275 */
276 entryColor: function(entryIndex) 276 entryColor: function(entryIndex)
277 { 277 {
278 var record = this._records[entryIndex]; 278 var record = this._records[entryIndex];
279 if (record === this._cpuThreadRecord || record === this._gpuThreadRecord ) 279 if (record === this._cpuThreadRecord || record === this._gpuThreadRecord )
280 return "#555"; 280 return "#555";
281 281
282 if (record.type() === WebInspector.TimelineModel.RecordType.JSFrame) 282 if (record.type() === WebInspector.TimelineModel.RecordType.JSFrame)
283 return WebInspector.TimelineFlameChartDataProvider.jsFrameColorGener ator().colorForID(record.data()["functionName"]); 283 return WebInspector.TimelineFlameChartDataProvider.jsFrameColorGener ator().colorForID(record.data()["functionName"]);
284 284
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
388 /** 388 /**
389 * @return {number} 389 * @return {number}
390 */ 390 */
391 paddingLeft: function() 391 paddingLeft: function()
392 { 392 {
393 return 0; 393 return 0;
394 }, 394 },
395 395
396 /** 396 /**
397 * @param {number} entryIndex 397 * @param {number} entryIndex
398 * @return {!string} 398 * @return {string}
399 */ 399 */
400 textColor: function(entryIndex) 400 textColor: function(entryIndex)
401 { 401 {
402 return "white"; 402 return "white";
403 }, 403 },
404 404
405 /** 405 /**
406 * @param {number} entryIndex 406 * @param {number} entryIndex
407 * @return {?WebInspector.TimelineSelection} 407 * @return {?WebInspector.TimelineSelection}
408 */ 408 */
(...skipping 199 matching lines...) Expand 10 before | Expand all | Expand 10 after
608 * @param {number} entryIndex 608 * @param {number} entryIndex
609 * @return {boolean} 609 * @return {boolean}
610 */ 610 */
611 canJumpToEntry: function(entryIndex) 611 canJumpToEntry: function(entryIndex)
612 { 612 {
613 return false; 613 return false;
614 }, 614 },
615 615
616 /** 616 /**
617 * @param {number} entryIndex 617 * @param {number} entryIndex
618 * @return {!string} 618 * @return {string}
619 */ 619 */
620 entryColor: function(entryIndex) 620 entryColor: function(entryIndex)
621 { 621 {
622 var event = this._entryEvents[entryIndex]; 622 var event = this._entryEvents[entryIndex];
623 if (!event) 623 if (!event)
624 return "#555"; 624 return "#555";
625 var style = WebInspector.TimelineUIUtils.styleForTimelineEvent(event.nam e); 625 var style = WebInspector.TimelineUIUtils.styleForTimelineEvent(event.nam e);
626 return style.category.fillColorStop1; 626 return style.category.fillColorStop1;
627 }, 627 },
628 628
(...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after
706 /** 706 /**
707 * @return {number} 707 * @return {number}
708 */ 708 */
709 paddingLeft: function() 709 paddingLeft: function()
710 { 710 {
711 return 0; 711 return 0;
712 }, 712 },
713 713
714 /** 714 /**
715 * @param {number} entryIndex 715 * @param {number} entryIndex
716 * @return {!string} 716 * @return {string}
717 */ 717 */
718 textColor: function(entryIndex) 718 textColor: function(entryIndex)
719 { 719 {
720 return "white"; 720 return "white";
721 }, 721 },
722 722
723 /** 723 /**
724 * @param {string} title 724 * @param {string} title
725 */ 725 */
726 _appendHeaderRecord: function(title) 726 _appendHeaderRecord: function(title)
(...skipping 244 matching lines...) Expand 10 before | Expand all | Expand 10 after
971 * @param {number} entryIndex 971 * @param {number} entryIndex
972 * @return {?WebInspector.TimelineSelection} 972 * @return {?WebInspector.TimelineSelection}
973 */ 973 */
974 createSelection: function(entryIndex) { }, 974 createSelection: function(entryIndex) { },
975 /** 975 /**
976 * @param {?WebInspector.TimelineSelection} selection 976 * @param {?WebInspector.TimelineSelection} selection
977 * @return {number} 977 * @return {number}
978 */ 978 */
979 entryIndexForSelection: function(selection) { } 979 entryIndexForSelection: function(selection) { }
980 } 980 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698