RESOLVED FIXED 126676
Web Inspector: Correctly track time bounds of WebInspector.Timeline and WebInspector.TimelineRecording
https://bugs.webkit.org/show_bug.cgi?id=126676
Summary Web Inspector: Correctly track time bounds of WebInspector.Timeline and WebIn...
Timothy Hatcher
Reported 2014-01-08 19:07:40 PST
The startTime and endTime were not correct when Network timeline needed to be taken into account (due to its dynamic loading nature.)
Attachments
Patch (11.23 KB, patch)
2014-01-08 19:11 PST, Timothy Hatcher
no flags
Patch (3.32 KB, patch)
2014-01-10 15:01 PST, Timothy Hatcher
timothy: commit-queue-
Radar WebKit Bug Importer
Comment 1 2014-01-08 19:08:24 PST
Timothy Hatcher
Comment 2 2014-01-08 19:11:56 PST
Joseph Pecoraro
Comment 3 2014-01-09 13:35:17 PST
Comment on attachment 220690 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=220690&action=review r=me > Source/WebInspectorUI/UserInterface/TimelineRecording.js:64 > + }.bind(this)); You could use the thisObject optional param to Map.prototype.forEach. E.g.: forEach(function() {...}, this); instead of forEach(function() {...}.bind(this)); > Source/WebInspectorUI/UserInterface/TimelineRecording.js:74 > + }.bind(this)); Ditto
Timothy Hatcher
Comment 4 2014-01-10 15:01:43 PST
Timothy Hatcher
Comment 5 2014-01-20 19:00:55 PST
Note You need to log in before you can comment on or make changes to this bug.