Bug 41816

Summary: Implement performance.timing.fetchStart
Product: WebKit Reporter: Tony Gentilcore <tonyg>
Component: WebCore Misc.Assignee: Tony Gentilcore <tonyg>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, dglazkov, fishd, japhet, pfeldman
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 41332    
Bug Blocks: 30685, 42018    
Attachments:
Description Flags
Patch
none
Mark in MainResourceLoader to make redirect time implementation cleaner
none
MFix typo in LayoutTest: navigationSTart -> navigationStart
none
Patch for landing tonyg: commit-queue-

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
DetailsFormatted DiffDiff
Mark in MainResourceLoader to make redirect time implementation cleaner (9.10 KB, patch)
2010-07-11 12:58 PDT, Tony Gentilcore 		no flags
DetailsFormatted DiffDiff
MFix typo in LayoutTest: navigationSTart -> navigationStart (9.10 KB, patch)
2010-07-11 13:07 PDT, Tony Gentilcore 		no flags
DetailsFormatted DiffDiff
Patch for landing (9.17 KB, patch)
2010-07-13 08:33 PDT, Tony Gentilcore 		tonyg: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Tony Gentilcore
Comment 1 2010-07-09 17:13:36 PDT
Created attachment 61116 [details] Patch
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.