Pull in gmock for standalone pdfium builds.

Pull in gmock for standalone pdfium builds.

For chromium checkouts, the top-level gmock is used instead.
Verify build with a simple test that ensures neither mock
method is fired.

R=thestig@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/3d25502adc5ae37d1c3c74d367f16f770c465109

[Tom Sepez, 2015-02-26 01:04:37]

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

[Tom Sepez, 2015-02-26 01:04:38]

Lei, for your consideration.  Thanks.

[Lei Zhang, 2015-02-26 01:13:42]

https://codereview.chromium.org/955513009/diff/20001/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/955513009/diff/20001/BUILD.gn#newcode834
BUILD.gn:834: "//testing/gmock",
nit: gmock before gtest, ditto in pdfium.gyp

https://codereview.chromium.org/955513009/diff/20001/testing/embedder_test.h
File testing/embedder_test.h (right):

https://codereview.chromium.org/955513009/diff/20001/testing/embedder_test.h#...
testing/embedder_test.h:16: #include "testing/gmock/include/gmock/gmock.h"
Is this needed in the header?

https://codereview.chromium.org/955513009/diff/20001/testing/gmock.gyp
File testing/gmock.gyp (right):

https://codereview.chromium.org/955513009/diff/20001/testing/gmock.gyp#newcode1
testing/gmock.gyp:1: # Copyright (c) 2009 The Chromium Authors. All rights
reserved.
nit: fix the copyright

https://codereview.chromium.org/955513009/diff/20001/testing/gmock.gyp#newcode36
testing/gmock.gyp:36: 'gmock/src/gmock-all.cc',  # Not needed by our build.
Just comment out gmock-all.cc on line 27?

https://codereview.chromium.org/955513009/diff/20001/testing/gmock.gyp#newcode51
testing/gmock.gyp:51: ['OS == "ios"', {
Do we need this

[Tom Sepez, 2015-02-27 18:39:21]

Also note that the placeholder fpdfformfill_embeddertest.cpp file, referred to
in the build files, is now present in the patch.

https://codereview.chromium.org/955513009/diff/20001/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/955513009/diff/20001/BUILD.gn#newcode834
BUILD.gn:834: "//testing/gmock",
On 2015/02/26 01:13:41, Lei Zhang wrote:
> nit: gmock before gtest, ditto in pdfium.gyp

Done.

https://codereview.chromium.org/955513009/diff/20001/testing/embedder_test.h
File testing/embedder_test.h (right):

https://codereview.chromium.org/955513009/diff/20001/testing/embedder_test.h#...
testing/embedder_test.h:16: #include "testing/gmock/include/gmock/gmock.h"
On 2015/02/26 01:13:42, Lei Zhang wrote:
> Is this needed in the header?
Moved to .cpp file.

https://codereview.chromium.org/955513009/diff/20001/testing/gmock.gyp
File testing/gmock.gyp (right):

https://codereview.chromium.org/955513009/diff/20001/testing/gmock.gyp#newcode36
testing/gmock.gyp:36: 'gmock/src/gmock-all.cc',  # Not needed by our build.
On 2015/02/26 01:13:42, Lei Zhang wrote:
> Just comment out gmock-all.cc on line 27?

Done.

https://codereview.chromium.org/955513009/diff/20001/testing/gmock.gyp#newcode51
testing/gmock.gyp:51: ['OS == "ios"', {
On 2015/02/26 01:13:42, Lei Zhang wrote:
> Do we need this

removed

[Lei Zhang, 2015-02-27 19:57:24]

lgtm

[Tom Sepez, 2015-02-27 20:56:30]

Committed patchset #4 (id:40002) manually as
3d25502adc5ae37d1c3c74d367f16f770c465109 (presubmit successful).