Remove FileUtilProxy:DeleteFile().

Remove FileUtilProxy:DeleteFile().

FileUtilProxy::DeleteFile() deletes a file on a TaskRunner and then
invokes a callback on the source thread. All call sites provide a null
callback. They might as well just post a base::DeleteFile() task to
the TaskRunner.

This is a prerequisite to stop allowing null callbacks in
base::PostTaskAndReplyWithResult().

BUG=162712
Committed: https://crrev.com/77c90085e7d90307d637256d0cb02718ab264c0a
Cr-Commit-Position: refs/heads/master@{#434765}

[fdoray, 2016-11-25 19:26:09]

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

[commit-bot: I haz the power, 2016-11-25 19:26:18]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-25 19:38:14]

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

[commit-bot: I haz the power, 2016-11-25 19:38:15]

Dry run: Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[fdoray, 2016-11-25 20:01:48]

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

[commit-bot: I haz the power, 2016-11-25 20:02:02]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-25 20:16:42]

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

[commit-bot: I haz the power, 2016-11-25 20:16:43]

Dry run: Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[fdoray, 2016-11-25 20:29:02]

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

[commit-bot: I haz the power, 2016-11-25 20:29:09]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-25 22:57:28]

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

[commit-bot: I haz the power, 2016-11-25 22:57:29]

Dry run: This issue passed the CQ dry run.

[fdoray, 2016-11-28 15:31:03]

fdoray@chromium.org changed reviewers:
+ dcheng@chromium.org

[fdoray, 2016-11-28 15:31:04]

PTAL

[dcheng, 2016-11-28 17:28:48]

lgtm

[fdoray, 2016-11-28 17:46:25]

fdoray@chromium.org changed reviewers:
+ erg@chromium.org, nparker@chromium.org

[fdoray, 2016-11-28 17:46:26]

nparker@chromium.org: Please review changes in chrome/browser/safe_browsing

erg@chromium.org: Please review changes in chrome/browser/ui/libgtkui/

[Elliot Glaysher, 2016-11-28 18:08:44]

lgtm

[Nathan Parker, 2016-11-28 18:26:31]

lgtm

[fdoray, 2016-11-28 18:32:42]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-11-28 18:33:14]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-28 23:05:50]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1480357962691480,
"parent_rev": "6410756988b35a801689b3be8ac525b7a85e5216", "commit_rev":
"e7c56847e90daa9e8564ab54d1723ca3a81fee73"}

[commit-bot: I haz the power, 2016-11-28 23:06:23]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-28 23:10:39]

Description was changed from

==========
Remove FileUtilProxy:DeleteFile().

FileUtilProxy::DeleteFile() deletes a file on a TaskRunner and then
invokes a callback on the source thread. All call sites provide a null
callback. They might as well just post a base::DeleteFile() task to
the TaskRunner.

This is a prerequisite to stop allowing null callbacks in
base::PostTaskAndReplyWithResult().

BUG=162712
==========

to

==========
Remove FileUtilProxy:DeleteFile().

FileUtilProxy::DeleteFile() deletes a file on a TaskRunner and then
invokes a callback on the source thread. All call sites provide a null
callback. They might as well just post a base::DeleteFile() task to
the TaskRunner.

This is a prerequisite to stop allowing null callbacks in
base::PostTaskAndReplyWithResult().

BUG=162712

Committed: https://crrev.com/77c90085e7d90307d637256d0cb02718ab264c0a
Cr-Commit-Position: refs/heads/master@{#434765}
==========

[commit-bot: I haz the power, 2016-11-28 23:10:40]

Patchset 3 (id:??) landed as
https://crrev.com/77c90085e7d90307d637256d0cb02718ab264c0a
Cr-Commit-Position: refs/heads/master@{#434765}