summaryrefslogtreecommitdiff
path: root/xfa/fde
diff options
context:
space:
mode:
Diffstat (limited to 'xfa/fde')
-rw-r--r--xfa/fde/cfde_texteditengine.cpp44
-rw-r--r--xfa/fde/cfde_texteditengine.h10
-rw-r--r--xfa/fde/cfde_texteditengine_unittest.cpp149
3 files changed, 127 insertions, 76 deletions
diff --git a/xfa/fde/cfde_texteditengine.cpp b/xfa/fde/cfde_texteditengine.cpp
index 8075ea7a98..0c64e39aeb 100644
--- a/xfa/fde/cfde_texteditengine.cpp
+++ b/xfa/fde/cfde_texteditengine.cpp
@@ -588,31 +588,29 @@ void CFDE_TextEditEngine::SelectAll() {
has_selection_ = true;
selection_.start_idx = 0;
- selection_.end_idx = text_length_ - 1;
+ selection_.count = text_length_;
}
void CFDE_TextEditEngine::ClearSelection() {
has_selection_ = false;
selection_.start_idx = 0;
- selection_.end_idx = 0;
+ selection_.count = 0;
}
-void CFDE_TextEditEngine::SetSelection(size_t start_idx, size_t end_idx) {
- // If the points are the same, then we pretend the selection doesn't exist
- // anymore.
- if (start_idx == end_idx) {
+void CFDE_TextEditEngine::SetSelection(size_t start_idx, size_t count) {
+ if (count == 0) {
ClearSelection();
return;
}
if (start_idx > text_length_)
return;
- if (end_idx > text_length_)
- end_idx = text_length_ - 1;
+ if (start_idx + count > text_length_)
+ count = text_length_ - start_idx;
has_selection_ = true;
selection_.start_idx = start_idx;
- selection_.end_idx = end_idx;
+ selection_.count = count;
}
WideString CFDE_TextEditEngine::GetSelectedText() const {
@@ -621,22 +619,30 @@ WideString CFDE_TextEditEngine::GetSelectedText() const {
WideString text;
if (selection_.start_idx < gap_position_) {
- if (selection_.end_idx < gap_position_) {
+ // Fully on left of gap.
+ if (selection_.start_idx + selection_.count < gap_position_) {
text += WideStringView(content_.data() + selection_.start_idx,
- selection_.end_idx - selection_.start_idx + 1);
+ selection_.count);
return text;
}
+ // Pre-gap text
text += WideStringView(content_.data() + selection_.start_idx,
gap_position_ - selection_.start_idx);
- text += WideStringView(
- content_.data() + gap_position_ + gap_size_,
- selection_.end_idx - (gap_position_ - selection_.start_idx) + 1);
+
+ if (selection_.count - (gap_position_ - selection_.start_idx) > 0) {
+ // Post-gap text
+ text += WideStringView(
+ content_.data() + gap_position_ + gap_size_,
+ selection_.count - (gap_position_ - selection_.start_idx));
+ }
+
return text;
}
+ // Fully right of gap
text += WideStringView(content_.data() + gap_size_ + selection_.start_idx,
- selection_.end_idx - selection_.start_idx + 1);
+ selection_.count);
return text;
}
@@ -645,8 +651,7 @@ WideString CFDE_TextEditEngine::DeleteSelectedText(
if (!has_selection_)
return L"";
- return Delete(selection_.start_idx,
- selection_.end_idx - selection_.start_idx + 1, add_operation);
+ return Delete(selection_.start_idx, selection_.count, add_operation);
}
WideString CFDE_TextEditEngine::Delete(size_t start_idx,
@@ -965,6 +970,9 @@ std::vector<CFX_RectF> CFDE_TextEditEngine::GetCharacterRectsInRange(
std::pair<size_t, size_t> CFDE_TextEditEngine::BoundsForWordAt(
size_t idx) const {
+ if (idx > text_length_)
+ return {0, 0};
+
CFDE_TextEditEngine::Iterator iter(this);
iter.SetAt(idx);
iter.FindNextBreakPos(true);
@@ -972,7 +980,7 @@ std::pair<size_t, size_t> CFDE_TextEditEngine::BoundsForWordAt(
iter.FindNextBreakPos(false);
size_t end_idx = iter.GetAt();
- return {start_idx, end_idx};
+ return {start_idx, end_idx - start_idx + 1};
}
CFDE_TextEditEngine::Iterator::Iterator(const CFDE_TextEditEngine* engine)
diff --git a/xfa/fde/cfde_texteditengine.h b/xfa/fde/cfde_texteditengine.h
index 750b62a2e1..40f12e5f76 100644
--- a/xfa/fde/cfde_texteditengine.h
+++ b/xfa/fde/cfde_texteditengine.h
@@ -140,12 +140,12 @@ class CFDE_TextEditEngine {
size_t GetIndexAtEndOfLine(size_t pos) { return 0; }
void SelectAll();
- void SetSelection(size_t start_idx, size_t end_idx);
+ void SetSelection(size_t start_idx, size_t count);
void ClearSelection();
bool HasSelection() const { return has_selection_; }
- // Returns <start, end> indices of the selection.
+ // Returns <start_idx, count> of the selection.
std::pair<size_t, size_t> GetSelection() const {
- return {selection_.start_idx, selection_.end_idx};
+ return {selection_.start_idx, selection_.count};
}
WideString GetSelectedText() const;
WideString DeleteSelectedText(
@@ -159,7 +159,7 @@ class CFDE_TextEditEngine {
size_t GetWidthOfChar(size_t idx);
// Non-const so we can force a Layout() if needed.
size_t GetIndexForPoint(const CFX_PointF& point);
- // <start_idx, end_idx>
+ // <start_idx, count>
std::pair<size_t, size_t> BoundsForWordAt(size_t idx) const;
// Returns <bidi level, character rect>
@@ -197,7 +197,7 @@ class CFDE_TextEditEngine {
struct Selection {
size_t start_idx;
- size_t end_idx;
+ size_t count;
};
CFX_RectF contents_bounding_box_;
diff --git a/xfa/fde/cfde_texteditengine_unittest.cpp b/xfa/fde/cfde_texteditengine_unittest.cpp
index 48ed647945..51940f21a8 100644
--- a/xfa/fde/cfde_texteditengine_unittest.cpp
+++ b/xfa/fde/cfde_texteditengine_unittest.cpp
@@ -244,10 +244,10 @@ TEST_F(CFDE_TextEditEngineTest, Selection) {
engine()->SelectAll();
size_t start_idx;
- size_t end_idx;
- std::tie(start_idx, end_idx) = engine()->GetSelection();
+ size_t count;
+ std::tie(start_idx, count) = engine()->GetSelection();
EXPECT_EQ(0U, start_idx);
- EXPECT_EQ(10U, end_idx);
+ EXPECT_EQ(11U, count);
// Selection before gap.
EXPECT_STREQ(L"Hello World", engine()->GetSelectedText().c_str());
@@ -272,7 +272,7 @@ TEST_F(CFDE_TextEditEngineTest, Selection) {
EXPECT_STREQ(L"", engine()->GetText().c_str());
engine()->Insert(0, L"Hello World");
- engine()->SetSelection(5, 9);
+ engine()->SetSelection(5, 5);
EXPECT_STREQ(L" Worl", engine()->DeleteSelectedText().c_str());
EXPECT_FALSE(engine()->HasSelection());
EXPECT_STREQ(L"Hellod", engine()->GetText().c_str());
@@ -419,112 +419,155 @@ TEST_F(CFDE_TextEditEngineTest, GetIndexForPoint) {
TEST_F(CFDE_TextEditEngineTest, BoundsForWordAt) {
size_t start_idx;
- size_t end_idx;
+ size_t count;
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(100);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(100);
EXPECT_EQ(0U, start_idx);
- EXPECT_EQ(0U, end_idx);
- engine()->SetSelection(start_idx, end_idx);
+ EXPECT_EQ(0U, count);
+ engine()->SetSelection(start_idx, count);
EXPECT_STREQ(L"", engine()->GetSelectedText().c_str());
engine()->Clear();
engine()->Insert(0, L"Hello");
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(0);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(0);
EXPECT_EQ(0U, start_idx);
- EXPECT_EQ(4U, end_idx);
- engine()->SetSelection(start_idx, end_idx);
+ EXPECT_EQ(5U, count);
+ engine()->SetSelection(start_idx, count);
EXPECT_STREQ(L"Hello", engine()->GetSelectedText().c_str());
engine()->Clear();
engine()->Insert(0, L"Hello World");
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(100);
- EXPECT_EQ(11U, start_idx);
- EXPECT_EQ(11U, end_idx);
- engine()->SetSelection(start_idx, end_idx);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(100);
+ EXPECT_EQ(0U, start_idx);
+ EXPECT_EQ(0U, count);
+ engine()->SetSelection(start_idx, count);
EXPECT_STREQ(L"", engine()->GetSelectedText().c_str());
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(0);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(0);
EXPECT_EQ(0U, start_idx);
- EXPECT_EQ(4U, end_idx);
- engine()->SetSelection(start_idx, end_idx);
+ EXPECT_EQ(5U, count);
+ engine()->SetSelection(start_idx, count);
EXPECT_STREQ(L"Hello", engine()->GetSelectedText().c_str());
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(1);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(1);
EXPECT_EQ(0U, start_idx);
- EXPECT_EQ(4U, end_idx);
- engine()->SetSelection(start_idx, end_idx);
+ EXPECT_EQ(5U, count);
+ engine()->SetSelection(start_idx, count);
EXPECT_STREQ(L"Hello", engine()->GetSelectedText().c_str());
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(4);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(4);
EXPECT_EQ(0U, start_idx);
- EXPECT_EQ(4U, end_idx);
- engine()->SetSelection(start_idx, end_idx);
+ EXPECT_EQ(5U, count);
+ engine()->SetSelection(start_idx, count);
EXPECT_STREQ(L"Hello", engine()->GetSelectedText().c_str());
// Select the space
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(5);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(5);
EXPECT_EQ(5U, start_idx);
- EXPECT_EQ(5U, end_idx);
- engine()->SetSelection(start_idx, end_idx);
- EXPECT_STREQ(L"", engine()->GetSelectedText().c_str());
+ EXPECT_EQ(1U, count);
+ engine()->SetSelection(start_idx, count);
+ EXPECT_STREQ(L" ", engine()->GetSelectedText().c_str());
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(6);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(6);
EXPECT_EQ(6U, start_idx);
- EXPECT_EQ(10U, end_idx);
- engine()->SetSelection(start_idx, end_idx);
+ EXPECT_EQ(5U, count);
+ engine()->SetSelection(start_idx, count);
EXPECT_STREQ(L"World", engine()->GetSelectedText().c_str());
engine()->Clear();
engine()->Insert(0, L"123 456 789");
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(5);
- engine()->SetSelection(start_idx, end_idx);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(5);
+ engine()->SetSelection(start_idx, count);
EXPECT_STREQ(L"456", engine()->GetSelectedText().c_str());
engine()->Clear();
engine()->Insert(0, L"123def789");
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(5);
- engine()->SetSelection(start_idx, end_idx);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(5);
+ engine()->SetSelection(start_idx, count);
EXPECT_STREQ(L"123def789", engine()->GetSelectedText().c_str());
engine()->Clear();
engine()->Insert(0, L"abc456ghi");
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(5);
- engine()->SetSelection(start_idx, end_idx);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(5);
+ engine()->SetSelection(start_idx, count);
EXPECT_STREQ(L"abc456ghi", engine()->GetSelectedText().c_str());
engine()->Clear();
engine()->Insert(0, L"hello, world");
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(0);
- engine()->SetSelection(start_idx, end_idx);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(0);
+ engine()->SetSelection(start_idx, count);
EXPECT_STREQ(L"hello", engine()->GetSelectedText().c_str());
engine()->Clear();
engine()->Insert(0, L"hello, world");
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(5);
- engine()->SetSelection(start_idx, end_idx);
- EXPECT_STREQ(L"", engine()->GetSelectedText().c_str());
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(5);
+ engine()->SetSelection(start_idx, count);
+ EXPECT_STREQ(L",", engine()->GetSelectedText().c_str());
engine()->Clear();
engine()->Insert(0, L"np-complete");
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(6);
- engine()->SetSelection(start_idx, end_idx);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(6);
+ engine()->SetSelection(start_idx, count);
EXPECT_STREQ(L"complete", engine()->GetSelectedText().c_str());
engine()->Clear();
engine()->Insert(0, L"(123) 456-7890");
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(0);
- engine()->SetSelection(start_idx, end_idx);
- EXPECT_STREQ(L"", engine()->GetSelectedText().c_str());
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(0);
+ engine()->SetSelection(start_idx, count);
+ EXPECT_STREQ(L"(", engine()->GetSelectedText().c_str());
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(1);
- engine()->SetSelection(start_idx, end_idx);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(1);
+ engine()->SetSelection(start_idx, count);
EXPECT_STREQ(L"123", engine()->GetSelectedText().c_str());
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(7);
- engine()->SetSelection(start_idx, end_idx);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(7);
+ engine()->SetSelection(start_idx, count);
EXPECT_STREQ(L"456", engine()->GetSelectedText().c_str());
- std::tie(start_idx, end_idx) = engine()->BoundsForWordAt(11);
- engine()->SetSelection(start_idx, end_idx);
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(11);
+ engine()->SetSelection(start_idx, count);
EXPECT_STREQ(L"7890", engine()->GetSelectedText().c_str());
+
+ // Tests from:
+ // http://unicode.org/Public/UNIDATA/auxiliary/WordBreakTest.html#samples
+ struct bounds {
+ size_t start;
+ size_t end;
+ };
+ struct {
+ const wchar_t* str;
+ std::vector<const wchar_t*> results;
+ } tests[] = {
+ // {L"\r\na\n\u0308", {L"\r\n", L"a", L"\n", L"\u0308"}},
+ // {L"a\u0308", {L"a\u0308"}},
+ // {L" \u200d\u0646", {L" \u200d", L"\u0646"}},
+ // {L"\u0646\u200d ", {L"\u0646\u200d", L" "}},
+ {L"AAA", {L"AAA"}},
+ {L"A:A", {L"A:A"}},
+ {L"A::A", {L"A", L":", L":", L"A"}},
+ // {L"\u05d0'", {L"\u05d0'"}},
+ // {L"\u05d0\"\u05d0", {L"\u05d0\"\u05d0"}},
+ {L"A00A", {L"A00A"}},
+ {L"0,0", {L"0,0"}},
+ {L"0,,0", {L"0", L",", L",", L"0"}},
+ {L"\u3031\u3031", {L"\u3031\u3031"}},
+ {L"A_0_\u3031_", {L"A_0_\u3031_"}},
+ {L"A__A", {L"A__A"}},
+ // {L"\u200d\u2640", {L"\u200d\u2640"}},
+ // {L"a\u0308\u200b\u0308b", {L"a\u0308\u200b\u0308b"}},
+ };
+
+ for (auto t : tests) {
+ engine()->Clear();
+ engine()->Insert(0, t.str);
+
+ size_t idx = 0;
+ for (const auto* res : t.results) {
+ std::tie(start_idx, count) = engine()->BoundsForWordAt(idx);
+ engine()->SetSelection(start_idx, count);
+ EXPECT_STREQ(res, engine()->GetSelectedText().c_str())
+ << "Input: '" << t.str << "'";
+ idx += count;
+ }
+ }
}