Correct mistakes in test case for 507316
authorTom Sepez <tsepez@chromium.org>
Fri, 2 Oct 2015 16:05:53 +0000 (09:05 -0700)
committerTom Sepez <tsepez@chromium.org>
Fri, 2 Oct 2015 16:05:53 +0000 (09:05 -0700)
Original patch by chamalsl.

Trailer size in bug_507316 was wrong.

embedder_test.cpp's GetPageTrampoline passed null parameter.
It will affect future test cases even if it does not affect
this.

BUG=507316
R=tsepez@chromium.org

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

testing/embedder_test.cpp
testing/resources/bug_507316.in
testing/resources/bug_507316.pdf

index eb16bc7..951da84 100644 (file)
@@ -337,7 +337,7 @@ FPDF_PAGE EmbedderTest::GetPageTrampoline(FPDF_FORMFILLINFO* info,
                                           FPDF_DOCUMENT document,
                                           int page_index) {
   EmbedderTest* test = static_cast<EmbedderTest*>(info);
-  return test->delegate_->GetPage(test->m_pFormfillinfo, document, page_index);
+  return test->delegate_->GetPage(test->form_handle(), document, page_index);
 }
 
 // Can't use gtest-provided main since we need to stash the path to the
index cd3f81e..87dd5c3 100644 (file)
@@ -119,7 +119,7 @@ endstream
 endobj
 {{xref}}
 trailer <<
-  /Size 15
+  /Size 16
   /Root 1 0 R
 >>
 {{startxref}}
index 13c7083..ecc205c 100644 (file)
@@ -137,7 +137,7 @@ xref
 0000001365 00000 n 
 0000001433 00000 n 
 trailer <<
-  /Size 15
+  /Size 16
   /Root 1 0 R
 >>
 startxref