Removing unneeded macro

Removing unneeded macro

BUG=NONE
TEST=NONE


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

[Lei Zhang, 2012-02-13 22:13:55]

http://codereview.chromium.org/9350051/diff/1/base/gtest_prod_util.h
File base/gtest_prod_util.h (right):

http://codereview.chromium.org/9350051/diff/1/base/gtest_prod_util.h#newcode44
base/gtest_prod_util.h:44: // Use FORWARD_DECLARE_TEST_ALL_PREFIXES to do so for
all possible prefixes.
Update comment too, otherwise LGTM.

[dpapad, 2012-02-13 22:16:12]

http://codereview.chromium.org/9350051/diff/1/base/gtest_prod_util.h
File base/gtest_prod_util.h (right):

http://codereview.chromium.org/9350051/diff/1/base/gtest_prod_util.h#newcode44
base/gtest_prod_util.h:44: // Use FORWARD_DECLARE_TEST_ALL_PREFIXES to do so for
all possible prefixes.
On 2012/02/13 22:13:55, Lei Zhang wrote:
> Update comment too, otherwise LGTM.

Done.

[dpapad, 2012-02-13 22:17:11]

@brettw: Owners LGTM needed.

[brettw, 2012-02-13 22:22:12]

lgtm

http://codereview.chromium.org/9350051/diff/4001/base/gtest_prod_util.h
File base/gtest_prod_util.h (right):

http://codereview.chromium.org/9350051/diff/4001/base/gtest_prod_util.h#newco...
base/gtest_prod_util.h:64: class test_case_name##_##test_name##_Test; \
Normally these would be indented 4 spaces.

[dpapad, 2012-02-13 22:30:04]

http://codereview.chromium.org/9350051/diff/4001/base/gtest_prod_util.h
File base/gtest_prod_util.h (right):

http://codereview.chromium.org/9350051/diff/4001/base/gtest_prod_util.h#newco...
base/gtest_prod_util.h:64: class test_case_name##_##test_name##_Test; \
On 2012/02/13 22:22:12, brettw wrote:
> Normally these would be indented 4 spaces.

In the same file at the top they are indented only 2 spaces. Doing the same here
for consistency.

[commit-bot: I haz the power, 2012-02-13 23:31:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dpapad@chromium.org/9350051/1005

[commit-bot: I haz the power, 2012-02-14 01:19:10]

Change committed as 121801

[Paweł Hajdan Jr., 2012-02-14 08:53:14]

LGTM, thank you.