Add sequence number to undo steps

third_party/WebKit/Source/core/editing/commands/UndoStep.h

 /*
  * 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 47 matching lines @@
   const VisibleSelection& endingSelection() const { return m_endingSelection; }
   void setStartingSelection(const VisibleSelection&);
   void setEndingSelection(const VisibleSelection&);
   Element* startingRootEditableElement() const {
     return m_startingRootEditableElement.get();
   }
   Element* endingRootEditableElement() const {
     return m_endingRootEditableElement.get();
   }
 
+  uint64_t sequence() const { return m_sequence; }

[yosin_UTC9, 2017/01/19 06:40:52]

s/sequence/sequenceNumber()/

Term "sequence" is ambiguous, it can be list, array, etc.

[Xiaocheng, 2017/01/19 07:52:30]

Will do.

+  void setSequence(uint64_t passedSequence) { m_sequence = passedSequence; }
+
   DECLARE_TRACE();
 
  private:
   UndoStep(Document*,
            const VisibleSelection& startingSelection,
            const VisibleSelection& endingSelection,
            InputEvent::InputType);
 
   Member<Document> m_document;
   VisibleSelection m_startingSelection;
   VisibleSelection m_endingSelection;
   HeapVector<Member<SimpleEditCommand>> m_commands;
   Member<Element> m_startingRootEditableElement;
   Member<Element> m_endingRootEditableElement;
   InputEvent::InputType m_inputType;
+  uint64_t m_sequence;

[yosin_UTC9, 2017/01/19 06:40:52]

Can we make |const unit64_t m_sequenceNumber| and initialized in constructor?
I fill changing sequence number is some what strange since usually sequence
number is used for object identifier.

[Xiaocheng, 2017/01/19 07:52:30]

For undo steps, is the creation order same as the push-to-undo-stack order? We
can initialize the sequence number only if this is true.

 };
 
 }  // namespace blink
 
 #endif