DocumentLoader::isCommittedButEmpty is timing-dependent.

Source/core/loader/DocumentLoader.h

Index: Source/core/loader/DocumentLoader.h
diff --git a/Source/core/loader/DocumentLoader.h b/Source/core/loader/DocumentLoader.h
index 43fd4d53f10bdc57efa5caa7df651072ed759ee2..083e9ede1c88842b89a6dc64a7cb2aaa8c1980e3 100644
--- a/Source/core/loader/DocumentLoader.h
+++ b/Source/core/loader/DocumentLoader.h
@@ -104,7 +104,7 @@ public:
     bool replacesCurrentHistoryItem() const { return m_replacesCurrentHistoryItem; }
     void setReplacesCurrentHistoryItem(bool replacesCurrentHistoryItem) { m_replacesCurrentHistoryItem = replacesCurrentHistoryItem; }
 
-    bool isCommittedButEmpty() const { return m_state == Committed; }
+    bool isCommittedButEmpty() const { return m_state == Committed || m_state == MainResourceDoneEmpty; }
 
     bool shouldContinueForNavigationPolicy(const ResourceRequest&, ContentSecurityPolicyDisposition shouldCheckMainWorldContentSecurityPolicy, NavigationPolicy = NavigationPolicyCurrentTab);
     NavigationType navigationType() const { return m_navigationType; }
@@ -234,7 +234,8 @@ private:
         Provisional,
         Committed,
         DataReceived,
-        MainResourceDone
+        MainResourceDone,
+        MainResourceDoneEmpty

[dcheng, 2015/09/21 16:43:52]

Would it make sense to call this "DoneButEmpty" to match the "CommittedButEmpty"
name earlier?

[Nate Chapin, 2015/09/21 16:45:30]

Yeah, probably. Will update before landing.

     };
     State m_state;