Implement FPDFAction_GetFilePath().

fpdfsdk/src/fpdfdoc_embeddertest.cpp

 // Copyright 2015 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.
 
 #include "../../public/fpdf_doc.h"
 #include "../../public/fpdfview.h"
 #include "../../testing/embedder_test.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 class FPDFDocEmbeddertest : public EmbedderTest {};
@@ skipping 17 matching lines @@
   // Page number directly in item from Dests dictionary.
   dest = FPDF_GetNamedDestByName(document(), "FirstAlternate");
   EXPECT_NE(nullptr, dest);
   EXPECT_EQ(11U, FPDFDest_GetPageIndex(document(), dest));
 
   // Invalid object reference in item from Dests NameTree.
   dest = FPDF_GetNamedDestByName(document(), "LastAlternate");
   EXPECT_NE(nullptr, dest);
   EXPECT_EQ(0U, FPDFDest_GetPageIndex(document(), dest));
 }
+
+TEST_F(FPDFDocEmbeddertest, ActionGetFilePath) {
+  EXPECT_TRUE(OpenDocument("testing/resources/launch_action.pdf"));
+
+  FPDF_PAGE page = FPDF_LoadPage(document(), 0);
+  ASSERT_TRUE(page);

[jun_fang, 2015/09/14 14:18:58]

EXPECT_NE(nullptr, page);

[Lei Zhang, 2015/09/15 08:24:50]

I want this test to be straight forward and only continue if all the expected
setup steps before FPDFAction_GetFilePath() works correctly. If anything fails,
and we continue, the rest of the test is going to be, as expected, a miserable
failure as well.

[jun_fang, 2015/09/15 08:35:03]

The returned value, page, is handle not boolean value. Do you really want to
compare a handle to boolean value?

[Lei Zhang, 2015/09/15 08:50:57]

So handles are really pointers, and the range of values for pointers are:
[nullptr, anything else that's not nullptr], which when passed to
ASSERT_TRUE(ptr), evals to [false, true]. This is fine since we don't care what
the value of the pointer is, as long as it's not nullptr.

This is the same as writing:

if (ptr) {  // pointer evaluated as a boolean on this line.
  ptr->DoSomething();
}

ASSERT_NE(nullptr, link) does the same thing, but it's just more verbose. It's
the equivalent to: if (link != nullptr) { ... }

+
+  // The target action is nearly the size of the whole page.
+  FPDF_LINK link = FPDFLink_GetLinkAtPoint(page, 100, 100);
+  ASSERT_TRUE(link);

[jun_fang, 2015/09/14 14:18:58]

EXPECT_NE(nullptr, link);

+
+  FPDF_ACTION action = FPDFLink_GetAction(link);
+  ASSERT_TRUE(action);

[jun_fang, 2015/09/14 14:18:58]

EXPECT_NE(nullptr, action);

+
+  const char kExpectedResult[] = "test.pdf";
+  const unsigned long kExpectedLength = strlen(kExpectedResult) + 1;
+  unsigned long bufsize = FPDFAction_GetFilePath(action, nullptr, 0);
+  ASSERT_EQ(kExpectedLength, bufsize);
+
+  char buf[kExpectedLength];
+  EXPECT_EQ(bufsize, FPDFAction_GetFilePath(action, buf, bufsize));
+  EXPECT_EQ(std::string(kExpectedResult), std::string(buf));
+
+  FPDF_ClosePage(page);
+}