| Index: content/browser/frame_host/navigation_entry_impl.cc
|
| diff --git a/content/browser/frame_host/navigation_entry_impl.cc b/content/browser/frame_host/navigation_entry_impl.cc
|
| index 904a9bf080f35625e24d2e2ea1fce88f89b2fd29..16029f8c8af330017d6940ddef79da44cdb2e339 100644
|
| --- a/content/browser/frame_host/navigation_entry_impl.cc
|
| +++ b/content/browser/frame_host/navigation_entry_impl.cc
|
| @@ -541,11 +541,8 @@ void NavigationEntryImpl::AddOrUpdateFrameEntry(FrameTreeNode* frame_tree_node,
|
| FindFrameEntry(frame_tree_node->parent());
|
| if (!parent_node) {
|
| // The renderer should not send a commit for a subframe before its parent.
|
| - // However, we may see commits of subframes when their parent or ancestor is
|
| - // still the initial about:blank page, and we don't currently keep a
|
| - // FrameNavigationEntry for that. We ignore such commits, similar to how we
|
| - // handle them at the top level.
|
| - // TODO(creis): Consider creating FNEs for initial about:blank commits.
|
| + // TODO(creis): Kill the renderer if we get here.
|
| + NOTREACHED() << "Shouldn't see a commit for a subframe before parent.";
|
| return;
|
| }
|
|
|
| @@ -561,11 +558,9 @@ void NavigationEntryImpl::AddOrUpdateFrameEntry(FrameTreeNode* frame_tree_node,
|
| }
|
| }
|
|
|
| - // No entry exists yet, so create a new one unless it's for about:blank.
|
| + // No entry exists yet, so create a new one.
|
| // Unordered list, since we expect to look up entries by frame sequence number
|
| // or unique name.
|
| - if (url == GURL(url::kAboutBlankURL))
|
| - return;
|
| FrameNavigationEntry* frame_entry = new FrameNavigationEntry(
|
| frame_tree_node_id, item_sequence_number, document_sequence_number,
|
| site_instance, url, referrer);
|
|
|