[All-In-One] Introduce BackspaceStateMachine

third_party/WebKit/Source/core/editing/BackspaceStateMachine.h

+// Copyright 2016 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 BackspaceStateMachine_h
+#define BackspaceStateMachine_h
+
+#include <stdint.h>
+
+namespace blink {
+
+class BackspaceStateMachine {

[yosin_UTC9, 2016/03/25 04:37:23]

Could you write unit test for |BackspaceStateMachine| class?
Note: we need to have |CORE_EXPORT| for class when using class in unit test.

Once we have unit test for |BackspaceStateMachine|, we can commit itself rather
than including changes of existing files.

[Seigo Nonaka, 2016/03/25 06:59:06]

Done.

+public:

[yosin_UTC9, 2016/03/25 04:37:23]

Please insert a following line before |public:|
   WTF_MAKE_NONCOPYABLE(BackspaceStateMachine);


You need to add #include "wtf/Noncopyable.h"

[Seigo Nonaka, 2016/03/25 06:59:06]

Done.

+    BackspaceStateMachine();
+
+    // Returns true when the state machine has stopped.
+    bool ComputeNextState(uint16_t codeUnit);

[yosin_UTC9, 2016/03/25 04:37:23]

Could you use |camelCase| for member function?
We've not moved to Chromium style yet.

[Seigo Nonaka, 2016/03/25 06:59:06]

Done.

+
+    // Finalize the state machine and returns the code unit count to be deleted.
+    // If the state machine hasn't finished, this method finishes the state
+    // machine first.
+    int FinalizeAndGetCodeUnitCountToBeDeleted();
+
+private:
+    int mCodeUnitsToBeDeleted;

[yosin_UTC9, 2016/03/25 04:37:23]

Blink uses |m_camelCase| for member variable.

Let's use Non-Static Class Member Initializers
(http://chromium-cpp.appspot.com/)

s/;/= 0/

[Seigo Nonaka, 2016/03/25 06:59:06]

Done.

+
+    // Used for composing supplementary code point with surrogate pairs.
+    uint16_t mTrailSurrogate;

[yosin_UTC9, 2016/03/25 04:37:23]

s/;/=0/

[Seigo Nonaka, 2016/03/25 06:59:06]

Done.

+};
+
+} // namespace blink
+
+#endif