Roll DEPS for gtest library.

Roll DEPS for gtest library.

From r643 to r677.

r644: Supports colored output on term type screen-256color.
r645: Prints a useful message when GetParam() is called in a non-parameterized test.
r646: Replaces unportable == with portable = in configure.ac.
r647: Fixes some compatibility issues with STLport.
r648: Implements support for calling Test::RecordProperty() outside of a test.
r649: Updates the list of reserved properties for RecordProperty.
r651: Updates the version number to 1.7.0
r652: prints type/value parameters when listing tests
r653: Fixes a thread annotation; updates CHANGES for 1.7.0
r654: When --gtest_filter is specified, XML report now doesn't contain information
r655: Fixes compatibility with C++11: (1 - 1) is no longer a NULL pointer constant.
r657: makes gtest-death-test_test.cc compile on platforms that don't support death
tests;
r658: allows gtest-config.in to work with an absoulte path for @top_srcdir@.
r659: fixes a typo in CMake script; h/t to Jay Mueller for reporting the issue
r661: Updates gtest.xcodeproj to be compatible with OS X 10.8.4 & Xcode 4.6.3
r662: supports a protocol for catching tests that prematurely exit
r663: avoids clash with the max() macro on Windows
r664: Cuts release 1.7.0
r668: Delete whitespace, and change the return type of
ImplicitlyConvertible::MakeFrom() to From&.
r669: Add MemorySanitizer annotations in gtest printers. Also remove unused variable
kPathSeparatorString.
r670: Fix warnings encountered with clang -Wall.
r671: Distinguish between C++11 language and library support for <initializer_list>.
r672: Improves documentation on gtest's macros.
r673: Define specialization of PrintTo(...) for ::std::tuple.
r674: Fix a couple of typos in Google Test's README.
r675: Make Google Test build cleanly on Visual Studio 2010, 2012, 2013.
r676: Add GTEST_MOVE macro, to support mocking methods with move-only return types.
r677: Suppress "Conditional expression is constant" warning on Visual Studio.

BUG=None
TEST=None
R=thakis@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=248285

[tfarina, 2014-01-30 02:36:52]

If trybot passes, is this OK Nico?

Thanks!

[Nico, 2014-01-30 03:01:18]

Do you have a list of changes?

But lgtm if it builds.


On Wed, Jan 29, 2014 at 6:36 PM, <tfarina@chromium.org> wrote:

> Reviewers: Nico,
>
> Message:
> If trybot passes, is this OK Nico?
>
> Thanks!
>
> Description:
> Roll DEPS for gtest library.
>
> From r643 to r677.
>
> BUG=None
> TEST=None
> R=thakis@chromium.org
>
> Please review this at https://codereview.chromium.org/149243010/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src
>
> Affected files (+1, -1 lines):
>   M DEPS
>
>
> Index: DEPS
> diff --git a/DEPS b/DEPS
> index 7612899f8641ea84986f4396595d694f90e054cc..
> f54b8760231129e3150634ea27ca73b8067cfeed 100644
> --- a/DEPS
> +++ b/DEPS
> @@ -74,7 +74,7 @@ deps = {
>      (Var("googlecode_url") % "open-vcdiff") + "/trunk@42",
>
>    "src/testing/gtest":
> -    (Var("googlecode_url") % "googletest") + "/trunk@643",
> +    (Var("googlecode_url") % "googletest") + "/trunk@677",
>
>    "src/testing/gmock":
>      (Var("googlecode_url") % "googlemock") + "/trunk@410",
>
>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[tfarina, 2014-01-30 03:07:30]

On 2014/01/30 03:01:18, Nico wrote:
> Do you have a list of changes?
>
Out of the 34 change sets, I could probably exclude those which are just wiki
edits and paste the remaining in the CL description. Is that worth my time on
that?

> But lgtm if it builds.
>

[Nico, 2014-01-30 03:16:30]

I tend to do it when rolling stuff, I think it's useful.
On Jan 29, 2014 7:07 PM, <tfarina@chromium.org> wrote:

> On 2014/01/30 03:01:18, Nico wrote:
>
>> Do you have a list of changes?
>>
>
> Out of the 34 change sets, I could probably exclude those which are just
> wiki
> edits and paste the remaining in the CL description. Is that worth my time
> on
> that?
>
>  But lgtm if it builds.
>>
>
>
>
> https://codereview.chromium.org/149243010/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[tfarina, 2014-01-30 03:31:02]

On 2014/01/30 03:16:30, Nico wrote:
> I tend to do it when rolling stuff, I think it's useful.
> On Jan 29, 2014 7:07 PM, <mailto:tfarina@chromium.org> wrote:
> 
> > On 2014/01/30 03:01:18, Nico wrote:
> >
> >> Do you have a list of changes?
> >>
> >
> > Out of the 34 change sets, I could probably exclude those which are just
> > wiki
> > edits and paste the remaining in the CL description. Is that worth my time
> > on
> > that?
> >
> >  But lgtm if it builds.
> >>
> >
> >
> >
> > https://codereview.chromium.org/149243010/
> >
> 
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:chromium-reviews+unsubscribe@chromium.org.

[tfarina, 2014-01-30 03:31:49]

On 2014/01/30 03:16:30, Nico wrote:
> I tend to do it when rolling stuff, I think it's useful.
Done.

err, I need to get a new mouse (this one is doing double clicks randomly, sorry
for the email spam).

[tfarina, 2014-01-31 13:07:06]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2014-01-31 13:07:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tfarina@chromium.org/149243010/1

[commit-bot: I haz the power, 2014-01-31 20:31:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-01-31 20:31:18]

The commit queue went berserk retrying too often for a
seemingly flaky test on builder android_dbg:
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...

[commit-bot: I haz the power, 2014-01-31 20:31:45]

CQ bit was unchecked on CL. Ignoring.

[commit-bot: I haz the power, 2014-01-31 20:31:45]

CQ bit was unchecked on CL. Ignoring.

[tfarina, 2014-01-31 22:07:41]

Committed patchset #1 manually as r248285 (presubmit successful).