Create FrameNavigationEntries for the initial blank page in subframes.

content/browser/frame_host/frame_tree_node.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_FRAME_HOST_FRAME_TREE_NODE_H_
 #define CONTENT_BROWSER_FRAME_HOST_FRAME_TREE_NODE_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 93 matching lines @@
   void SetOpener(FrameTreeNode* opener);
 
   FrameTreeNode* child_at(size_t index) const {
     return children_[index];
   }
 
   const GURL& current_url() const {
     return current_url_;
   }
 
-  void set_current_url(const GURL& url) {
-    current_url_ = url;
+  // Sets the last committed URL for this frame and updates
+  // has_committed_real_load accordingly.
+  void SetCurrentURL(const GURL& url);
+
+  bool has_committed_real_load() const {

[Avi (use Gerrit), 2015/07/23 20:27:58]

// Returns true iff SetCurrentURL has been called

?

[Charlie Reis, 2015/07/23 22:02:22]

...with a non-blank URL.

Done.

+    return has_committed_real_load_;
   }
 
   // Set the current origin and notify proxies about the update.
   void SetCurrentOrigin(const url::DeprecatedSerializedOrigin& origin);
 
   // Set the current name and notify proxies about the update.
   void SetFrameName(const std::string& name);
 
   blink::WebSandboxFlags effective_sandbox_flags() {
     return effective_sandbox_flags_;
@@ skipping 108 matching lines @@
   // An observer that clears this node's |opener_| if the opener is destroyed.
   // This observer is added to the |opener_|'s observer list when the |opener_|
   // is set to a non-null node, and it is removed from that list when |opener_|
   // changes or when this node is destroyed.  It is also cleared if |opener_|
   // is disowned.
   scoped_ptr<OpenerDestroyedObserver> opener_observer_;
 
   // The immediate children of this specific frame.
   ScopedVector<FrameTreeNode> children_;
 
-  // Track the current frame's last committed URL, so we can estimate the
-  // process impact of out-of-process iframes.
-  // TODO(creis): Remove this when we can store subframe URLs in the
-  // NavigationController.
+  // Track the current frame's last committed URL.
+  // TODO(creis): Consider storing a reference to the last committed
+  // FrameNavigationEntry here once those are created in all modes.
   GURL current_url_;
 
+  // Whether this frame has committed any real load, replacing its initial
+  // about:blank page.
+  bool has_committed_real_load_;
+
   // Track information that needs to be replicated to processes that have
   // proxies for this frame.
   FrameReplicationState replication_state_;
 
   // Track the effective sandbox flags for this frame.  When a parent frame
   // dynamically updates sandbox flags for a child frame, the child's updated
   // sandbox flags are stored in replication_state_.sandbox_flags. However, the
   // update only takes effect on the next frame navigation, so the effective
   // sandbox flags are tracked separately here.  When enforcing sandbox flags
   // directives in the browser process, |effective_sandbox_flags_| should be
@@ skipping 11 matching lines @@
 
   // List of objects observing this FrameTreeNode.
   base::ObserverList<Observer> observers_;
 
   DISALLOW_COPY_AND_ASSIGN(FrameTreeNode);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_FRAME_TREE_NODE_H_