Convert CPDFSDK_InterForm to take a CPDFSDK_FormFillEnvironment

fpdfsdk/javascript/Field.cpp

Index: fpdfsdk/javascript/Field.cpp
diff --git a/fpdfsdk/javascript/Field.cpp b/fpdfsdk/javascript/Field.cpp
index fa2ac70aa515fee47b79a0acfd0b85b241c5f955..6d11560a17b1b21779dbd662dd0d23178657d5e0 100644
--- a/fpdfsdk/javascript/Field.cpp
+++ b/fpdfsdk/javascript/Field.cpp
@@ -296,8 +296,10 @@ void Field::UpdateFormField(CPDFSDK_Document* pDocument,
     pInterForm->GetWidgets(pFormField, &widgets);
 
     for (CPDFSDK_Widget* pWidget : widgets) {
-      CPDFSDK_Document* pDoc = pWidget->GetInterForm()->GetDocument();
-      pDoc->UpdateAllViews(nullptr, pWidget);
+      pWidget->GetInterForm()
+          ->GetFormFillEnv()
+          ->GetSDKDocument()

[Tom Sepez, 2016/10/06 18:42:06]

is the document invariant across the all widgets?  Is there another way to get
it without all this work in every loop iteration?

[dsinclair, 2016/10/06 19:21:23]

a document has a single InterForm, but I haven't determined if all widgets have
to be in the same document. There is other code which makes it seem that we can
have different documents in the CJS_Runtime and the CJS_Context. Which also
seems wonky to me.

I'm putting a TODO here to figure this out in the future.

+          ->UpdateAllViews(nullptr, pWidget);
     }
   }
 
@@ -330,8 +332,8 @@ void Field::UpdateFormControl(CPDFSDK_Document* pDocument,
 
     if (bRefresh) {
       CPDFSDK_InterForm* pInterForm = pWidget->GetInterForm();
-      CPDFSDK_Document* pDoc = pInterForm->GetDocument();
-      pDoc->UpdateAllViews(nullptr, pWidget);
+      pInterForm->GetFormFillEnv()->GetSDKDocument()->UpdateAllViews(nullptr,
+                                                                     pWidget);
     }
   }