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

Unified Diff: Source/devtools/front_end/timeline/TimelineModel.js

Issue 363343004: DevTools: Unify main thread name across vanilla and tracing timeline modes. (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 side-by-side diff with in-line comments
Download patch
Index: Source/devtools/front_end/timeline/TimelineModel.js
diff --git a/Source/devtools/front_end/timeline/TimelineModel.js b/Source/devtools/front_end/timeline/TimelineModel.js
index ded583bc855af1e6152b5fb7e3857dba560315f2..a86d751da2a2561fa50344eb5d8d03ca221ea4cc 100644
--- a/Source/devtools/front_end/timeline/TimelineModel.js
+++ b/Source/devtools/front_end/timeline/TimelineModel.js
@@ -112,6 +112,8 @@ WebInspector.TimelineModel.Events = {
RecordFilterChanged: "RecordFilterChanged"
}
+WebInspector.TimelineModel.MainThreadName = "main";
+
/**
* @param {!Array.<!WebInspector.TimelineModel.Record>} recordsArray
* @param {?function(!WebInspector.TimelineModel.Record)|?function(!WebInspector.TimelineModel.Record,number)} preOrderCallback
@@ -371,7 +373,7 @@ WebInspector.TimelineModel.Record.prototype = {
startTime: function() { },
/**
- * @return {string|undefined}
+ * @return {string}
*/
thread: function() { },
@@ -493,7 +495,7 @@ WebInspector.TimelineMergingRecordBuffer.prototype = {
*/
process: function(thread, records)
{
- if (thread) {
+ if (thread !== WebInspector.TimelineModel.MainThreadName) {
this._backgroundRecordsBuffer = this._backgroundRecordsBuffer.concat(records);
return [];
}
« no previous file with comments | « Source/devtools/front_end/timeline/TimelineFrameModel.js ('k') | Source/devtools/front_end/timeline/TimelineModelImpl.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698