Add loadedNonEmptyDocument flag in FrameLoader for Resource Timing

Source/core/loader/cache/CachedResourceLoader.cpp

Index: Source/core/loader/cache/CachedResourceLoader.cpp
diff --git a/Source/core/loader/cache/CachedResourceLoader.cpp b/Source/core/loader/cache/CachedResourceLoader.cpp
index b9bd310bf98a01fe45e043a9696fd8375fed8a97..7ade664101cdefa72f36ddc8f24437e9166029c8 100644
--- a/Source/core/loader/cache/CachedResourceLoader.cpp
+++ b/Source/core/loader/cache/CachedResourceLoader.cpp
@@ -571,7 +571,7 @@ void CachedResourceLoader::storeResourceTimingInitiatorInformation(const CachedR
 {
     if (resource->type() == CachedResource::MainResource) {
         // <iframe>s should report the initial navigation requested by the parent document, but not subsequent navigations.
-        if (frame()->ownerElement() && m_documentLoader->frameLoader()->stateMachine()->committingFirstRealLoad()) {
+        if (frame()->ownerElement() && !frame()->ownerElement()->loadedNonEmptyDocument()) {

[Nate Chapin, 2013/05/07 16:40:55]

Could we set loadedNonEmptyDocument here, rather than in FrameLoader? Or maybe
in CachedResourceLoader::loadDone()? The 5-case if statement in otherwise
unrelated code in FrameLoader worries me.

[Pan, 2013/05/07 23:39:28]

great, why not? I was in trouble to find a place in FrameLoader yesterday, make
them them together is really good. thanks! :)

             InitiatorInfo info = { frame()->ownerElement()->localName(), monotonicallyIncreasingTime() };
             m_initiatorMap.add(resource.get(), info);
         }