Pull in gmock through DEPS

Pull in gmock through DEPS

README.chromium hints that we didn't do this previously because of gmock using svn externals. But
maruel recently landed a patch that makes gclient --ignore-externals.

BUG=none
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=72866

[Nico, 2011-01-27 19:02:40]

Works fine locally. I have a thread open with the infrastructure folks about the
try results.

[Evan Martin, 2011-01-27 19:35:43]

LGTM

http://codereview.chromium.org/6241018/diff/66002/DEPS
File DEPS (right):

http://codereview.chromium.org/6241018/diff/66002/DEPS#newcode37
DEPS:37: "http://googlemock.googlecode.com/svn/trunk@300",
README.chromium says current revision 282.  Does it matter?

(All that really matters is that you use the one that your version of gtest
wants -- they frequently change and become incompatible.)

[Nico, 2011-01-27 19:42:21]

thanks!

http://codereview.chromium.org/6241018/diff/66002/DEPS
File DEPS (right):

http://codereview.chromium.org/6241018/diff/66002/DEPS#newcode37
DEPS:37: "http://googlemock.googlecode.com/svn/trunk@300",
On 2011/01/27 19:35:44, Evan Martin wrote:
> README.chromium says current revision 282.  Does it matter?
> 
> (All that really matters is that you use the one that your version of gtest
> wants -- they frequently change and become incompatible.)

Yes, it matters. 282 doesn't work. (It got updated in
http://codereview.chromium.org/3427004 to 300, but both reviewer and patch
author failed to notice that the readme needed to be updated ;-) )

[Evan Martin, 2011-01-27 19:48:58]

On Thu, Jan 27, 2011 at 11:42 AM,  <thakis@chromium.org> wrote:
> Yes, it matters. 282 doesn't work. (It got updated in
> http://codereview.chromium.org/3427004 to 300, but both reviewer and
> patch author failed to notice that the readme needed to be updated ;-) )

I think the reviewer is completely at fault there and the author is blameless!

LGTM++