RFR(L/XS) : 8222414 : bring googlemock v1.8.1

[Erik Joelsson]

Hello Igor,

Build changes look ok, with a (very) minor style suggestion [1] (point 
18). We try to avoid padding continuation line breaks, so I would 
appreciate if CompileGtest.gmk:68 could be indented just 4 extra spaces 
compared to line 67. If you want the file names to line up, it's 
acceptable to break after the := on 67.

[1] http://openjdk.java.net/groups/build/doc/code-conventions.html

/Erik

On 2019-05-22 22:20, Igor Ignatyev wrote:
> http://cr.openjdk.java.net/~iignatyev//8222414/webrev.01
>> 21638 lines changed: 21628 ins; 0 del; 10 mod;
> Hi all,
>
> could you please review this patch which makes mocking framework from google test available for hotspot tests?
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8222414
> testing: tier1
> webrevs:
>   - http://cr.openjdk.java.net/~iignatyev//8222414/webrev.00
>> 21605 lines changed: 21605 ins; 0 del; 0 mod;
> * moves gtest from test/fmw/gtest to test/fmw/gtest/googletest;
> * adds only required (w/o build-aux, docs, make, msvc, scripts, tests directories and make related files) source of gmock to test/fmw/gtest/googlemock;
>
> - http://cr.openjdk.java.net/~iignatyev//8222414/webrev.0-1
>> 33 lines changed: 23 ins; 0 del; 10 mod;
> * makes necessary changes in makefiles
> * calls gmock framework init function
> * works around the conflict b/w :testing::internal::Log function defined by gmock and Log macro defined by hotspot
>
> - http://cr.openjdk.java.net/~iignatyev//8222414/webrev.01
>> 21638 lines changed: 21628 ins; 0 del; 10 mod;
> all the changes together
>
>
> thanks,
> -- Igor