commit 624b166d4f50874f93c6d0824e91cd5b955c0487
author Tom Sepez <tsepez@chromium.org> Thu Jan 15 18:14:43 2015 -0800
committer Tom Sepez <tsepez@chromium.org> Thu Jan 15 18:14:43 2015 -0800
tree 7b0384c8d70d8915b465f7144d87aeb45a996d9a
parent d0edcea1a1b22813f593022ca29fca79074757bf

Allow FPDFBookmark_GetFirstChild to return the top level bookmark

Previously when passing a NULL bookmark to FPDFBookmark_GetFirstChild it returned NULL instead of returning the top level bookmark. This change removes the early exit in this case allowing the top level bookmark to be retrieved.

BUG=https://code.google.com/p/pdfium/issues/detail?id=110
R=bo_xu@foxitsoftware.com, tsepez@chromium.org

Review URL: https://codereview.chromium.org/847243005

fpdfsdk/src/fpdfdoc.cpp

diff --git a/fpdfsdk/src/fpdfdoc.cpp b/fpdfsdk/src/fpdfdoc.cpp
index feb7cfb..41f5c8a 100644
--- a/fpdfsdk/src/fpdfdoc.cpp
+++ b/fpdfsdk/src/fpdfdoc.cpp
@@ -29,7 +29,7 @@
 
 DLLEXPORT FPDF_BOOKMARK STDCALL FPDFBookmark_GetFirstChild(FPDF_DOCUMENT document, FPDF_BOOKMARK pDict)
 {
-    if (!document || !pDict)
+    if (!document)
         return NULL;
     CPDF_Document* pDoc = (CPDF_Document*)document;
     CPDF_BookmarkTree tree(pDoc);