Report nav timing 2 instance as soon as it's requested.

third_party/WebKit/Source/core/loader/DocumentLoader.cpp

 /*
  * Copyright (C) 2006, 2007, 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 11 matching lines @@
  * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
  * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
  * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
  * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "core/loader/DocumentLoader.h"
 
+#include <memory>
 #include "core/dom/Document.h"
 #include "core/dom/WeakIdentifierMap.h"
 #include "core/events/Event.h"
 #include "core/frame/Deprecation.h"
 #include "core/frame/FrameHost.h"
 #include "core/frame/LocalDOMWindow.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/Settings.h"
 #include "core/frame/csp/ContentSecurityPolicy.h"
 #include "core/html/HTMLFrameOwnerElement.h"
@@ skipping 16 matching lines @@
 #include "platform/HTTPNames.h"
 #include "platform/UserGestureIndicator.h"
 #include "platform/loader/fetch/FetchInitiatorTypeNames.h"
 #include "platform/loader/fetch/FetchRequest.h"
 #include "platform/loader/fetch/FetchUtils.h"
 #include "platform/loader/fetch/MemoryCache.h"
 #include "platform/loader/fetch/ResourceFetcher.h"
 #include "platform/mhtml/ArchiveResource.h"
 #include "platform/network/ContentSecurityPolicyResponseHeaders.h"
 #include "platform/network/HTTPParsers.h"
+#include "platform/network/ResourceTimingInfo.h"

[Kunihiko Sakamoto, 2017/02/13 08:22:29]

Still needed?

[sunjian, 2017/02/14 21:29:02]

It compiles fine without it. But it is actually a direct dependency. Maybe some
other header files included it also. But I think it's better to have it?

[Kunihiko Sakamoto, 2017/02/15 06:35:30]

Oh I see. Let's keep it.

[sunjian, 2017/02/15 20:30:28]

Acknowledged.

 #include "platform/network/mime/MIMETypeRegistry.h"
 #include "platform/plugins/PluginData.h"
 #include "platform/weborigin/SchemeRegistry.h"
 #include "platform/weborigin/SecurityPolicy.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebDocumentSubresourceFilter.h"
 #include "wtf/Assertions.h"
 #include "wtf/AutoReset.h"
 #include "wtf/text/WTFString.h"
-#include <memory>
 
 namespace blink {
 
 static bool isArchiveMIMEType(const String& mimeType) {
   return equalIgnoringCase("multipart/related", mimeType);
 }
 
 static bool shouldInheritSecurityOriginFromOwner(const KURL& url) {
   // https://html.spec.whatwg.org/multipage/browsers.html#origin
   //
@@ skipping 68 matching lines @@
   visitor->trace(m_applicationCacheHost);
   visitor->trace(m_contentSecurityPolicy);
   RawResourceClient::trace(visitor);
 }
 
 unsigned long DocumentLoader::mainResourceIdentifier() const {
   return m_mainResource ? m_mainResource->identifier() : 0;
 }
 
 ResourceTimingInfo* DocumentLoader::getNavigationTimingInfo() const {
+  DCHECK(fetcher());
   return fetcher()->getNavigationTimingInfo();
 }
 
 const ResourceRequest& DocumentLoader::originalRequest() const {
   return m_originalRequest;
 }
 
 const ResourceRequest& DocumentLoader::getRequest() const {
   return m_request;
 }
@@ skipping 637 matching lines @@
                              m_writer ? m_writer->encoding() : emptyAtom, true,
                              ForceSynchronousParsing);
   if (!source.isNull())
     m_writer->appendReplacingData(source);
   endWriting();
 }
 
 DEFINE_WEAK_IDENTIFIER_MAP(DocumentLoader);
 
 }  // namespace blink