| Summary: | Web Inspector: console.timeStamp() marked are not shown in Timeline overview | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Andrey Kosyakov <caseq> | ||||||
| Component: | Web Inspector (Deprecated) | Assignee: | Andrey Kosyakov <caseq> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | apavlov, bmcquade, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys | ||||||
| Priority: | P2 | ||||||||
| Version: | 528+ (Nightly build) | ||||||||
| Hardware: | All | ||||||||
| OS: | All | ||||||||
| Attachments: |
|
||||||||
Created attachment 157657 [details]
Patch
Comment on attachment 157657 [details]
Patch
A test?
Created attachment 157689 [details]
Patch
Committed r125272: <http://trac.webkit.org/changeset/125272> |
0. Open Web Inspector, switch to the Timeline panel 1. Start timeline 2. Type console.timeStamp("foo") in console 3. Stop timeline 4. Note timestamp events appear in the detailed view 4. Note there are no orange vertical markers for the timestamp events in the time bars of either overview or detailed view. This is a regression introduced at r119826 (http://trac.webkit.org/changeset/119826) -- timestamp events currently don't have a frame id. They probably should, but we should display timestamp for all frames anyway.