diff options
| author | thestig <thestig@chromium.org> | 2016-06-22 07:03:23 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-06-22 07:03:23 -0700 |
| commit | 29ce9238c6a05abb1de5e4175c97e4fcadd5640f (patch) | |
| tree | ddb81bfe0829948ca70d8b3f6b56bf68c6c9346b /core/fpdfapi/fpdf_parser | |
| parent | 907a5223ea5abd09878de20cc74c59ebd0d6c3c8 (diff) | |
| download | pdfium-29ce9238c6a05abb1de5e4175c97e4fcadd5640f.tar.xz | |
Fix memory leaks in EmbedderTest::OpenDocument().
EmbedderTest::OpenDocument() does not gracefully handle the case of
being called twice in a given test case. So avoid doing that.
Review-Url: https://codereview.chromium.org/2088093002
Diffstat (limited to 'core/fpdfapi/fpdf_parser')
| -rw-r--r-- | core/fpdfapi/fpdf_parser/cpdf_parser_embeddertest.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/core/fpdfapi/fpdf_parser/cpdf_parser_embeddertest.cpp b/core/fpdfapi/fpdf_parser/cpdf_parser_embeddertest.cpp index 66b29a91c4..f523d6b31b 100644 --- a/core/fpdfapi/fpdf_parser/cpdf_parser_embeddertest.cpp +++ b/core/fpdfapi/fpdf_parser/cpdf_parser_embeddertest.cpp @@ -35,8 +35,11 @@ TEST_F(CPDFParserEmbeddertest, Feature_Linearized_Loading) { EXPECT_TRUE(OpenDocument("feature_linearized_loading.pdf", nullptr, true)); } -TEST_F(CPDFParserEmbeddertest, Bug_325) { +TEST_F(CPDFParserEmbeddertest, Bug_325a) { EXPECT_FALSE(OpenDocument("bug_325_a.pdf")); +} + +TEST_F(CPDFParserEmbeddertest, Bug_325b) { EXPECT_FALSE(OpenDocument("bug_325_b.pdf")); } |