chromium/src
0
Fork 0
Code Activity

Refactor common test code to compare PDF text runs

Browse Source 
This CL simplifies the code in pdfium_page_unittest.cc by refactoring
similar implementations of comparing PDF text runs into a new method.

Bug: 1008775
Change-Id: Ia6eedf03d3fb01118c919ae2f464bec0de842824
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946190
Commit-Queue: Kalpak Tapas <katapas@microsoft.com>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#721855}
This commit is contained in:
Kalpak Tapas
2019-12-05 04:31:59 +00:00
committed by Commit Bot
parent 02a9cb76bb
commit f960fffd60
1 changed files with 28 additions and 32 deletions

60
pdf/pdfium/pdfium_page_unittest.cc

@ -33,6 +33,28 @@ TEST(PDFiumPageHelperDeathTest, ToPDFiumRotation) {
#endif
}
void CompareTextRuns(
const pp::PDF::PrivateAccessibilityTextRunInfo& expected_text_run,
const pp::PDF::PrivateAccessibilityTextRunInfo actual_text_run) {
EXPECT_EQ(expected_text_run.len, actual_text_run.len);
CompareRect(expected_text_run.bounds, actual_text_run.bounds);
EXPECT_EQ(expected_text_run.direction, actual_text_run.direction);
const pp::PDF::PrivateAccessibilityTextStyleInfo& expected_style =
expected_text_run.style;
const pp::PDF::PrivateAccessibilityTextStyleInfo& actual_style =
actual_text_run.style;
EXPECT_EQ(expected_style.font_name, actual_style.font_name);
EXPECT_EQ(expected_style.font_weight, actual_style.font_weight);
EXPECT_EQ(expected_style.render_mode, actual_style.render_mode);
EXPECT_EQ(expected_style.font_size, actual_style.font_size);
EXPECT_EQ(expected_style.fill_color, actual_style.fill_color);
EXPECT_EQ(expected_style.stroke_color, actual_style.stroke_color);
EXPECT_EQ(expected_style.is_italic, actual_style.is_italic);
EXPECT_EQ(expected_style.is_bold, actual_style.is_bold);
}
} // namespace
using PDFiumPageTest = PDFiumTestBase;
@ -264,23 +286,9 @@ TEST_F(PDFiumPageTextTest, GetTextRunInfo) {
for (const auto& expected_text_run : expected_text_runs) {
text_run_info_result = engine->GetTextRunInfo(0, current_char_index);
ASSERT_TRUE(text_run_info_result.has_value());
const auto& text_run_info = text_run_info_result.value();
EXPECT_EQ(expected_text_run.len, text_run_info.len);
CompareRect(expected_text_run.bounds, text_run_info.bounds);
const pp::PDF::PrivateAccessibilityTextStyleInfo& expected_style =
expected_text_run.style;
const pp::PDF::PrivateAccessibilityTextStyleInfo& text_run_info_style =
text_run_info.style;
EXPECT_EQ(expected_text_run.direction, text_run_info.direction);
EXPECT_EQ(expected_style.font_name, text_run_info_style.font_name);
EXPECT_EQ(expected_style.font_weight, text_run_info_style.font_weight);
EXPECT_EQ(expected_style.render_mode, text_run_info_style.render_mode);
EXPECT_EQ(expected_style.font_size, text_run_info_style.font_size);
EXPECT_EQ(expected_style.fill_color, text_run_info_style.fill_color);
EXPECT_EQ(expected_style.stroke_color, text_run_info_style.stroke_color);
EXPECT_EQ(expected_style.is_italic, text_run_info_style.is_italic);
EXPECT_EQ(expected_text_run.style.is_bold, text_run_info_style.is_bold);
current_char_index += text_run_info.len;
const auto& actual_text_run = text_run_info_result.value();
CompareTextRuns(expected_text_run, actual_text_run);
current_char_index += actual_text_run.len;
}
// Test char index outside char range returns nullopt
@ -332,21 +340,9 @@ TEST_F(PDFiumPageTextTest, TestHighlightTextRunInfo) {
base::Optional<pp::PDF::PrivateAccessibilityTextRunInfo>
text_run_info_result = engine->GetTextRunInfo(0, current_char_index);
ASSERT_TRUE(text_run_info_result.has_value());
const auto& text_run_info = text_run_info_result.value();
EXPECT_EQ(expected_text_run.len, text_run_info.len);
CompareRect(expected_text_run.bounds, text_run_info.bounds);
EXPECT_EQ(expected_text_run.direction, text_run_info.direction);
EXPECT_EQ(kExpectedStyle.font_name, text_run_info.style.font_name);
EXPECT_EQ(kExpectedStyle.font_weight, text_run_info.style.font_weight);
EXPECT_EQ(kExpectedStyle.render_mode, text_run_info.style.render_mode);
EXPECT_EQ(kExpectedStyle.font_size, text_run_info.style.font_size);
EXPECT_EQ(kExpectedStyle.fill_color, text_run_info.style.fill_color);
EXPECT_EQ(kExpectedStyle.stroke_color, text_run_info.style.stroke_color);
EXPECT_EQ(kExpectedStyle.is_italic, text_run_info.style.is_italic);
EXPECT_EQ(kExpectedStyle.is_bold, text_run_info.style.is_bold);
current_char_index += text_run_info.len;
const auto& actual_text_run = text_run_info_result.value();
CompareTextRuns(expected_text_run, actual_text_run);
current_char_index += actual_text_run.len;
}
}