diff options
author | weili <weili@chromium.org> | 2016-08-23 21:10:57 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-08-23 21:10:57 -0700 |
commit | 0dadcc6fdab7ad1f2ee95d763f31aad5d3534f93 (patch) | |
tree | d62cd8cc5b33e8b3707bbdb6eb5bbf6269f52110 /testing/js_embedder_test.cpp | |
parent | e07edce5b253bc4f2bef6888c5b1cbf0b320a919 (diff) | |
download | pdfium-0dadcc6fdab7ad1f2ee95d763f31aad5d3534f93.tar.xz |
Fix page leaks in an embedder test
Embedder test's delegate function GetPage() calls FPDF_LoadPage() to
load a page which may be already loaded by embedder test itself.
Thus the page's ref count is increased unnecessarily.
This causes the page to be leaked. Fix this by putting the page map in
embedder test class and guarantee the page is loaded only once.
Also, fix leaks in this embedder tests by unloading the loaded pages to
properly release the resource.
BUG=pdfium:242
Review-Url: https://codereview.chromium.org/2258333002
Diffstat (limited to 'testing/js_embedder_test.cpp')
0 files changed, 0 insertions, 0 deletions