Merge to XFA: Make conversion between CPDF_Action and its dictionary explicit.

fpdfsdk/src/fpdfdoc.cpp

Index: fpdfsdk/src/fpdfdoc.cpp
diff --git a/fpdfsdk/src/fpdfdoc.cpp b/fpdfsdk/src/fpdfdoc.cpp
index 19b45818294f20d2373722f0065b9613c870dc47..9e4267ffc7990e8c0884702032ca7feac5e73ce9 100644
--- a/fpdfsdk/src/fpdfdoc.cpp
+++ b/fpdfsdk/src/fpdfdoc.cpp
@@ -99,14 +99,14 @@ DLLEXPORT FPDF_ACTION STDCALL FPDFBookmark_GetAction(FPDF_BOOKMARK pDict)
 	if (!pDict)
 		return NULL;
 	CPDF_Bookmark bookmark((CPDF_Dictionary*)pDict);
-	return bookmark.GetAction();
+	return bookmark.GetAction().GetDict();
 }
 
 DLLEXPORT unsigned long STDCALL FPDFAction_GetType(FPDF_ACTION pDict)
 {
 	if (!pDict)
 		return 0;
-	CPDF_Action action = (CPDF_Dictionary*)pDict;
+	CPDF_Action action((CPDF_Dictionary*)pDict);
 	CPDF_Action::ActionType type = action.GetType();
 	switch (type) {
 		case CPDF_Action::GoTo:
@@ -130,7 +130,7 @@ DLLEXPORT FPDF_DEST STDCALL FPDFAction_GetDest(FPDF_DOCUMENT document, FPDF_ACTI
 	if (!pDict)
 		return NULL;
 	CPDF_Document* pDoc = ((CPDFXFA_Document*)document)->GetPDFDoc();
-	CPDF_Action action = (CPDF_Dictionary*)pDict;
+	CPDF_Action action((CPDF_Dictionary*)pDict);
 	return action.GetDest(pDoc);
 }
 
@@ -142,7 +142,7 @@ DLLEXPORT unsigned long STDCALL FPDFAction_GetURIPath(FPDF_DOCUMENT document, FP
 	if (!pDict)
 		return 0;
 	CPDF_Document* pDoc = ((CPDFXFA_Document*)document)->GetPDFDoc();
-	CPDF_Action action = (CPDF_Dictionary*)pDict;
+	CPDF_Action action((CPDF_Dictionary*)pDict);
 	CFX_ByteString path = action.GetURI(pDoc);
 	unsigned long len = path.GetLength() + 1;
 	if (buffer != NULL && buflen >= len)
@@ -207,7 +207,7 @@ DLLEXPORT FPDF_ACTION STDCALL FPDFLink_GetAction(FPDF_LINK pDict)
 	if (!pDict)
 		return NULL;
 	CPDF_Link link = (CPDF_Dictionary*)pDict;
-	return link.GetAction();
+	return link.GetAction().GetDict();
 }
 
 DLLEXPORT FPDF_BOOL STDCALL FPDFLink_Enumerate(FPDF_PAGE page, int* startPos, FPDF_LINK* linkAnnot)