Commit 3ce17df4 authored by Zhuoyu Qian's avatar Zhuoyu Qian Committed by Commit Bot

Rename GetText() to GetTextState().

This CL rename the TextIteratorAlgorithm member GetText() to GetTextState().

Change-Id: I782bd5b8faaea7f220e15f5c3a8d929d40f17422
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2376928Reviewed-by: default avatarXiaocheng Hu <xiaochengh@chromium.org>
Commit-Queue: Zhuoyu Qian <zhuoyu.qian@samsung.com>
Cr-Commit-Position: refs/heads/master@{#802047}
parent 83392f74
...@@ -1053,7 +1053,7 @@ static String CreatePlainText(const EphemeralRangeTemplate<Strategy>& range, ...@@ -1053,7 +1053,7 @@ static String CreatePlainText(const EphemeralRangeTemplate<Strategy>& range,
builder.ReserveCapacity(kInitialCapacity); builder.ReserveCapacity(kInitialCapacity);
for (; !it.AtEnd(); it.Advance()) for (; !it.AtEnd(); it.Advance())
it.GetText().AppendTextToStringBuilder(builder); it.GetTextState().AppendTextToStringBuilder(builder);
if (builder.IsEmpty()) if (builder.IsEmpty())
return g_empty_string; return g_empty_string;
......
...@@ -87,8 +87,7 @@ class TextIteratorAlgorithm { ...@@ -87,8 +87,7 @@ class TextIteratorAlgorithm {
// Returns the position after |char16_offset| in current text run. // Returns the position after |char16_offset| in current text run.
PositionTemplate<Strategy> GetPositionAfter(int char16_offset) const; PositionTemplate<Strategy> GetPositionAfter(int char16_offset) const;
// TODO(xiaochengh): Rename to |GetTextState()|. const TextIteratorTextState& GetTextState() const { return text_state_; }
const TextIteratorTextState& GetText() const { return text_state_; }
int length() const { return text_state_.length(); } int length() const { return text_state_.length(); }
UChar CharacterAt(unsigned index) const { UChar CharacterAt(unsigned index) const {
return text_state_.CharacterAt(index); return text_state_.CharacterAt(index);
......
...@@ -144,7 +144,7 @@ std::string TextIteratorTest::IterateWithIterator( ...@@ -144,7 +144,7 @@ std::string TextIteratorTest::IterateWithIterator(
StringBuilder text_chunks; StringBuilder text_chunks;
for (; !iterator.AtEnd(); iterator.Advance()) { for (; !iterator.AtEnd(); iterator.Advance()) {
text_chunks.Append('['); text_chunks.Append('[');
text_chunks.Append(iterator.GetText().GetTextForTesting()); text_chunks.Append(iterator.GetTextState().GetTextForTesting());
text_chunks.Append(']'); text_chunks.Append(']');
} }
return text_chunks.ToString().Utf8(); return text_chunks.ToString().Utf8();
...@@ -701,14 +701,14 @@ TEST_P(TextIteratorTest, StartAtFirstLetter) { ...@@ -701,14 +701,14 @@ TEST_P(TextIteratorTest, StartAtFirstLetter) {
TextIterator iter(start, end); TextIterator iter(start, end);
EXPECT_FALSE(iter.AtEnd()); EXPECT_FALSE(iter.AtEnd());
EXPECT_EQ("A", iter.GetText().GetTextForTesting()); EXPECT_EQ("A", iter.GetTextState().GetTextForTesting());
EXPECT_EQ(text, iter.CurrentContainer()); EXPECT_EQ(text, iter.CurrentContainer());
EXPECT_EQ(Position(text, 0), iter.StartPositionInCurrentContainer()); EXPECT_EQ(Position(text, 0), iter.StartPositionInCurrentContainer());
EXPECT_EQ(Position(text, 1), iter.EndPositionInCurrentContainer()); EXPECT_EQ(Position(text, 1), iter.EndPositionInCurrentContainer());
iter.Advance(); iter.Advance();
EXPECT_FALSE(iter.AtEnd()); EXPECT_FALSE(iter.AtEnd());
EXPECT_EQ("xyz", iter.GetText().GetTextForTesting()); EXPECT_EQ("xyz", iter.GetTextState().GetTextForTesting());
EXPECT_EQ(text, iter.CurrentContainer()); EXPECT_EQ(text, iter.CurrentContainer());
EXPECT_EQ(Position(text, 1), iter.StartPositionInCurrentContainer()); EXPECT_EQ(Position(text, 1), iter.StartPositionInCurrentContainer());
EXPECT_EQ(Position(text, 4), iter.EndPositionInCurrentContainer()); EXPECT_EQ(Position(text, 4), iter.EndPositionInCurrentContainer());
...@@ -728,21 +728,21 @@ TEST_P(TextIteratorTest, StartInMultiCharFirstLetterWithCollapsedSpace) { ...@@ -728,21 +728,21 @@ TEST_P(TextIteratorTest, StartInMultiCharFirstLetterWithCollapsedSpace) {
TextIterator iter(start, end); TextIterator iter(start, end);
EXPECT_FALSE(iter.AtEnd()); EXPECT_FALSE(iter.AtEnd());
EXPECT_EQ("A)", iter.GetText().GetTextForTesting()); EXPECT_EQ("A)", iter.GetTextState().GetTextForTesting());
EXPECT_EQ(text, iter.CurrentContainer()); EXPECT_EQ(text, iter.CurrentContainer());
EXPECT_EQ(Position(text, 3), iter.StartPositionInCurrentContainer()); EXPECT_EQ(Position(text, 3), iter.StartPositionInCurrentContainer());
EXPECT_EQ(Position(text, 5), iter.EndPositionInCurrentContainer()); EXPECT_EQ(Position(text, 5), iter.EndPositionInCurrentContainer());
iter.Advance(); iter.Advance();
EXPECT_FALSE(iter.AtEnd()); EXPECT_FALSE(iter.AtEnd());
EXPECT_EQ(" ", iter.GetText().GetTextForTesting()); EXPECT_EQ(" ", iter.GetTextState().GetTextForTesting());
EXPECT_EQ(text, iter.CurrentContainer()); EXPECT_EQ(text, iter.CurrentContainer());
EXPECT_EQ(Position(text, 5), iter.StartPositionInCurrentContainer()); EXPECT_EQ(Position(text, 5), iter.StartPositionInCurrentContainer());
EXPECT_EQ(Position(text, 6), iter.EndPositionInCurrentContainer()); EXPECT_EQ(Position(text, 6), iter.EndPositionInCurrentContainer());
iter.Advance(); iter.Advance();
EXPECT_FALSE(iter.AtEnd()); EXPECT_FALSE(iter.AtEnd());
EXPECT_EQ("xyz", iter.GetText().GetTextForTesting()); EXPECT_EQ("xyz", iter.GetTextState().GetTextForTesting());
EXPECT_EQ(text, iter.CurrentContainer()); EXPECT_EQ(text, iter.CurrentContainer());
EXPECT_EQ(Position(text, 7), iter.StartPositionInCurrentContainer()); EXPECT_EQ(Position(text, 7), iter.StartPositionInCurrentContainer());
EXPECT_EQ(Position(text, 10), iter.EndPositionInCurrentContainer()); EXPECT_EQ(Position(text, 10), iter.EndPositionInCurrentContainer());
...@@ -762,7 +762,7 @@ TEST_P(TextIteratorTest, StartAndEndInMultiCharFirstLetterWithCollapsedSpace) { ...@@ -762,7 +762,7 @@ TEST_P(TextIteratorTest, StartAndEndInMultiCharFirstLetterWithCollapsedSpace) {
TextIterator iter(start, end); TextIterator iter(start, end);
EXPECT_FALSE(iter.AtEnd()); EXPECT_FALSE(iter.AtEnd());
EXPECT_EQ("A", iter.GetText().GetTextForTesting()); EXPECT_EQ("A", iter.GetTextState().GetTextForTesting());
EXPECT_EQ(text, iter.CurrentContainer()); EXPECT_EQ(text, iter.CurrentContainer());
EXPECT_EQ(Position(text, 3), iter.StartPositionInCurrentContainer()); EXPECT_EQ(Position(text, 3), iter.StartPositionInCurrentContainer());
EXPECT_EQ(Position(text, 4), iter.EndPositionInCurrentContainer()); EXPECT_EQ(Position(text, 4), iter.EndPositionInCurrentContainer());
...@@ -781,7 +781,7 @@ TEST_P(TextIteratorTest, StartAtRemainingText) { ...@@ -781,7 +781,7 @@ TEST_P(TextIteratorTest, StartAtRemainingText) {
TextIterator iter(start, end); TextIterator iter(start, end);
EXPECT_FALSE(iter.AtEnd()); EXPECT_FALSE(iter.AtEnd());
EXPECT_EQ("xyz", iter.GetText().GetTextForTesting()); EXPECT_EQ("xyz", iter.GetTextState().GetTextForTesting());
EXPECT_EQ(text, iter.CurrentContainer()); EXPECT_EQ(text, iter.CurrentContainer());
EXPECT_EQ(Position(text, 1), iter.StartPositionInCurrentContainer()); EXPECT_EQ(Position(text, 1), iter.StartPositionInCurrentContainer());
EXPECT_EQ(Position(text, 4), iter.EndPositionInCurrentContainer()); EXPECT_EQ(Position(text, 4), iter.EndPositionInCurrentContainer());
...@@ -800,14 +800,14 @@ TEST_P(TextIteratorTest, StartAtFirstLetterInPre) { ...@@ -800,14 +800,14 @@ TEST_P(TextIteratorTest, StartAtFirstLetterInPre) {
TextIterator iter(start, end); TextIterator iter(start, end);
EXPECT_FALSE(iter.AtEnd()); EXPECT_FALSE(iter.AtEnd());
EXPECT_EQ("A", iter.GetText().GetTextForTesting()); EXPECT_EQ("A", iter.GetTextState().GetTextForTesting());
EXPECT_EQ(text, iter.CurrentContainer()); EXPECT_EQ(text, iter.CurrentContainer());
EXPECT_EQ(Position(text, 0), iter.StartPositionInCurrentContainer()); EXPECT_EQ(Position(text, 0), iter.StartPositionInCurrentContainer());
EXPECT_EQ(Position(text, 1), iter.EndPositionInCurrentContainer()); EXPECT_EQ(Position(text, 1), iter.EndPositionInCurrentContainer());
iter.Advance(); iter.Advance();
EXPECT_FALSE(iter.AtEnd()); EXPECT_FALSE(iter.AtEnd());
EXPECT_EQ("xyz", iter.GetText().GetTextForTesting()); EXPECT_EQ("xyz", iter.GetTextState().GetTextForTesting());
EXPECT_EQ(text, iter.CurrentContainer()); EXPECT_EQ(text, iter.CurrentContainer());
EXPECT_EQ(Position(text, 1), iter.StartPositionInCurrentContainer()); EXPECT_EQ(Position(text, 1), iter.StartPositionInCurrentContainer());
EXPECT_EQ(Position(text, 4), iter.EndPositionInCurrentContainer()); EXPECT_EQ(Position(text, 4), iter.EndPositionInCurrentContainer());
...@@ -827,14 +827,14 @@ TEST_P(TextIteratorTest, StartInMultiCharFirstLetterInPre) { ...@@ -827,14 +827,14 @@ TEST_P(TextIteratorTest, StartInMultiCharFirstLetterInPre) {
TextIterator iter(start, end); TextIterator iter(start, end);
EXPECT_FALSE(iter.AtEnd()); EXPECT_FALSE(iter.AtEnd());
EXPECT_EQ("A)", iter.GetText().GetTextForTesting()); EXPECT_EQ("A)", iter.GetTextState().GetTextForTesting());
EXPECT_EQ(text, iter.CurrentContainer()); EXPECT_EQ(text, iter.CurrentContainer());
EXPECT_EQ(Position(text, 1), iter.StartPositionInCurrentContainer()); EXPECT_EQ(Position(text, 1), iter.StartPositionInCurrentContainer());
EXPECT_EQ(Position(text, 3), iter.EndPositionInCurrentContainer()); EXPECT_EQ(Position(text, 3), iter.EndPositionInCurrentContainer());
iter.Advance(); iter.Advance();
EXPECT_FALSE(iter.AtEnd()); EXPECT_FALSE(iter.AtEnd());
EXPECT_EQ("xyz", iter.GetText().GetTextForTesting()); EXPECT_EQ("xyz", iter.GetTextState().GetTextForTesting());
EXPECT_EQ(text, iter.CurrentContainer()); EXPECT_EQ(text, iter.CurrentContainer());
EXPECT_EQ(Position(text, 3), iter.StartPositionInCurrentContainer()); EXPECT_EQ(Position(text, 3), iter.StartPositionInCurrentContainer());
EXPECT_EQ(Position(text, 6), iter.EndPositionInCurrentContainer()); EXPECT_EQ(Position(text, 6), iter.EndPositionInCurrentContainer());
...@@ -854,7 +854,7 @@ TEST_P(TextIteratorTest, StartAndEndInMultiCharFirstLetterInPre) { ...@@ -854,7 +854,7 @@ TEST_P(TextIteratorTest, StartAndEndInMultiCharFirstLetterInPre) {
TextIterator iter(start, end); TextIterator iter(start, end);
EXPECT_FALSE(iter.AtEnd()); EXPECT_FALSE(iter.AtEnd());
EXPECT_EQ("A", iter.GetText().GetTextForTesting()); EXPECT_EQ("A", iter.GetTextState().GetTextForTesting());
EXPECT_EQ(text, iter.CurrentContainer()); EXPECT_EQ(text, iter.CurrentContainer());
EXPECT_EQ(Position(text, 1), iter.StartPositionInCurrentContainer()); EXPECT_EQ(Position(text, 1), iter.StartPositionInCurrentContainer());
EXPECT_EQ(Position(text, 2), iter.EndPositionInCurrentContainer()); EXPECT_EQ(Position(text, 2), iter.EndPositionInCurrentContainer());
...@@ -873,7 +873,7 @@ TEST_P(TextIteratorTest, StartAtRemainingTextInPre) { ...@@ -873,7 +873,7 @@ TEST_P(TextIteratorTest, StartAtRemainingTextInPre) {
TextIterator iter(start, end); TextIterator iter(start, end);
EXPECT_FALSE(iter.AtEnd()); EXPECT_FALSE(iter.AtEnd());
EXPECT_EQ("xyz", iter.GetText().GetTextForTesting()); EXPECT_EQ("xyz", iter.GetTextState().GetTextForTesting());
EXPECT_EQ(text, iter.CurrentContainer()); EXPECT_EQ(text, iter.CurrentContainer());
EXPECT_EQ(Position(text, 1), iter.StartPositionInCurrentContainer()); EXPECT_EQ(Position(text, 1), iter.StartPositionInCurrentContainer());
EXPECT_EQ(Position(text, 4), iter.EndPositionInCurrentContainer()); EXPECT_EQ(Position(text, 4), iter.EndPositionInCurrentContainer());
......
...@@ -121,7 +121,7 @@ NSAttributedString* AttributedSubstringFromRange(const EphemeralRange& range, ...@@ -121,7 +121,7 @@ NSAttributedString* AttributedSubstringFromRange(const EphemeralRange& range,
else else
[attrs removeObjectForKey:NSBackgroundColorAttributeName]; [attrs removeObjectForKey:NSBackgroundColorAttributeName];
String characters = it.GetText().GetTextForTesting(); String characters = it.GetTextState().GetTextForTesting();
characters.Ensure16Bit(); characters.Ensure16Bit();
NSString* substring = NSString* substring =
[[[NSString alloc] initWithCharacters:characters.Characters16() [[[NSString alloc] initWithCharacters:characters.Characters16()
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment