WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
123356
Web Inspector: Change how the load and content ready event timestamps are tracked.
https://bugs.webkit.org/show_bug.cgi?id=123356
Summary
Web Inspector: Change how the load and content ready event timestamps are tra...
Timothy Hatcher
Reported
2013-10-25 13:14:46 PDT
We should track the timestamps per frame. This will be needed by some up coming Timeline changes.
Attachments
Patch
(7.13 KB, patch)
2013-10-25 13:16 PDT
,
Timothy Hatcher
no flags
Details
Formatted Diff
Diff
Patch
(7.14 KB, patch)
2013-10-25 13:37 PDT
,
Timothy Hatcher
joepeck
: review+
timothy
: commit-queue-
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2013-10-25 13:14:56 PDT
<
rdar://problem/15322597
>
Timothy Hatcher
Comment 2
2013-10-25 13:16:50 PDT
Created
attachment 215202
[details]
Patch
Timothy Hatcher
Comment 3
2013-10-25 13:37:03 PDT
Created
attachment 215207
[details]
Patch
Joseph Pecoraro
Comment 4
2013-10-25 15:33:30 PDT
Comment on
attachment 215207
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=215207&action=review
r=me
> Source/WebInspectorUI/UserInterface/TimelineManager.js:-61 > - return this._loadEventTime;
You are removing this._loadEventTime, you can remove this instance variable from: 1. The constructor 2. _clear 3. Replace the use in _addRecord to look at the MainFrame On IRC you've handled these.
Timothy Hatcher
Comment 5
2014-01-20 18:59:49 PST
https://trac.webkit.org/changeset/162398
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug