Migrate TRACED_FOREACH to C++11.

Migrate TRACED_FOREACH to C++11.

R=svenpanne@chromium.org

Committed: https://crrev.com/a0859c26bf569611ffddbd90eb431fe683667687
Cr-Commit-Position: refs/heads/master@{#26096}

[Benedikt Meurer, 2015-01-16 08:22:38]

PTAL

[Sven Panne, 2015-01-16 08:30:38]

https://codereview.chromium.org/851243002/diff/1/testing/gtest-support.h
File testing/gtest-support.h (right):

https://codereview.chromium.org/851243002/diff/1/testing/gtest-support.h#newc...
testing/gtest-support.h:38: for (_type const _var : _container)                 
                  \
I know that this is the literal translation (more exact: generalization) of the
previous code, but 2 comments:

   * Why should we still pass the type? We could use "auto".

   * I think that the type is always primitive, but nevertheless the recommended
and future-proof way would be using a const ref.

In a nutshell: Why not simply "for (const auto& _var: container)"?

[Sven Panne, 2015-01-16 09:55:03]

LGTM. As discussed offline: Let's first land this as it is, and perhaps remove
the type and use auto in a separate CL.

[Benedikt Meurer, 2015-01-16 09:55:13]

The CQ bit was checked by bmeurer@chromium.org

[commit-bot: I haz the power, 2015-01-16 09:56:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/851243002/1

[commit-bot: I haz the power, 2015-01-16 10:40:31]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-01-16 10:40:40]

Patchset 1 (id:??) landed as
https://crrev.com/a0859c26bf569611ffddbd90eb431fe683667687
Cr-Commit-Position: refs/heads/master@{#26096}