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

Unified Diff: Source/devtools/front_end/sdk/TimelineManager.js

Issue 476773002: Nuke TimelineAllEventsReceived event (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 4 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/sdk/TimelineManager.js
diff --git a/Source/devtools/front_end/sdk/TimelineManager.js b/Source/devtools/front_end/sdk/TimelineManager.js
index 8500f4ebdd506f900055c23073909218b5bf9736..782683b90ad0a1b5a73a9b5b71cc41792ba97bf6 100644
--- a/Source/devtools/front_end/sdk/TimelineManager.js
+++ b/Source/devtools/front_end/sdk/TimelineManager.js
@@ -46,7 +46,6 @@ WebInspector.TimelineManager.EventTypes = {
TimelineStarted: "TimelineStarted",
TimelineStopped: "TimelineStopped",
TimelineEventRecorded: "TimelineEventRecorded",
- TimelineAllEventsReceived: "TimelineAllEventsReceived",
TimelineProgress: "TimelineProgress"
}
@@ -139,12 +138,11 @@ WebInspector.TimelineManager.prototype = {
*/
_stopped: function(consoleTimeline, events)
{
- this.dispatchEventToListeners(WebInspector.TimelineManager.EventTypes.TimelineStopped, consoleTimeline);
- if (events) {
- for (var i = 0; i < events.length; ++i)
- this._dispatcher.eventRecorded(events[i]);
- }
- this.dispatchEventToListeners(WebInspector.TimelineManager.EventTypes.TimelineAllEventsReceived, 0);
+ var data = {
+ consoleTimeline: consoleTimeline,
+ events: events || []
+ };
+ this.dispatchEventToListeners(WebInspector.TimelineManager.EventTypes.TimelineStopped, data);
},
_configureCpuProfilerSamplingInterval: function()
« no previous file with comments | « LayoutTests/inspector/console/console-timeline-expected.txt ('k') | Source/devtools/front_end/timeline/TimelineModelImpl.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698