Mojo: add native type mapping for base::FilePath

Mojo: add native type mapping for base::FilePath

BUG=598821
Committed: https://crrev.com/cac9288c20778ff7507779a65533fc12d9a57d8c
Cr-Commit-Position: refs/heads/master@{#388225}

[yzshen1, 2016-04-13 20:42:08]

yzshen@chromium.org changed reviewers:
+ ben@chromium.org

[yzshen1, 2016-04-13 20:42:09]

Hi, Ben.

Would you please take a look? Thanks!

This CL depends on https://codereview.chromium.org/1832703002/

[Ben Goodger (Google), 2016-04-13 20:45:57]

could this live in //base/mojo ?

[yzshen1, 2016-04-13 20:52:16]

On 2016/04/13 20:45:57, Ben Goodger (Google) wrote:
> could this live in //base/mojo ?

I have chatted about this with John.
Although we won't have circular dependencies between build targets using
//base/mojo, there is concern about having files from two top-level folders
//base and //mojo include each other.

Do you think that is an issue? Thanks!

[Ben Goodger (Google), 2016-04-13 20:57:54]

On 2016/04/13 20:52:16, yzshen1 wrote:
> On 2016/04/13 20:45:57, Ben Goodger (Google) wrote:
> > could this live in //base/mojo ?
> 
> I have chatted about this with John.
> Although we won't have circular dependencies between build targets using
> //base/mojo, there is concern about having files from two top-level folders
> //base and //mojo include each other.
> 
> Do you think that is an issue? Thanks!

OK. lgtm

[yzshen1, 2016-04-19 15:44:04]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2016-04-19 15:44:12]

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

[commit-bot: I haz the power, 2016-04-19 16:54:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 19:12:57]

Description was changed from

==========
Mojo: add native type mapping for base::FilePath

BUG=598821
==========

to

==========
Mojo: add native type mapping for base::FilePath

BUG=598821

Committed: https://crrev.com/cac9288c20778ff7507779a65533fc12d9a57d8c
Cr-Commit-Position: refs/heads/master@{#388225}
==========

[commit-bot: I haz the power, 2016-04-22 19:12:58]

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