Adds resource_provider::ResourceProvider

Adds resource_provider::ResourceProvider

It's used to fetch resources for mojo apps.

R=ben@chromium.org

Committed: https://crrev.com/6de67f80026697580ed4cefa11a72193243bd4a0
Cr-Commit-Position: refs/heads/master@{#328014}

Committed: https://crrev.com/63795badcd1a2ba1a37605540bbbb12465d908ec
Cr-Commit-Position: refs/heads/master@{#328064}

[sky, 2015-05-01 15:25:25]

I realized reading the data and passing that around, while useful in some
situations, isn't always ideal. So, I'm going to change this to pass around
handles. Doing so requires some other deeper changes first. Will update this
soonish.

[Ben Goodger (Google), 2015-05-01 15:34:56]

I will look at this when I get in.

On Fri, May 1, 2015 at 11:25 AM, <sky@chromium.org> wrote:

> I realized reading the data and passing that around, while useful in some
> situations, isn't always ideal. So, I'm going to change this to pass around
> handles. Doing so requires some other deeper changes first. Will update
> this
> soonish.
>
> https://codereview.chromium.org/1108403008/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[sky, 2015-05-01 15:40:16]

You can hold off for now as I need to sort out access to platform
handles. That is here: https://codereview.chromium.org/1113393002/ .

On Fri, May 1, 2015 at 8:34 AM, Ben Goodger (Google) <ben@chromium.org> wrote:
> I will look at this when I get in.
>
> On Fri, May 1, 2015 at 11:25 AM, <sky@chromium.org> wrote:
>>
>> I realized reading the data and passing that around, while useful in some
>> situations, isn't always ideal. So, I'm going to change this to pass
>> around
>> handles. Doing so requires some other deeper changes first. Will update
>> this
>> soonish.
>>
>> https://codereview.chromium.org/1108403008/
>
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Ben Goodger (Google), 2015-05-01 17:19:07]

https://codereview.chromium.org/1108403008/diff/1/components/resource_provide...
File components/resource_provider/public/interfaces/resource_provider.mojom
(right):

https://codereview.chromium.org/1108403008/diff/1/components/resource_provide...
components/resource_provider/public/interfaces/resource_provider.mojom:12: //
must be relative, eg no leading '/'.
You should document what the root is relative to?

[sky, 2015-05-01 20:35:15]

I've updated the patch to pass around platform handles as well as to take a
request for an array of paths rather than a single path.

[Ben Goodger (Google), 2015-05-01 21:20:44]

lgtm

[sky, 2015-05-01 22:21:41]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2015-05-01 22:23:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1108403008/40001

[commit-bot: I haz the power, 2015-05-01 23:21:58]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-05-01 23:23:43]

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

[Dan Beam, 2015-05-01 23:44:48]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/1113223003/ by dbeam@chromium.org.

The reason for reverting is: Broke compile:
http://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN/builds/853/s....

[sky, 2015-05-02 15:36:45]

This was reverted because of accidental change here:
https://codereview.chromium.org/1108403008/diff/60001/mandoline/app/desktop/l...
. I've fixed that and am going to reland.

[sky, 2015-05-02 15:36:50]

The CQ bit was checked by sky@chromium.org

[sky, 2015-05-02 15:36:50]

The patchset sent to the CQ was uploaded after l-g-t-m from ben@chromium.org
Link to the patchset: https://codereview.chromium.org/1108403008/#ps80001
(title: "fix")

[commit-bot: I haz the power, 2015-05-02 15:36:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1108403008/80001

[commit-bot: I haz the power, 2015-05-02 16:29:13]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-05-02 16:30:03]

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