diff options
author | Nicolas Pena <npm@chromium.org> | 2017-03-20 15:20:58 -0400 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2017-03-20 20:05:03 +0000 |
commit | 35292ad2070aa3cabdef4ea23ffaaf38e1a7de3a (patch) | |
tree | 0d651eb2e571c9f881826d4e2744ae9f7ac935e1 /core/fpdfapi/edit/cpdf_pagecontentgenerator_unittest.cpp | |
parent | 20df310406636d19a3238ea300bf303a7cd44d2e (diff) | |
download | pdfium-35292ad2070aa3cabdef4ea23ffaaf38e1a7de3a.tar.xz |
Allow processing fonts with indirect dict
BUG=pdfium:667
Change-Id: I6e7620948086f3f620e4cde778a4144570e1bd6d
Reviewed-on: https://pdfium-review.googlesource.com/3106
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Nicolás Peña <npm@chromium.org>
Diffstat (limited to 'core/fpdfapi/edit/cpdf_pagecontentgenerator_unittest.cpp')
-rw-r--r-- | core/fpdfapi/edit/cpdf_pagecontentgenerator_unittest.cpp | 56 |
1 files changed, 55 insertions, 1 deletions
diff --git a/core/fpdfapi/edit/cpdf_pagecontentgenerator_unittest.cpp b/core/fpdfapi/edit/cpdf_pagecontentgenerator_unittest.cpp index b9510b7803..331cec3813 100644 --- a/core/fpdfapi/edit/cpdf_pagecontentgenerator_unittest.cpp +++ b/core/fpdfapi/edit/cpdf_pagecontentgenerator_unittest.cpp @@ -10,7 +10,9 @@ #include "core/fpdfapi/page/cpdf_pathobject.h" #include "core/fpdfapi/page/cpdf_textobject.h" #include "core/fpdfapi/parser/cpdf_document.h" +#include "core/fpdfapi/parser/cpdf_name.h" #include "core/fpdfapi/parser/cpdf_parser.h" +#include "core/fpdfapi/parser/cpdf_reference.h" #include "testing/gtest/include/gtest/gtest.h" #include "third_party/base/ptr_util.h" @@ -151,7 +153,8 @@ TEST_F(CPDF_PageContentGeneratorTest, ProcessGraphics) { pathString2.Mid(55, pathString2.GetLength() - 83)); } -TEST_F(CPDF_PageContentGeneratorTest, ProcessText) { +TEST_F(CPDF_PageContentGeneratorTest, ProcessStandardText) { + // Checking font whose font dictionary is not yet indirect object. auto pDoc = pdfium::MakeUnique<CPDF_Document>(nullptr); pDoc->CreateNewDoc(); CPDF_Dictionary* pPageDict = pDoc->CreateNewPage(0); @@ -176,3 +179,54 @@ TEST_F(CPDF_PageContentGeneratorTest, ProcessText) { EXPECT_EQ("Type1", fontDict->GetStringFor("Subtype")); EXPECT_EQ("Times-Roman", fontDict->GetStringFor("BaseFont")); } + +TEST_F(CPDF_PageContentGeneratorTest, ProcessText) { + // Checking font whose font dictionary is already an indirect object. + auto pDoc = pdfium::MakeUnique<CPDF_Document>(nullptr); + pDoc->CreateNewDoc(); + CPDF_Dictionary* pPageDict = pDoc->CreateNewPage(0); + auto pTestPage = pdfium::MakeUnique<CPDF_Page>(pDoc.get(), pPageDict, false); + CPDF_PageContentGenerator generator(pTestPage.get()); + + CFX_ByteTextBuf buf; + { + // Set the text object font and text + auto pTextObj = pdfium::MakeUnique<CPDF_TextObject>(); + CPDF_Dictionary* pDict = pDoc->NewIndirect<CPDF_Dictionary>(); + pDict->SetNewFor<CPDF_Name>("Type", "Font"); + pDict->SetNewFor<CPDF_Name>("Subtype", "TrueType"); + CPDF_Font* pFont = CPDF_Font::GetStockFont(pDoc.get(), "Arial"); + pDict->SetNewFor<CPDF_Name>("BaseFont", pFont->GetBaseFont()); + + CPDF_Dictionary* pDesc = pDoc->NewIndirect<CPDF_Dictionary>(); + pDesc->SetNewFor<CPDF_Name>("Type", "FontDescriptor"); + pDesc->SetNewFor<CPDF_Name>("FontName", pFont->GetBaseFont()); + pDict->SetNewFor<CPDF_Reference>("FontDescriptor", pDoc.get(), + pDesc->GetObjNum()); + + CPDF_Font* loadedFont = pDoc->LoadFont(pDict); + pTextObj->m_TextState.SetFont(loadedFont); + pTextObj->m_TextState.SetFontSize(15.5f); + pTextObj->SetText("I am indirect"); + + TestProcessText(&generator, &buf, pTextObj.get()); + } + + CFX_ByteString textString = buf.MakeString(); + EXPECT_LT(63, textString.GetLength()); + EXPECT_EQ("BT 1 0 0 1 0 0 Tm /", textString.Left(19)); + EXPECT_EQ(" 15.5 Tf <4920616D20696E646972656374> Tj ET\n", + textString.Right(44)); + CPDF_Dictionary* fontDict = TestGetResource( + &generator, "Font", textString.Mid(19, textString.GetLength() - 63)); + ASSERT_TRUE(fontDict); + EXPECT_TRUE(fontDict->GetObjNum()); + EXPECT_EQ("Font", fontDict->GetStringFor("Type")); + EXPECT_EQ("TrueType", fontDict->GetStringFor("Subtype")); + EXPECT_EQ("Helvetica", fontDict->GetStringFor("BaseFont")); + CPDF_Dictionary* fontDesc = fontDict->GetDictFor("FontDescriptor"); + ASSERT_TRUE(fontDesc); + EXPECT_TRUE(fontDesc->GetObjNum()); + EXPECT_EQ("FontDescriptor", fontDesc->GetStringFor("Type")); + EXPECT_EQ("Helvetica", fontDesc->GetStringFor("FontName")); +} |