Fix leaks in the embedder tests themselves.

Fix leaks in the embedder tests themselves.

Also change EmbedderTest::TearDown() to match the destruction order in
Chromium's PDF code.

R=tsepez@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/61ffad8df484ab9b3f7d2f5519ec470fbc023b88

[Lei Zhang, 2015-05-15 06:48:56]

thestig@chromium.org changed reviewers:
+ tsepez@chromium.org

[Lei Zhang, 2015-05-15 06:48:57]

https://codereview.chromium.org/1138143003/diff/1/testing/embedder_test.cpp
File testing/embedder_test.cpp (right):

https://codereview.chromium.org/1138143003/diff/1/testing/embedder_test.cpp#n...
testing/embedder_test.cpp:215: FORM_DoDocumentAAction(form_handle_,
FPDFDOC_AACTION_WC);
See PDFiumEngine::~PDFiumEngine().

[Tom Sepez, 2015-05-15 15:12:40]

https://codereview.chromium.org/1138143003/diff/1/testing/embedder_test.cpp
File testing/embedder_test.cpp (right):

https://codereview.chromium.org/1138143003/diff/1/testing/embedder_test.cpp#n...
testing/embedder_test.cpp:215: FORM_DoDocumentAAction(form_handle_,
FPDFDOC_AACTION_WC);
On 2015/05/15 06:48:57, Lei Zhang wrote:
> See PDFiumEngine::~PDFiumEngine().

See comment I left there -- why doesn't document_ need to outlive FF env?

[Lei Zhang, 2015-05-15 22:10:21]

I added one more file in patch set 2, so all of our code does teardown in the
same order.

[Tom Sepez, 2015-05-15 22:11:10]

lgtm

[Lei Zhang, 2015-05-15 22:13:31]

Committed patchset #2 (id:20001) manually as
61ffad8df484ab9b3f7d2f5519ec470fbc023b88 (presubmit successful).