CR tweaks that missed the last CL

CR tweaks that missed the last CL

In crrev.com/1676943002 a couple of CR comments were made after the
commit button was checked. This applies those fixes.

BUG=440500
Committed: https://crrev.com/c6f6c69e52a78e9feeb7f7b82283764386f5570f
Cr-Commit-Position: refs/heads/master@{#376843}

[brucedawson, 2016-02-17 21:45:19]

Description was changed from

==========
CR tweaks that missed the last CL

In crrev.com/1676943002 a couple of CR comments were made after the
commit button was checked. This applies those fixes.

BUG=440500
==========

to

==========
CR tweaks that missed the last CL

In crrev.com/1676943002 a couple of CR comments were made after the
commit button was checked. This applies those fixes.

BUG=440500
==========

[brucedawson, 2016-02-17 21:45:19]

brucedawson@chromium.org changed reviewers:
+ thakis@chromium.org

[brucedawson, 2016-02-17 21:45:45]

Here's the promised fixes to that earlier change.

[brucedawson, 2016-02-22 20:55:37]

PTAL

[Nico, 2016-02-22 21:01:06]

lgtm

https://codereview.chromium.org/1708663003/diff/1/build/vs_toolchain.py
File build/vs_toolchain.py (right):

https://codereview.chromium.org/1708663003/diff/1/build/vs_toolchain.py#newco...
build/vs_toolchain.py:188: print 'Copying %s to %s...' % (ucrt_src_dir,
target_dir)
consider removing print, the other _CopyRuntimeImpl() callers also don't print

[brucedawson, 2016-02-22 21:20:11]

https://codereview.chromium.org/1708663003/diff/1/build/vs_toolchain.py
File build/vs_toolchain.py (right):

https://codereview.chromium.org/1708663003/diff/1/build/vs_toolchain.py#newco...
build/vs_toolchain.py:188: print 'Copying %s to %s...' % (ucrt_src_dir,
target_dir)
On 2016/02/22 21:01:06, Nico wrote:
> consider removing print, the other _CopyRuntimeImpl() callers also don't print

The print is here because _CopyRuntimeImpl is called with verbose=False, so
without a print here the copying of these files would be silent, which is
inconsistent with the copying of other files.

The verbose=False flag is used because otherwise all ~40 files are listed
multiple times and the output becomes unmanageably busy.

[brucedawson, 2016-02-22 21:41:52]

The CQ bit was checked by brucedawson@chromium.org

[commit-bot: I haz the power, 2016-02-22 21:44:34]

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

[commit-bot: I haz the power, 2016-02-22 23:09:25]

Description was changed from

==========
CR tweaks that missed the last CL

In crrev.com/1676943002 a couple of CR comments were made after the
commit button was checked. This applies those fixes.

BUG=440500
==========

to

==========
CR tweaks that missed the last CL

In crrev.com/1676943002 a couple of CR comments were made after the
commit button was checked. This applies those fixes.

BUG=440500
==========

[commit-bot: I haz the power, 2016-02-22 23:09:26]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-22 23:11:11]

Description was changed from

==========
CR tweaks that missed the last CL

In crrev.com/1676943002 a couple of CR comments were made after the
commit button was checked. This applies those fixes.

BUG=440500
==========

to

==========
CR tweaks that missed the last CL

In crrev.com/1676943002 a couple of CR comments were made after the
commit button was checked. This applies those fixes.

BUG=440500

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

[commit-bot: I haz the power, 2016-02-22 23:11:11]

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