Cleanup blink-side PlzNavigate logic

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

 /*
  * Copyright (C) 2006, 2007, 2008, 2009 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 116 matching lines @@
     return m_replacesCurrentHistoryItem;
   }
   void setReplacesCurrentHistoryItem(bool replacesCurrentHistoryItem) {
     m_replacesCurrentHistoryItem = replacesCurrentHistoryItem;
   }
 
   bool isCommittedButEmpty() const {
     return m_state >= Committed && !m_dataReceived;
   }
 
+  // Without PlzNavigate, this is only true for a narrow window during
+  // navigation start. For PlzNavigate, a navigation sent to the browser will
+  // leave a dummy DocumentLoader in the NotStarted state until the navigation
+  // actually handled in the renderer.

[ananta, 2017/03/01 23:13:17]

Please rephrase to navigation is actually?

[Nate Chapin, 2017/03/01 23:37:46]

Done.

+  bool didStart() const { return m_state != NotStarted; }
+
   void setSentDidFinishLoad() { m_state = SentDidFinishLoad; }
   bool sentDidFinishLoad() const { return m_state == SentDidFinishLoad; }
 
   FrameLoadType loadType() const { return m_loadType; }
   void setLoadType(FrameLoadType loadType) { m_loadType = loadType; }
 
   NavigationType getNavigationType() const { return m_navigationType; }
   void setNavigationType(NavigationType navigationType) {
     m_navigationType = navigationType;
   }
@@ skipping 146 matching lines @@
   // Used to protect against reentrancy into dataReceived().
   bool m_inDataReceived;
   RefPtr<SharedBuffer> m_dataBuffer;
 };
 
 DECLARE_WEAK_IDENTIFIER_MAP(DocumentLoader);
 
 }  // namespace blink
 
 #endif  // DocumentLoader_h