commit 2468fcd959022fb118323c6016b36d195c2a0843
author Lei Zhang <thestig@chromium.org> Fri Jan 31 18:19:59 2020 +0000
committer Chromium commit bot <commit-bot@chromium.org> Fri Jan 31 18:19:59 2020 +0000
tree 520930e1a2b2fff0f376a676943ccdb15bc61924
parent bc2b59e3f6fe000d1e9045c6383ca080d1b3b7e1

Reduce the number of calls to HasPermissions().

Call CPDFSDK_FormFillEnvironment::HasPermissions(flag1 | flag2) instead
of HasPermissions(flag1) || HasPermissions(flag2).

Change-Id: I6649e794c887cda71ba05f0bb1c2e070c50345a8
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/65840
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>

fxjs/cjs_field.cpp

diff --git a/fxjs/cjs_field.cpp b/fxjs/cjs_field.cpp
index 538e9aa..3e07a3a 100644
--- a/fxjs/cjs_field.cpp
+++ b/fxjs/cjs_field.cpp
@@ -609,15 +609,12 @@
 
 bool CJS_Field::AttachField(CJS_Document* pDocument,
                             const WideString& csFieldName) {
-  using pdfium::access_permissions::kFillForm;
-  using pdfium::access_permissions::kModifyAnnotation;
-  using pdfium::access_permissions::kModifyContent;
-
   m_pJSDoc.Reset(pDocument);
   m_pFormFillEnv.Reset(pDocument->GetFormFillEnv());
-  m_bCanSet = m_pFormFillEnv->HasPermissions(kFillForm) ||
-              m_pFormFillEnv->HasPermissions(kModifyAnnotation) ||
-              m_pFormFillEnv->HasPermissions(kModifyContent);
+  m_bCanSet = m_pFormFillEnv->HasPermissions(
+      pdfium::access_permissions::kFillForm |
+      pdfium::access_permissions::kModifyAnnotation |
+      pdfium::access_permissions::kModifyContent);
 
   CPDFSDK_InteractiveForm* pRDForm = m_pFormFillEnv->GetInteractiveForm();
   CPDF_InteractiveForm* pForm = pRDForm->GetInteractiveForm();