Move gtest param interface to the necessary class.

Move gtest param interface to the necessary class.

Parameterized test is used only for ArcSessionOobeOptInNegotiatorTest now.
So, move the inheritance WithParamInterface from base class
to the actually using class.
It should be much clear policy, and it also allows other
test classes inheriting the base class to use parameterized tests.

BUG=None
TEST=Ran bots.

Review-Url: https://codereview.chromium.org/2691213002
Cr-Commit-Position: refs/heads/master@{#450422}
Committed: https://chromium.googlesource.com/chromium/src/+/00e934cfb7e965d9f37193aaa1adc685e1ed7d82

[hidehiko, 2017-02-14 04:59:57]

Description was changed from

==========
Move gtest param interface to the necessary class.

Parameterized test is used only for ArcSessionOobeOptInNegotiatorTest.
Inherit WithParamInterface<> so that other tests can use
parameterized tests.

BUG=None
TEST=Ran bots.
==========

to

==========
Move gtest param interface to the necessary class.

Parameterized test is used only for ArcSessionOobeOptInNegotiatorTest now.
So, move the inheritance WithParamInterface from base class
to the actually using class.
It should be much clear policy, and it also allows other
test classes inheriting the base class to use parameterized tests.

BUG=None
TEST=Ran bots.
==========

[hidehiko, 2017-02-14 05:00:15]

The CQ bit was checked by hidehiko@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-14 05:00:47]

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

[hidehiko, 2017-02-14 05:00:55]

hidehiko@chromium.org changed reviewers:
+ yusukes@chromium.org

[hidehiko, 2017-02-14 05:00:56]

PTAL.

[commit-bot: I haz the power, 2017-02-14 05:18:38]

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

[commit-bot: I haz the power, 2017-02-14 05:18:38]

Dry run: This issue passed the CQ dry run.

[Yusuke Sato, 2017-02-14 18:09:06]

lgtm

[hidehiko, 2017-02-14 18:41:34]

The CQ bit was checked by hidehiko@chromium.org

[commit-bot: I haz the power, 2017-02-14 18:42:24]

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, 2017-02-14 18:48:49]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487097694918050, "parent_rev":
"9d2fa6e9d57a7743491cda0cd064a13a7a4cd37f", "commit_rev":
"00e934cfb7e965d9f37193aaa1adc685e1ed7d82"}

[commit-bot: I haz the power, 2017-02-14 18:49:28]

Description was changed from

==========
Move gtest param interface to the necessary class.

Parameterized test is used only for ArcSessionOobeOptInNegotiatorTest now.
So, move the inheritance WithParamInterface from base class
to the actually using class.
It should be much clear policy, and it also allows other
test classes inheriting the base class to use parameterized tests.

BUG=None
TEST=Ran bots.
==========

to

==========
Move gtest param interface to the necessary class.

Parameterized test is used only for ArcSessionOobeOptInNegotiatorTest now.
So, move the inheritance WithParamInterface from base class
to the actually using class.
It should be much clear policy, and it also allows other
test classes inheriting the base class to use parameterized tests.

BUG=None
TEST=Ran bots.

Review-Url: https://codereview.chromium.org/2691213002
Cr-Commit-Position: refs/heads/master@{#450422}
Committed:
https://chromium.googlesource.com/chromium/src/+/00e934cfb7e965d9f37193aaa1ad...
==========

[commit-bot: I haz the power, 2017-02-14 18:49:29]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/00e934cfb7e965d9f37193aaa1ad...