Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(125)

Side by Side Diff: fpdfsdk/fpdfppo.cpp

Issue 2345063002: Use string pools in some dictionaries (Closed)
Patch Set: more Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 PDFium Authors. All rights reserved. 1 // Copyright 2014 PDFium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com 5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
6 6
7 #include "public/fpdf_ppo.h" 7 #include "public/fpdf_ppo.h"
8 8
9 #include <map> 9 #include <map>
10 #include <memory> 10 #include <memory>
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
63 DInfoDict->SetFor("Producer", new CPDF_String(producerstr, FALSE)); 63 DInfoDict->SetFor("Producer", new CPDF_String(producerstr, FALSE));
64 64
65 CFX_ByteString cbRootType = pNewRoot->GetStringFor("Type", ""); 65 CFX_ByteString cbRootType = pNewRoot->GetStringFor("Type", "");
66 if (cbRootType.IsEmpty()) 66 if (cbRootType.IsEmpty())
67 pNewRoot->SetFor("Type", new CPDF_Name("Catalog")); 67 pNewRoot->SetFor("Type", new CPDF_Name("Catalog"));
68 68
69 CPDF_Object* pElement = pNewRoot->GetObjectFor("Pages"); 69 CPDF_Object* pElement = pNewRoot->GetObjectFor("Pages");
70 CPDF_Dictionary* pNewPages = 70 CPDF_Dictionary* pNewPages =
71 pElement ? ToDictionary(pElement->GetDirect()) : nullptr; 71 pElement ? ToDictionary(pElement->GetDirect()) : nullptr;
72 if (!pNewPages) { 72 if (!pNewPages) {
73 pNewPages = new CPDF_Dictionary; 73 pNewPages = new CPDF_Dictionary(pDestPDFDoc->GetByteStringPool());
74 uint32_t NewPagesON = pDestPDFDoc->AddIndirectObject(pNewPages); 74 uint32_t NewPagesON = pDestPDFDoc->AddIndirectObject(pNewPages);
75 pNewRoot->SetFor("Pages", new CPDF_Reference(pDestPDFDoc, NewPagesON)); 75 pNewRoot->SetFor("Pages", new CPDF_Reference(pDestPDFDoc, NewPagesON));
76 } 76 }
77 77
78 CFX_ByteString cbPageType = pNewPages->GetStringFor("Type", ""); 78 CFX_ByteString cbPageType = pNewPages->GetStringFor("Type", "");
79 if (cbPageType == "") { 79 if (cbPageType == "") {
80 pNewPages->SetFor("Type", new CPDF_Name("Pages")); 80 pNewPages->SetFor("Type", new CPDF_Name("Pages"));
81 } 81 }
82 82
83 CPDF_Array* pKeysArray = pNewPages->GetArrayFor("Kids"); 83 CPDF_Array* pKeysArray = pNewPages->GetArrayFor("Kids");
(...skipping 315 matching lines...) Expand 10 before | Expand all | Expand 10 after
399 if (!pSrcDict) 399 if (!pSrcDict)
400 return FALSE; 400 return FALSE;
401 401
402 CPDF_Dictionary* pDstDict = pDstDoc->GetRoot(); 402 CPDF_Dictionary* pDstDict = pDstDoc->GetRoot();
403 if (!pDstDict) 403 if (!pDstDict)
404 return FALSE; 404 return FALSE;
405 405
406 pDstDict->SetFor("ViewerPreferences", pSrcDict->CloneDirectObject()); 406 pDstDict->SetFor("ViewerPreferences", pSrcDict->CloneDirectObject());
407 return TRUE; 407 return TRUE;
408 } 408 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698