Move Touch to base namespace, remove SetLastModifiedTime.

Move Touch to base namespace, remove SetLastModifiedTime.

This implements callers of SetLastModifiedTime in terms of base::Touch.
SetLastModifiedTime was just a shorthand for touch that confusingly set
both the last modified and last access times, and was only used by 7 files.
It seems better to just have the callers call Touch directly.

BUG=
R=viettrungluu@chromium.org

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

[brettw, 2013-12-04 20:35:48]

This actually removes a function so don't rubberstamp.

[viettrungluu, 2013-12-04 21:09:22]

LGTM w/nit.

https://codereview.chromium.org/100453006/diff/1/base/files/file_util_proxy.cc
File base/files/file_util_proxy.cc (right):

https://codereview.chromium.org/100453006/diff/1/base/files/file_util_proxy.c...
base/files/file_util_proxy.cc:360: Bind(&base::TouchFile, file_path,
last_access_time, last_modified_time),
nit: you're already in the base namespace (ditto for various other occurrences
of base:: in this file, which you can take care of as a drive-by if you feel
like)

https://codereview.chromium.org/100453006/diff/1/chrome/browser/browsing_data...
File chrome/browser/browsing_data/browsing_data_remover_unittest.cc (right):

https://codereview.chromium.org/100453006/diff/1/chrome/browser/browsing_data...
chrome/browser/browsing_data/browsing_data_remover_unittest.cc:566: base::Time
now = base::Time::Now();
(Note that the new version isn't absolutely equivalent to the old version, since
the different calls to Now() in the old version may return different values. I
think it's fine for this test though.)

https://codereview.chromium.org/100453006/diff/1/content/browser/storage_part...
File content/browser/storage_partition_impl_unittest.cc (right):

https://codereview.chromium.org/100453006/diff/1/content/browser/storage_part...
content/browser/storage_partition_impl_unittest.cc:213: base::Time now =
base::Time::Now();
(ditto)

[brettw, 2013-12-04 21:38:17]

https://codereview.chromium.org/100453006/diff/1/base/files/file_util_proxy.cc
File base/files/file_util_proxy.cc (right):

https://codereview.chromium.org/100453006/diff/1/base/files/file_util_proxy.c...
base/files/file_util_proxy.cc:360: Bind(&base::TouchFile, file_path,
last_access_time, last_modified_time),
I'll get around to removing base from this file. I didn't want to change them in
this patch since it's not a search-and-replace one and I didn't want to add
noise.

[brettw, 2013-12-04 22:49:04]

Committed patchset #1 manually as r238803.