RESOLVED FIXED 41816
Implement performance.timing.fetchStart
https://bugs.webkit.org/show_bug.cgi?id=41816
Summary Implement performance.timing.fetchStart
Tony Gentilcore
Reported 2010-07-07 17:23:58 PDT
Implement performance.timing.fetch start for Web Timing.
Attachments
Patch (8.55 KB, patch)
2010-07-09 17:13 PDT, Tony Gentilcore
no flags
Mark in MainResourceLoader to make redirect time implementation cleaner (9.10 KB, patch)
2010-07-11 12:58 PDT, Tony Gentilcore
no flags
MFix typo in LayoutTest: navigationSTart -> navigationStart (9.10 KB, patch)
2010-07-11 13:07 PDT, Tony Gentilcore
no flags
Patch for landing (9.17 KB, patch)
2010-07-13 08:33 PDT, Tony Gentilcore
tonyg: commit-queue-
Tony Gentilcore
Comment 1 2010-07-09 17:13:36 PDT
Tony Gentilcore
Comment 2 2010-07-11 12:58:58 PDT
Created attachment 61179 [details] Mark in MainResourceLoader to make redirect time implementation cleaner
Tony Gentilcore
Comment 3 2010-07-11 13:07:07 PDT
Created attachment 61180 [details] MFix typo in LayoutTest: navigationSTart -> navigationStart
Darin Fisher (:fishd, Google)
Comment 4 2010-07-12 22:09:39 PDT
Comment on attachment 61180 [details] MFix typo in LayoutTest: navigationSTart -> navigationStart WebCore/ChangeLog:11 + (WebCore::MainResourceLoader::willSendRequest): Record the fetchStart time for each request for the main resource. This means it is called for each server redirect, overwritting the previous value. In 42018, this will be modified such that if a previous value exists, it is stored as the redirect time. In bug 42018? (might be good to link to it.) R=me
Tony Gentilcore
Comment 5 2010-07-13 08:33:49 PDT
Created attachment 61381 [details] Patch for landing
Tony Gentilcore
Comment 6 2010-07-13 08:34:07 PDT
(In reply to comment #4) > (From update of attachment 61180 [details]) > WebCore/ChangeLog:11 > + (WebCore::MainResourceLoader::willSendRequest): Record the fetchStart time for each request for the main resource. This means it is called for each server redirect, overwritting the previous value. In 42018, this will be modified such that if a previous value exists, it is stored as the redirect time. > In bug 42018? (might be good to link to it.) Done. > > R=me
Tony Gentilcore
Comment 7 2010-07-13 16:34:42 PDT
Committed r63231 and then r63232 to fix the ChangeLog merge error.
Note You need to log in before you can comment on or make changes to this bug.