Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(904)

Unified Diff: Source/core/loader/HistoryItem.h

Issue 418103003: Add a frame sequence number to HistoryItem (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | Source/core/loader/HistoryItem.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/loader/HistoryItem.h
diff --git a/Source/core/loader/HistoryItem.h b/Source/core/loader/HistoryItem.h
index 80a31cb4dbfe128e925733de7646978127b613a2..018e5db24463535ed1065e285efffac89b78bbb2 100644
--- a/Source/core/loader/HistoryItem.h
+++ b/Source/core/loader/HistoryItem.h
@@ -94,6 +94,9 @@ public:
void setDocumentSequenceNumber(long long number) { m_documentSequenceNumber = number; }
long long documentSequenceNumber() const { return m_documentSequenceNumber; }
+ void setFrameSequenceNumber(long long number) { m_frameSequenceNumber = number; }
+ long long frameSequenceNumber() const { return m_frameSequenceNumber; }
+
void setFormInfoFromRequest(const ResourceRequest&);
void setFormData(PassRefPtr<FormData>);
void setFormContentType(const AtomicString&);
@@ -124,6 +127,10 @@ private:
// such HistoryItem to another preserves the document.
int64_t m_documentSequenceNumber;
+ // Used to match frames across serialization/deserialization more reliably
darin (slow to review) 2014/07/25 04:39:06 I think it would be more helpful to add a comment
+ // than is possible using unique names.
+ int64_t m_frameSequenceNumber;
+
// Support for HTML5 History
RefPtr<SerializedScriptValue> m_stateObject;
« no previous file with comments | « no previous file | Source/core/loader/HistoryItem.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698