Index: content/browser/frame_host/frame_navigation_entry.cc |
diff --git a/content/browser/frame_host/frame_navigation_entry.cc b/content/browser/frame_host/frame_navigation_entry.cc |
index 0889b48387d2848ce40450b281f9d670f281416c..fe8809c8fc648d69d48c394beb7269085464b242 100644 |
--- a/content/browser/frame_host/frame_navigation_entry.cc |
+++ b/content/browser/frame_host/frame_navigation_entry.cc |
@@ -59,14 +59,17 @@ |
void FrameNavigationEntry::set_item_sequence_number( |
int64_t item_sequence_number) { |
- // TODO(creis): Assert that this does not change after being assigned, once |
- // location.replace is classified as NEW_PAGE rather than EXISTING_PAGE. |
- // Same for document sequence number. See https://crbug.com/596707. |
+ // Once assigned, the item sequence number shouldn't change. |
+ DCHECK(item_sequence_number_ == -1 || |
+ item_sequence_number_ == item_sequence_number); |
item_sequence_number_ = item_sequence_number; |
} |
void FrameNavigationEntry::set_document_sequence_number( |
int64_t document_sequence_number) { |
+ // Once assigned, the document sequence number shouldn't change. |
+ DCHECK(document_sequence_number_ == -1 || |
+ document_sequence_number_ == document_sequence_number); |
document_sequence_number_ = document_sequence_number; |
} |