Bug 160821 – Rename DocumentLoadTiming and ResourceLoadTiming

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Johan K. Jensen]

Rename ResourceLoadTiming to reflect that it only has timing about the network request.
Rename DocumentLoadTiming to handle generic timing information for both the main document and subresources.

[Johan K. Jensen]

Created attachment 285954 [details]
Patch

[Alex Christensen]

Comment on attachment 285954 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=285954&action=review

> Source/WebCore/CMakeLists.txt:1915
> +    loader/LoadTiming.cpp

This should be put in correct alphabetical order.

> Source/WebCore/loader/DocumentLoader.h:259
> +        void resetTiming() { m_loadTiming = LoadTiming(); }

Does { } work?

> Source/WebCore/loader/LoadTiming.h:27
> +#ifndef LoadTiming_h
> +#define LoadTiming_h

#pragma once

> Source/WebCore/platform/network/NetworkLoadTiming.h:28
> +#ifndef NetworkLoadTiming_h
> +#define NetworkLoadTiming_h

#pragma once, as long as we're touching this file.

[Johan K. Jensen]

Created attachment 285959 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 285959 [details]
Patch

Clearing flags on attachment: 285959

Committed r204429: <http://trac.webkit.org/changeset/204429>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.