Add OBB mounter Mojo interface

Add OBB mounte Mojo interface

Add a new interface ObbMounter, and expose it to Android.
Actual features are not implemented yet.

BUG=613480
Committed: https://crrev.com/4265d7151753f4e3a7a4cbe688b423c376dd9503
Cr-Commit-Position: refs/heads/master@{#395557}

Committed: https://crrev.com/593d78aa7f3dd422ae42723c2dc039b7059704da
Cr-Commit-Position: refs/heads/master@{#395572}

[hashimoto, 2016-05-20 07:53:16]

Patchset #1 (id:1) has been deleted

[hashimoto, 2016-05-20 07:53:40]

Patchset #1 (id:20001) has been deleted

[hashimoto, 2016-05-20 07:58:17]

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

[hidehiko, 2016-05-20 14:15:46]

https://codereview.chromium.org/1995243002/diff/40001/components/arc/common/a...
File components/arc/common/arc_bridge.mojom (right):

https://codereview.chromium.org/1995243002/diff/40001/components/arc/common/a...
components/arc/common/arc_bridge.mojom:81: [MinVersion=12]
OnObbMounterInstanceReady@118(ObbMounterInstance instance_ptr);
ID looks conflicting. Could you rebase?

[hashimoto, 2016-05-23 04:46:47]

https://codereview.chromium.org/1995243002/diff/40001/components/arc/common/a...
File components/arc/common/arc_bridge.mojom (right):

https://codereview.chromium.org/1995243002/diff/40001/components/arc/common/a...
components/arc/common/arc_bridge.mojom:81: [MinVersion=12]
OnObbMounterInstanceReady@118(ObbMounterInstance instance_ptr);
On 2016/05/20 14:15:46, hidehiko wrote:
> ID looks conflicting. Could you rebase?

Done.

[hashimoto, 2016-05-23 08:56:31]

Description was changed from

==========
Add OBB mounter Mojo interface

Add a new interface ObbMounter, and expose it to Android.
Actual features are not implemented yet.

BUG=613480
==========

to

==========
Add volume manager Mojo interface

Add a new interface VolumeManager, and expose it to Android.
Actual features are not implemented yet.

BUG=613480
==========

[hashimoto, 2016-05-23 08:58:38]

Description was changed from

==========
Add volume manager Mojo interface

Add a new interface VolumeManager, and expose it to Android.
Actual features are not implemented yet.

BUG=613480
==========

to

==========
Add OBB mounte Mojo interface

Add a new interface ObbMounter, and expose it to Android.
Actual features are not implemented yet.

BUG=613480
==========

[hashimoto, 2016-05-23 08:58:44]

Patchset #3 (id:80001) has been deleted

[hidehiko, 2016-05-23 09:14:46]

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

[hidehiko, 2016-05-23 09:14:47]

+rickyz.

Ricky, could you review mojom files as an OWNER?

LGTM for components/arc.

https://codereview.chromium.org/1995243002/diff/60001/components/arc/obb_moun...
File components/arc/obb_mounter/arc_obb_mounter_bridge.h (right):

https://codereview.chromium.org/1995243002/diff/60001/components/arc/obb_moun...
components/arc/obb_mounter/arc_obb_mounter_bridge.h:15: class
ArcObbMounterBridge : public ArcService,
Could you add quick summary of what this class does?

[rickyz (no longer on Chrome), 2016-05-24 04:39:58]

On 2016/05/23 at 09:14:47, hidehiko wrote:
> +rickyz.
> 
> Ricky, could you review mojom files as an OWNER?
> 
> LGTM for components/arc.
> 
>
https://codereview.chromium.org/1995243002/diff/60001/components/arc/obb_moun...
> File components/arc/obb_mounter/arc_obb_mounter_bridge.h (right):
> 
>
https://codereview.chromium.org/1995243002/diff/60001/components/arc/obb_moun...
> components/arc/obb_mounter/arc_obb_mounter_bridge.h:15: class
ArcObbMounterBridge : public ArcService,
> Could you add quick summary of what this class does?

mojo lgtm

[hashimoto, 2016-05-24 06:32:13]

https://codereview.chromium.org/1995243002/diff/60001/components/arc/obb_moun...
File components/arc/obb_mounter/arc_obb_mounter_bridge.h (right):

https://codereview.chromium.org/1995243002/diff/60001/components/arc/obb_moun...
components/arc/obb_mounter/arc_obb_mounter_bridge.h:15: class
ArcObbMounterBridge : public ArcService,
On 2016/05/23 09:14:47, hidehiko wrote:
> Could you add quick summary of what this class does?

Done.

[hashimoto, 2016-05-24 09:37:02]

The CQ bit was checked by hashimoto@chromium.org

[hashimoto, 2016-05-24 09:37:02]

The patchset sent to the CQ was uploaded after l-g-t-m from
hidehiko@chromium.org, rickyz@chromium.org
Link to the patchset: https://codereview.chromium.org/1995243002/#ps120001
(title: "rebase")

[commit-bot: I haz the power, 2016-05-24 09:37:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1995243002/120001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1995243002/120001

[commit-bot: I haz the power, 2016-05-24 09:41:41]

Description was changed from

==========
Add OBB mounte Mojo interface

Add a new interface ObbMounter, and expose it to Android.
Actual features are not implemented yet.

BUG=613480
==========

to

==========
Add OBB mounte Mojo interface

Add a new interface ObbMounter, and expose it to Android.
Actual features are not implemented yet.

BUG=613480
==========

[commit-bot: I haz the power, 2016-05-24 09:41:42]

Committed patchset #4 (id:120001)

[commit-bot: I haz the power, 2016-05-24 09:44:35]

Description was changed from

==========
Add OBB mounte Mojo interface

Add a new interface ObbMounter, and expose it to Android.
Actual features are not implemented yet.

BUG=613480
==========

to

==========
Add OBB mounte Mojo interface

Add a new interface ObbMounter, and expose it to Android.
Actual features are not implemented yet.

BUG=613480

Committed: https://crrev.com/4265d7151753f4e3a7a4cbe688b423c376dd9503
Cr-Commit-Position: refs/heads/master@{#395557}
==========

[commit-bot: I haz the power, 2016-05-24 09:44:37]

Patchset 4 (id:??) landed as
https://crrev.com/4265d7151753f4e3a7a4cbe688b423c376dd9503
Cr-Commit-Position: refs/heads/master@{#395557}

[hashimoto, 2016-05-24 10:12:16]

A revert of this CL (patchset #4 id:120001) has been created in
https://codereview.chromium.org/2001013004/ by hashimoto@chromium.org.

The reason for reverting is: Conflicting with
https://codereview.chromium.org/1996933002.

[hashimoto, 2016-05-24 10:24:38]

Patchset #5 (id:140001) has been deleted

[hashimoto, 2016-05-24 10:35:12]

Patchset #5 (id:160001) has been deleted

[hashimoto, 2016-05-24 10:36:01]

Description was changed from

==========
Add OBB mounte Mojo interface

Add a new interface ObbMounter, and expose it to Android.
Actual features are not implemented yet.

BUG=613480

Committed: https://crrev.com/4265d7151753f4e3a7a4cbe688b423c376dd9503
Cr-Commit-Position: refs/heads/master@{#395557}
==========

to

==========
Add OBB mounte Mojo interface

Add a new interface ObbMounter, and expose it to Android.
Actual features are not implemented yet.

BUG=613480

Committed: https://crrev.com/4265d7151753f4e3a7a4cbe688b423c376dd9503
Cr-Commit-Position: refs/heads/master@{#395557}
==========

[hashimoto, 2016-05-24 10:36:07]

The CQ bit was checked by hashimoto@chromium.org

[hashimoto, 2016-05-24 10:36:07]

The patchset sent to the CQ was uploaded after l-g-t-m from
hidehiko@chromium.org, rickyz@chromium.org
Link to the patchset: https://codereview.chromium.org/1995243002/#ps180001
(title: "rebase")

[commit-bot: I haz the power, 2016-05-24 10:36:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1995243002/180001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1995243002/180001

[commit-bot: I haz the power, 2016-05-24 11:44:24]

Description was changed from

==========
Add OBB mounte Mojo interface

Add a new interface ObbMounter, and expose it to Android.
Actual features are not implemented yet.

BUG=613480

Committed: https://crrev.com/4265d7151753f4e3a7a4cbe688b423c376dd9503
Cr-Commit-Position: refs/heads/master@{#395557}
==========

to

==========
Add OBB mounte Mojo interface

Add a new interface ObbMounter, and expose it to Android.
Actual features are not implemented yet.

BUG=613480

Committed: https://crrev.com/4265d7151753f4e3a7a4cbe688b423c376dd9503
Cr-Commit-Position: refs/heads/master@{#395557}
==========

[commit-bot: I haz the power, 2016-05-24 11:44:25]

Committed patchset #5 (id:180001)

[commit-bot: I haz the power, 2016-05-24 11:45:53]

Description was changed from

==========
Add OBB mounte Mojo interface

Add a new interface ObbMounter, and expose it to Android.
Actual features are not implemented yet.

BUG=613480

Committed: https://crrev.com/4265d7151753f4e3a7a4cbe688b423c376dd9503
Cr-Commit-Position: refs/heads/master@{#395557}
==========

to

==========
Add OBB mounte Mojo interface

Add a new interface ObbMounter, and expose it to Android.
Actual features are not implemented yet.

BUG=613480

Committed: https://crrev.com/4265d7151753f4e3a7a4cbe688b423c376dd9503
Cr-Commit-Position: refs/heads/master@{#395557}

Committed: https://crrev.com/593d78aa7f3dd422ae42723c2dc039b7059704da
Cr-Commit-Position: refs/heads/master@{#395572}
==========

[commit-bot: I haz the power, 2016-05-24 11:45:55]

Patchset 5 (id:??) landed as
https://crrev.com/593d78aa7f3dd422ae42723c2dc039b7059704da
Cr-Commit-Position: refs/heads/master@{#395572}