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:

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	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
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.