arc: Add methods to IntentHelperInstance to read ARC contents

arc: Add methods to IntentHelperInstance to read ARC contents

BUG=654684
Committed: https://crrev.com/6f10b685c5aa13623e76e101e5f22e6b164221d7
Cr-Commit-Position: refs/heads/master@{#430185}

[hashimoto, 2016-11-02 08:12:19]

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

[commit-bot: I haz the power, 2016-11-02 08:12:37]

Dry run: 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-11-02 08:36:21]

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

[commit-bot: I haz the power, 2016-11-02 08:36:21]

Dry run: This issue passed the CQ dry run.

[hashimoto, 2016-11-02 08:43:53]

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

[hashimoto, 2016-11-02 08:43:54]

This is part of a series of CLs:
1. https://codereview.chromium.org/2469963003 Add methods to
IntentHelperInstance to read ARC contents <- THIS
2. https://codereview.chromium.org/2467213002 Implement Read and GetFileSize for
ARC content file system
3. https://codereview.chromium.org/2464333003 Add utility functions to implement
ARC content file system
4. https://codereview.chromium.org/2469243002 Implement
ArcContentFileSystemAsyncFileUtil::GetFileInfo
5. https://codereview.chromium.org/2470843004 Implement
ArcContentFileSystemFileStreamReader::Read/GetLength
6. https://codereview.chromium.org/2464333004 Convert content: and file: URLs
from ARC to externalfile: URLs

[hashimoto, 2016-11-02 08:46:43]

Correction: #2 (2467213002) is not meant to be submitted. The next CL is #3
(2464333003).

[rickyz (no longer on Chrome), 2016-11-02 19:55:02]

mojom lgtm (just to make sure, these are IPCs implemented on the Android side
that Chrome calls, right?

[hashimoto, 2016-11-04 00:12:19]

On 2016/11/02 19:55:02, rickyz wrote:
> mojom lgtm (just to make sure, these are IPCs implemented on the Android side
> that Chrome calls, right?

Yes, Android side will be implemented with Android's ContentResolver so that
Chrome can access the contents.

[hidehiko, 2016-11-04 17:08:10]

lgtm

[hashimoto, 2016-11-06 04:26:19]

The CQ bit was checked by hashimoto@chromium.org

[commit-bot: I haz the power, 2016-11-06 04:26:29]

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-11-06 04:57:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-06 04:59:56]

Description was changed from

==========
arc: Add methods to IntentHelperInstance to read ARC contents

BUG=654684
==========

to

==========
arc: Add methods to IntentHelperInstance to read ARC contents

BUG=654684

Committed: https://crrev.com/6f10b685c5aa13623e76e101e5f22e6b164221d7
Cr-Commit-Position: refs/heads/master@{#430185}
==========

[commit-bot: I haz the power, 2016-11-06 04:59:56]

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