Changeset 109221 in webkit

Timestamp:

Feb 29, 2012 9:35:22 AM (12 years ago)

Author:

caseq@chromium.org

Message:

Web Inspector: timeline markers are not shown on the timeline panel
​https://bugs.webkit.org/show_bug.cgi?id=79921

Reviewed by Pavel Feldman.

• inspector/front-end/TimelinePanel.js:

(WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline.addTimestampRecords):
(WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/front-end/TimelinePanel.js (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-02-29  Andrey Kosyakov  <caseq@chromium.org>
+
+        Web Inspector: timeline markers are not shown on the timeline panel
+        https://bugs.webkit.org/show_bug.cgi?id=79921
+
+        Reviewed by Pavel Feldman.
+
+        * inspector/front-end/TimelinePanel.js:
+        (WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline.addTimestampRecords):
+        (WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline):
+
 2012-02-29  Kenichi Ishibashi  <bashi@chromium.org>
 

trunk/Source/WebCore/inspector/front-end/TimelinePanel.js

@@ -460 +460 @@
         ++this._allRecordsCount;
         var recordTypes = WebInspector.TimelineModel.RecordType;
-        if (record.type === recordTypes.MarkDOMContent || record.type === recordTypes.MarkLoad || record.type === recordTypes.TimeStamp)
-            this._timeStampRecords.push(formattedRecord);
+        var timeStampRecords = this._timeStampRecords;
+        function addTimestampRecords(record)
+        {
+            if (record.type === recordTypes.MarkDOMContent || record.type === recordTypes.MarkLoad || record.type === recordTypes.TimeStamp)
+                timeStampRecords.push(record);
+        }
+        WebInspector.TimelinePanel.forAllRecords([ formattedRecord ], addTimestampRecords);
     },