Bug 42917

Summary:

Move DocumentLoadTiming struct to a new file

Product:

WebKit

Reporter:

Tony Gentilcore <tonyg>

Component:

New Bugs

Assignee:

Tony Gentilcore <tonyg>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, commit-queue, eric, fishd, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Other

OS:

OS X 10.5

Attachments:

Description	Flags
Patch	none
Patch for landing	none

Description Tony Gentilcore 2010-07-23 16:24:37 PDT

Move DocumentLoadTiming struct to a new file

Comment 1 Tony Gentilcore 2010-07-23 16:27:33 PDT

Created attachment 62472 [details]
Patch

Comment 2 Darin Fisher (:fishd, Google) 2010-07-25 21:00:56 PDT

Comment on attachment 62472 [details]
Patch

WebCore/loader/DocumentLoadTiming.h:35
 +        : navigationStart(0.0)
nit: indent initializer list by 4 spaces


r=me otherwise

Comment 3 Tony Gentilcore 2010-07-26 08:15:08 PDT

Created attachment 62575 [details]
Patch for landing

Comment 4 Tony Gentilcore 2010-07-26 08:15:57 PDT

(In reply to comment #2)
> (From update of attachment 62472 [details])
> WebCore/loader/DocumentLoadTiming.h:35
>  +        : navigationStart(0.0)
> nit: indent initializer list by 4 spaces

Oops. Good catch. Fixed.

> 
> 
> r=me otherwise

Comment 5 WebKit Commit Bot 2010-07-26 10:03:55 PDT

Comment on attachment 62575 [details]
Patch for landing

Clearing flags on attachment: 62575

Committed r64051: <http://trac.webkit.org/changeset/64051>

Comment 6 WebKit Commit Bot 2010-07-26 10:03:59 PDT

All reviewed patches have been landed.  Closing bug.

Comment 7 WebKit Review Bot 2010-07-26 10:45:10 PDT

http://trac.webkit.org/changeset/64051 might have broken Chromium Win Release
The following changes are on the blame list:
http://trac.webkit.org/changeset/64050
http://trac.webkit.org/changeset/64051