arc: Use ARC_GET_INSTANCE_FOR_METHOD for getting app instance

arc: Use ARC_GET_INSTANCE_FOR_METHOD for getting app instance

This is safer than manually specifying min versions.

BUG=649782
TEST=git try, ARC still works

Review-Url: https://codereview.chromium.org/2613383002
Cr-Commit-Position: refs/heads/master@{#442295}
Committed: https://chromium.googlesource.com/chromium/src/+/d0c37cba3d84dc551c4d4b98a3dcf23321ed4b17

[Yusuke Sato, 2017-01-07 01:26:30]

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

[commit-bot: I haz the power, 2017-01-07 01:26:56]

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

[Yusuke Sato, 2017-01-07 01:31:10]

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

[Yusuke Sato, 2017-01-07 01:31:10]

PTAL too.

Once this is landed, I think we can rename GetInstanceForVersion() to something
like GetInstanceForVersionForInternalUseOnly() or
GetInstanceForVersionDoNotCallDirectly().

[Yusuke Sato, 2017-01-07 01:33:13]

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

[commit-bot: I haz the power, 2017-01-07 01:33:39]

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

[Yusuke Sato, 2017-01-07 02:00:34]

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

[Yusuke Sato, 2017-01-07 02:00:35]

+xiyuan for owner review.

Please have a look. We're trying to stop specifying mojo interface versions
manually. This CL replaces them with ARC_GET_INSTANCE_FOR_METHOD() calls that
automatically infer versions. Thanks.

[commit-bot: I haz the power, 2017-01-07 02:01:55]

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

[commit-bot: I haz the power, 2017-01-07 02:01:56]

Dry run: This issue passed the CQ dry run.

[xiyuan, 2017-01-09 17:18:59]

lgtm

[Luis Héctor Chávez, 2017-01-09 17:23:28]

On 2017/01/07 01:31:10, Yusuke Sato wrote:
> PTAL too.
> 
> Once this is landed, I think we can rename GetInstanceForVersion() to
something
> like GetInstanceForVersionForInternalUseOnly() or
> GetInstanceForVersionDoNotCallDirectly().

I like these scary names.

lgtm

[Yusuke Sato, 2017-01-09 17:42:45]

The CQ bit was checked by yusukes@chromium.org

[commit-bot: I haz the power, 2017-01-09 17:43:07]

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-01-09 18:20:57]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483983765855240,
"parent_rev": "abbb2541620aa14479081acce93c94c52523b6d5", "commit_rev":
"d0c37cba3d84dc551c4d4b98a3dcf23321ed4b17"}

[commit-bot: I haz the power, 2017-01-09 18:21:23]

Description was changed from

==========
arc: Use ARC_GET_INSTANCE_FOR_METHOD for getting app instance

This is safer than manually specifying min versions.

BUG=649782
TEST=git try, ARC still works
==========

to

==========
arc: Use ARC_GET_INSTANCE_FOR_METHOD for getting app instance

This is safer than manually specifying min versions.

BUG=649782
TEST=git try, ARC still works

Review-Url: https://codereview.chromium.org/2613383002
Cr-Commit-Position: refs/heads/master@{#442295}
Committed:
https://chromium.googlesource.com/chromium/src/+/d0c37cba3d84dc551c4d4b98a3dc...
==========

[commit-bot: I haz the power, 2017-01-09 18:21:24]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/d0c37cba3d84dc551c4d4b98a3dc...