This patch removes the need to add MAYBE lines for nearly every test that goes into find_request_ma…

Small optimization in disabling tests on Android Release in find_request_manager_browsertest.cc.

This patch removes the need to add MAYBE lines for nearly every test that goes into find_request_manager_browsertest.cc. It does this by defining a MAYBE() function instead of MAYBE/DISABLED definitions for every test name.

BUG=615291
Committed: https://crrev.com/cc9a00f96ebe2ec23c1626e5e42de5313e968ef7
Cr-Commit-Position: refs/heads/master@{#403898}

[paulmeyer, 2016-07-04 19:28:38]

paulmeyer@chromium.org changed reviewers:
+ jochen@chromium.org

[jochen (gone - plz use gerrit), 2016-07-05 14:18:51]

can you please reformat the CL description to match
https://www.chromium.org/developers/contributing-code#TOC-Writing-change-list...
and reference the bug 615291?

with that, lgtm

[paulmeyer, 2016-07-06 13:12:32]

Description was changed from

==========
This patch removes the need to add MAYBE lines for nearly every test that goes
into find_request_manager_browsertest.cc.
==========

to

==========
Small optimization in disabling tests on Android Release in
find_request_manager_browsertest.cc.

This patch removes the need to add MAYBE lines for nearly every test that goes
into find_request_manager_browsertest.cc. It does this by defining a MAYBE()
function instead of multiple definitions for every test name.

BUG=615291
==========

[paulmeyer, 2016-07-06 13:12:58]

Description was changed from

==========
Small optimization in disabling tests on Android Release in
find_request_manager_browsertest.cc.

This patch removes the need to add MAYBE lines for nearly every test that goes
into find_request_manager_browsertest.cc. It does this by defining a MAYBE()
function instead of multiple definitions for every test name.

BUG=615291
==========

to

==========
Small optimization in disabling tests on Android Release in
find_request_manager_browsertest.cc.

This patch removes the need to add MAYBE lines for nearly every test that goes
into find_request_manager_browsertest.cc. It does this by defining a MAYBE()
function instead of multiple MAYBE/DISABLED definitions for every test name.

BUG=615291
==========

[paulmeyer, 2016-07-06 13:13:09]

Description was changed from

==========
Small optimization in disabling tests on Android Release in
find_request_manager_browsertest.cc.

This patch removes the need to add MAYBE lines for nearly every test that goes
into find_request_manager_browsertest.cc. It does this by defining a MAYBE()
function instead of multiple MAYBE/DISABLED definitions for every test name.

BUG=615291
==========

to

==========
Small optimization in disabling tests on Android Release in
find_request_manager_browsertest.cc.

This patch removes the need to add MAYBE lines for nearly every test that goes
into find_request_manager_browsertest.cc. It does this by defining a MAYBE()
function instead of MAYBE/DISABLED definitions for every test name.

BUG=615291
==========

[paulmeyer, 2016-07-06 13:13:14]

The CQ bit was checked by paulmeyer@chromium.org

[commit-bot: I haz the power, 2016-07-06 13:13:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-06 13:44:39]

Description was changed from

==========
Small optimization in disabling tests on Android Release in
find_request_manager_browsertest.cc.

This patch removes the need to add MAYBE lines for nearly every test that goes
into find_request_manager_browsertest.cc. It does this by defining a MAYBE()
function instead of MAYBE/DISABLED definitions for every test name.

BUG=615291
==========

to

==========
Small optimization in disabling tests on Android Release in
find_request_manager_browsertest.cc.

This patch removes the need to add MAYBE lines for nearly every test that goes
into find_request_manager_browsertest.cc. It does this by defining a MAYBE()
function instead of MAYBE/DISABLED definitions for every test name.

BUG=615291
==========

[commit-bot: I haz the power, 2016-07-06 13:44:40]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-06 13:46:17]

Description was changed from

==========
Small optimization in disabling tests on Android Release in
find_request_manager_browsertest.cc.

This patch removes the need to add MAYBE lines for nearly every test that goes
into find_request_manager_browsertest.cc. It does this by defining a MAYBE()
function instead of MAYBE/DISABLED definitions for every test name.

BUG=615291
==========

to

==========
Small optimization in disabling tests on Android Release in
find_request_manager_browsertest.cc.

This patch removes the need to add MAYBE lines for nearly every test that goes
into find_request_manager_browsertest.cc. It does this by defining a MAYBE()
function instead of MAYBE/DISABLED definitions for every test name.

BUG=615291

Committed: https://crrev.com/cc9a00f96ebe2ec23c1626e5e42de5313e968ef7
Cr-Commit-Position: refs/heads/master@{#403898}
==========

[commit-bot: I haz the power, 2016-07-06 13:46:18]

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