Delete CopyRecursiveDirNoCache from test_file_util.

Delete CopyRecursiveDirNoCache from test_file_util. 

This function was used in only one place and that place was wrong. 

The implementation was long and complicated on Windows, and long and 
complicated and copied verbatim from elsewhere with 3 lines different on Posix. 

The place it was used was in the proxy launcher when copying the profile. It's 
not clear to me why we wouldn't want the profile files in the filesystem cache 
when running tests. Quite the opposite, we want the tests to run as fast as 
possible. 

The only place this should matter is in the startup tests. And the startup 
tests do things to the profile after it gets copied that should page some files 
back in! This adds another step to the startup tests to evict the profile files 
for cold startup tests only. 

This is a reland of r191854 which broke the startup test. The previous patch replaced the CopyRecursive call with CopyDirectory which does something slightly different. This new patch implements a CopyDirectoryContents function to do what's required here.




patch from issue 13394003

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=192940

[brettw, 2013-04-05 23:36:58]

Patchset 1 is the same as the previousl one in
https://codereview.chromium.org/13394003/

Patchset 2 is the new one, so you can diff and see what changed.

[cpu_(ooo_6.6-7.5), 2013-04-08 17:47:00]

lgtm,

Check if you are actually trying startup_test which is what caused the rollback.

[brettw, 2013-04-08 23:00:58]

Committed patchset #2 manually as r192940 (presubmit successful).

[kojih, 2013-04-10 08:55:12]

Hi brettw,
please see:
https://code.google.com/p/chromium/issues/detail?id=226099