Consolidate test support code.

Consolidate test support code.

Removes duplication between pdfium_test and pdfium_embeddertest

R=thestig@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/6d34d056a7188b7ae9800fe3ceb68645e0442879

[Tom Sepez, 2015-10-19 18:37:27]

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

[Tom Sepez, 2015-10-19 18:37:27]

Lei, for review.
Dan, FYI for pdfium_shell use.

[dsinclair, 2015-10-19 18:49:48]

dsinclair@chromium.org changed reviewers:
+ dsinclair@chromium.org

[dsinclair, 2015-10-19 18:49:48]

https://codereview.chromium.org/1416713002/diff/20001/testing/test_support.h
File testing/test_support.h (right):

https://codereview.chromium.org/1416713002/diff/20001/testing/test_support.h#...
testing/test_support.h:15: 
What's the PDFium stance on namespaces? Should this all be inside something
like:

namespace testing {

}?

[Lei Zhang, 2015-10-19 18:51:26]

Can test_support be its own target?

[Tom Sepez, 2015-10-19 18:57:36]

https://codereview.chromium.org/1416713002/diff/20001/testing/test_support.h
File testing/test_support.h (right):

https://codereview.chromium.org/1416713002/diff/20001/testing/test_support.h#...
testing/test_support.h:15: 
On 2015/10/19 18:49:48, dsinclair wrote:
> What's the PDFium stance on namespaces? 
Fast and loose.  There's probably a task to namespace the whole product someday.

Should this all be inside something
> like:
> 
> namespace testing {
> 
> }?
Probably, but I think that falls into the larger task.

[Tom Sepez, 2015-10-19 19:17:09]

On 2015/10/19 18:51:26, Lei Zhang wrote:
> Can test_support be its own target?
Done.

[Lei Zhang, 2015-10-19 19:21:43]

lgtm

[Tom Sepez, 2015-10-19 21:03:35]

Committed patchset #7 (id:120001) manually as
6d34d056a7188b7ae9800fe3ceb68645e0442879 (presubmit successful).