Bug 41816 - Implement performance.timing.fetchStart
Summary: Implement performance.timing.fetchStart
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Tony Gentilcore
URL:
Keywords:
Depends on: 41332
Blocks: 30685 42018
  Show dependency treegraph
 
Reported: 2010-07-07 17:23 PDT by Tony Gentilcore
Modified: 2010-07-13 16:34 PDT (History)
5 users (show)

See Also:


Attachments
Patch (8.55 KB, patch)
2010-07-09 17:13 PDT, Tony Gentilcore
no flags Details | Formatted Diff | Diff
Mark in MainResourceLoader to make redirect time implementation cleaner (9.10 KB, patch)
2010-07-11 12:58 PDT, Tony Gentilcore
no flags Details | Formatted Diff | Diff
MFix typo in LayoutTest: navigationSTart -> navigationStart (9.10 KB, patch)
2010-07-11 13:07 PDT, Tony Gentilcore
no flags Details | Formatted Diff | Diff
Patch for landing (9.17 KB, patch)
2010-07-13 08:33 PDT, Tony Gentilcore
tonyg: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tony Gentilcore 2010-07-07 17:23:58 PDT
Implement performance.timing.fetch start for Web Timing.
Comment 1 Tony Gentilcore 2010-07-09 17:13:36 PDT
Created attachment 61116 [details]
Patch
Comment 2 Tony Gentilcore 2010-07-11 12:58:58 PDT
Created attachment 61179 [details]
Mark in MainResourceLoader to make redirect time implementation cleaner
Comment 3 Tony Gentilcore 2010-07-11 13:07:07 PDT
Created attachment 61180 [details]
MFix typo in LayoutTest: navigationSTart -> navigationStart
Comment 4 Darin Fisher (:fishd, Google) 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
Comment 5 Tony Gentilcore 2010-07-13 08:33:49 PDT
Created attachment 61381 [details]
Patch for landing
Comment 6 Tony Gentilcore 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
Comment 7 Tony Gentilcore 2010-07-13 16:34:42 PDT
Committed r63231 and then r63232 to fix the ChangeLog merge error.