Adds some tests for dart::bin::Directory.

Adds some tests for dart::bin::Directory.

R=asiva@google.com

Committed: https://github.com/dart-lang/sdk/commit/720ebb4a9ccba47ab821d1be6e113a2207052a09

[zra, 2016-08-25 17:57:51]

zra@google.com changed reviewers:
+ asiva@google.com, johnmccutchan@google.com

[siva, 2016-08-25 18:11:51]

lgtm

https://codereview.chromium.org/2277883005/diff/20001/runtime/bin/directory_t...
File runtime/bin/directory_test.cc (right):

https://codereview.chromium.org/2277883005/diff/20001/runtime/bin/directory_t...
runtime/bin/directory_test.cc:57: dart::bin::Directory::Delete(temp_dir, false);
is there a return code for this to ensure the delete was successful?

[zra, 2016-08-25 19:47:16]

https://codereview.chromium.org/2277883005/diff/20001/runtime/bin/directory_t...
File runtime/bin/directory_test.cc (right):

https://codereview.chromium.org/2277883005/diff/20001/runtime/bin/directory_t...
runtime/bin/directory_test.cc:57: dart::bin::Directory::Delete(temp_dir, false);
On 2016/08/25 18:11:51, siva wrote:
> is there a return code for this to ensure the delete was successful?

Yes. Returns bool. Added EXPECT here and others.

[zra, 2016-08-25 19:47:44]

Description was changed from

==========
Adds some tests for dart::bin::Directory.
==========

to

==========
Adds some tests for dart::bin::Directory.

R=asiva@google.com

Committed:
https://github.com/dart-lang/sdk/commit/720ebb4a9ccba47ab821d1be6e113a2207052a09
==========

[zra, 2016-08-25 19:47:45]

Committed patchset #3 (id:40001) manually as
720ebb4a9ccba47ab821d1be6e113a2207052a09 (presubmit successful).