| Summary: | Web Inspector: iOS 8: Resources Timeline Data does not show up | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Joseph Pecoraro <joepeck> | ||||
| Component: | Web Inspector | Assignee: | Joseph Pecoraro <joepeck> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | burg, commit-queue, graouts, joepeck, jonowells, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | 528+ (Nightly build) | ||||||
| Hardware: | All | ||||||
| OS: | All | ||||||
| Attachments: |
|
||||||
|
Description
Joseph Pecoraro
2015-06-29 15:40:48 PDT
Created attachment 255786 [details]
[PATCH] Proposed Fix
This is relatively ugly, but I couldn't really think up of a better approach. No matter how early we try to reset the base timestamp, it will never be right, because the "will send request" for the navigation's new main resource, will have always happened in the previous page's "timeline". Let me know what you think.
Attachment 255786 [details] did not pass style-queue:
ERROR: Source/WebInspectorUI/ChangeLog:12: Need whitespace between colon and description [changelog/filechangedescriptionwhitespace] [5]
ERROR: Source/WebInspectorUI/ChangeLog:15: Need whitespace between colon and description [changelog/filechangedescriptionwhitespace] [5]
Total errors found: 2 in 5 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 255786 [details]
[PATCH] Proposed Fix
Looks good.
Comment on attachment 255786 [details] [PATCH] Proposed Fix Clearing flags on attachment: 255786 Committed r186098: <http://trac.webkit.org/changeset/186098> All reviewed patches have been landed. Closing bug. |