Allow directories for GN data lists.

Allow directories for GN data lists.

Allow the members of the "data" variable in GN targets to be either files or directories. This means storing them as strings rather than SourceFile objects, and rebasing as necessary. The documentation is updated accordingly.

Added a move constructor for OutputFile. While move constructors are discouraged, OutputFile is just a wrapper around a std::string for type checking, and I think this is in the spirit of things.

Committed: https://crrev.com/e903c0f86b126b25fedaa0439230da1576aeb476
Cr-Commit-Position: refs/heads/master@{#332712}

[brettw, 2015-06-03 21:58:55]

brettw@chromium.org changed reviewers:
+ scottmg@chromium.org

[brettw, 2015-06-03 21:59:02]

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

[commit-bot: I haz the power, 2015-06-03 22:00:05]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1155303008/1

[scottmg, 2015-06-03 22:11:31]

https://codereview.chromium.org/1155303008/diff/1/tools/gn/runtime_deps_unitt...
File tools/gn/runtime_deps_unittest.cc (right):

https://codereview.chromium.org/1155303008/diff/1/tools/gn/runtime_deps_unitt...
tools/gn/runtime_deps_unittest.cc:194:
dep_copy.data().push_back("//dep_copy.data");
should there be a test for directories here since they're rebased differently
depending on last char?

[commit-bot: I haz the power, 2015-06-03 22:16:41]

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

[commit-bot: I haz the power, 2015-06-03 22:16:42]

Dry run: This issue passed the CQ dry run.

[brettw, 2015-06-03 22:18:18]

Good idea, done.

[brettw, 2015-06-03 22:18:22]

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

[scottmg, 2015-06-03 22:19:09]

lgtm

[commit-bot: I haz the power, 2015-06-03 22:20:24]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1155303008/20001

[brettw, 2015-06-03 22:21:24]

The CQ bit was unchecked by brettw@chromium.org

[brettw, 2015-06-03 22:21:28]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2015-06-03 22:22:05]

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

[commit-bot: I haz the power, 2015-06-03 22:40:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-06-03 22:41:12]

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

[Dirk Pranke, 2015-06-03 23:10:40]

I have confirmed that this fixes the issue w/ isolates.