Rename methods in CPDF_Interform.

core/fpdfdoc/doc_form.cpp

 // Copyright 2014 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #include <vector>
 
 #include "core/fpdfapi/fpdf_font/include/cpdf_fontencoding.h"
 #include "core/fpdfapi/fpdf_page/include/cpdf_page.h"
@@ skipping 276 matching lines @@
   for (auto it : m_ControlMap)
     delete it.second;
 
   int nCount = m_pFieldTree->m_Root.CountFields();
   for (int i = 0; i < nCount; ++i) {
     delete m_pFieldTree->m_Root.GetField(i);
   }
 }
 
 FX_BOOL CPDF_InterForm::s_bUpdateAP = TRUE;
-FX_BOOL CPDF_InterForm::UpdatingAPEnabled() {
+FX_BOOL CPDF_InterForm::UpdateAPEnabled() {

[Lei Zhang, 2016/07/13 01:48:02]

This one is hard to parse. I read it as "Update APEnabled" the first time
around, instead of "UpdatedAP Enabled".

Does "IsUpdateAPEnabled" seem clear?

[Lei Zhang, 2016/07/13 01:48:02]

Can you add some line breaks while you are here?

[jaepark, 2016/07/13 02:02:57]

Done.

[jaepark, 2016/07/13 02:02:57]

Done.

   return s_bUpdateAP;
 }
-void CPDF_InterForm::EnableUpdateAP(FX_BOOL bUpdateAP) {
+void CPDF_InterForm::SetUpdateAP(FX_BOOL bUpdateAP) {
   s_bUpdateAP = bUpdateAP;
 }
 CFX_ByteString CPDF_InterForm::GenerateNewResourceName(
     const CPDF_Dictionary* pResDict,
     const FX_CHAR* csType,
     int iMinLen,
     const FX_CHAR* csPrefix) {
   CFX_ByteString csStr = csPrefix;
   CFX_ByteString csBType = csType;
   if (csStr.IsEmpty()) {
@@ skipping 878 matching lines @@
     FDF_ImportField(pField, L"", bNotify);
   }
   if (bNotify && m_pFormNotify)
     m_pFormNotify->AfterFormImportData(this);
   return TRUE;
 }
 
 void CPDF_InterForm::SetFormNotify(IPDF_FormNotify* pNotify) {
   m_pFormNotify = pNotify;
 }