Use WTF::String and WTF::Vector in state machine tests

Use WTF::String and WTF::Vector in state machine tests

THis patch replaces |std::string| and |std::vector| usage to |WTF::String| and
|WTF::Vector| in "BackwardGraphemeBoundaryStateMachineTest.cpp" and
"ForwardGraphemeBoundaryStateMachineTest.cpp".

THis patch is follow up of crrev.com/1857633002

BUG=594923
TEST=n/a; no behavior changes

Committed: https://crrev.com/9107fc9d20437051ab0f6ba948458ea53bc2e24e
Cr-Commit-Position: refs/heads/master@{#386925}

[yosin_UTC9, 2016-04-13 05:23:23]

Description was changed from

==========
2016-04-13T13:51:14

Follow up of crrev.com/1857633002

BUG=594923
==========

to

==========
Use WTF::String and WTF::Vector in state machine tests

THis patch replaces |std::string| and |std::vector| usage to |WTF::String| and
|WTF::Vector| in "BackwardGraphemeBoundaryStateMachineTest.cpp" and
"ForwardGraphemeBoundaryStateMachineTest.cpp".

THis patch is follow up of crrev.com/1857633002

BUG=594923
TEST=n/a; no behavior changes
==========

[yosin_UTC9, 2016-04-13 05:24:03]

yosin@chromium.org changed reviewers:
+ tkent@chromium.org

[yosin_UTC9, 2016-04-13 05:24:03]

PTAL

[tkent, 2016-04-13 05:31:55]

https://codereview.chromium.org/1884893002/diff/1/third_party/WebKit/Source/c...
File
third_party/WebKit/Source/core/editing/state_machines/StateMachineTestUtil.h
(right):

https://codereview.chromium.org/1884893002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/editing/state_machines/StateMachineTestUtil.h:17:
WTF_MAKE_NONCOPYABLE(GraphemeStateMachineTestBase);
Can you use DISALLOW_COPY_AND_ASSIGN() from base/macros.h?
WebKit/Source/DEPS allows to include base/macros.h.

There are other WTF_MAKE_NONCOPYABLE instances in this CL.

[yosin_UTC9, 2016-04-13 05:46:07]

On 2016/04/13 at 05:31:55, tkent wrote:
>
https://codereview.chromium.org/1884893002/diff/1/third_party/WebKit/Source/c...
> File
third_party/WebKit/Source/core/editing/state_machines/StateMachineTestUtil.h
(right):
> 
>
https://codereview.chromium.org/1884893002/diff/1/third_party/WebKit/Source/c...
>
third_party/WebKit/Source/core/editing/state_machines/StateMachineTestUtil.h:17:
WTF_MAKE_NONCOPYABLE(GraphemeStateMachineTestBase);
> Can you use DISALLOW_COPY_AND_ASSIGN() from base/macros.h?
> WebKit/Source/DEPS allows to include base/macros.h.
> 
> There are other WTF_MAKE_NONCOPYABLE instances in this CL.

Done. I've never known that we can use |DISALLOW_COPY_AND_ASSIGN()|.

PTAL

[tkent, 2016-04-13 05:49:25]

lgtm

[yosin_UTC9, 2016-04-13 06:13:21]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-04-13 06:13:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1884893002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1884893002/20001

[commit-bot: I haz the power, 2016-04-13 06:52:30]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-13 06:53:45]

Description was changed from

==========
Use WTF::String and WTF::Vector in state machine tests

THis patch replaces |std::string| and |std::vector| usage to |WTF::String| and
|WTF::Vector| in "BackwardGraphemeBoundaryStateMachineTest.cpp" and
"ForwardGraphemeBoundaryStateMachineTest.cpp".

THis patch is follow up of crrev.com/1857633002

BUG=594923
TEST=n/a; no behavior changes
==========

to

==========
Use WTF::String and WTF::Vector in state machine tests

THis patch replaces |std::string| and |std::vector| usage to |WTF::String| and
|WTF::Vector| in "BackwardGraphemeBoundaryStateMachineTest.cpp" and
"ForwardGraphemeBoundaryStateMachineTest.cpp".

THis patch is follow up of crrev.com/1857633002

BUG=594923
TEST=n/a; no behavior changes

Committed: https://crrev.com/9107fc9d20437051ab0f6ba948458ea53bc2e24e
Cr-Commit-Position: refs/heads/master@{#386925}
==========

[commit-bot: I haz the power, 2016-04-13 06:53:46]

Patchset 2 (id:??) landed as
https://crrev.com/9107fc9d20437051ab0f6ba948458ea53bc2e24e
Cr-Commit-Position: refs/heads/master@{#386925}