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

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

Issue 270693003: Timeline: make frames in TimelineView selectable (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: bail out early if selection is of wrong type Created 6 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 | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright 2014 The Chromium Authors. All rights reserved. 2 * Copyright 2014 The Chromium Authors. All rights reserved.
3 * Use of this source code is governed by a BSD-style license that can be 3 * Use of this source code is governed by a BSD-style license that can be
4 * found in the LICENSE file. 4 * found in the LICENSE file.
5 */ 5 */
6 6
7 /** 7 /**
8 * @constructor 8 * @constructor
9 * @implements {WebInspector.TimelineModeView} 9 * @implements {WebInspector.TimelineModeView}
10 * @implements {WebInspector.FlameChartDelegate} 10 * @implements {WebInspector.FlameChartDelegate}
(...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after
88 { 88 {
89 this._mainView.setWindowTimes(startTime, endTime); 89 this._mainView.setWindowTimes(startTime, endTime);
90 }, 90 },
91 91
92 /** 92 /**
93 * @param {number} width 93 * @param {number} width
94 */ 94 */
95 setSidebarSize: function(width) {}, 95 setSidebarSize: function(width) {},
96 96
97 /** 97 /**
98 * @param {?WebInspector.TimelineModel.Record} record 98 * @param {?WebInspector.TimelineSelection} selection
99 */ 99 */
100 setSelectedRecord: function(record) {}, 100 setSelection: function(selection) {},
101 101
102 /** 102 /**
103 * @param {!WebInspector.Event} event 103 * @param {!WebInspector.Event} event
104 */ 104 */
105 _onEntrySelected: function(event) 105 _onEntrySelected: function(event)
106 { 106 {
107 var index = /** @type {number} */ (event.data); 107 var index = /** @type {number} */ (event.data);
108 var record = this._dataProvider._recordAt(index); 108 var record = this._dataProvider._recordAt(index);
109 if (!record || this._dataProvider._isHeaderRecord(record)) { 109 if (!record || this._dataProvider._isHeaderRecord(record)) {
110 this._delegate.showInDetails("", document.createTextNode("")); 110 this._delegate.showInDetails("", document.createTextNode(""));
(...skipping 465 matching lines...) Expand 10 before | Expand all | Expand 10 after
576 /** 576 /**
577 * @param {string} string 577 * @param {string} string
578 * @return {string} 578 * @return {string}
579 */ 579 */
580 colorForString: function(string) 580 colorForString: function(string)
581 { 581 {
582 var hash = WebInspector.TraceViewPalette._stringHash(string); 582 var hash = WebInspector.TraceViewPalette._stringHash(string);
583 return this._palette[hash % this._palette.length]; 583 return this._palette[hash % this._palette.length];
584 } 584 }
585 }; 585 };
OLDNEW
« no previous file with comments | « Source/devtools/front_end/timeline/TimelinePanel.js ('k') | Source/devtools/front_end/timeline/TimelineView.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698