Remove DumpWithoutCrashing for HasPermissionsForFileSystemFile/CanCommitURL.

Remove DumpWithoutCrashing for HasPermissionsForFileSystemFile/CanCommitURL.

This DumpWithoutCrashing turned out to be non-actionable (see
https://crbug.com/654479#c35).  Furthermore some forbidden accesses seem
unavoidable and in such cases DumpWithoutCrashing causes unnecessary
jank/latency during legitimate drag-and-drop scenarios (in case of
cross-site dragover/dragenter/dragleave that drags filesystem URIs - see
https://crbug.com/705295#c30).

BUG=654479, 705295
Review-Url: https://codereview.chromium.org/2826693003
Cr-Commit-Position: refs/heads/master@{#465317}
Committed: https://chromium.googlesource.com/chromium/src/+/c1ef2baa3dc05e2879e28bc22301e6bd713adad3

[Łukasz Anforowicz, 2017-04-18 18:18:30]

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

[commit-bot: I haz the power, 2017-04-18 18:19:10]

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

[Łukasz Anforowicz, 2017-04-18 18:21:31]

lukasza@chromium.org changed reviewers:
+ nick@chromium.org

[Łukasz Anforowicz, 2017-04-18 18:21:32]

nick@ - can you PTAL?  This CL just flush out the easy part of
https://crbug.com/705295...  This CL is arguably different and separate from the
actual fix - it seems better to explicitly remove the DwoC rather than burying
the removal in a CL that first moves the DwoC around (which is what the actual
fix for https://crbug.com/705295 needs to do).

[ncarter (slow), 2017-04-18 18:55:41]

lgtm

[ncarter (slow), 2017-04-18 18:55:46]

The CQ bit was unchecked by nick@chromium.org

[ncarter (slow), 2017-04-18 18:55:47]

The CQ bit was checked by nick@chromium.org

[commit-bot: I haz the power, 2017-04-18 18:56:16]

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, 2017-04-18 19:11:16]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492541747053260, "parent_rev":
"b3960c81c0209e718309ca0b09f3048653f0dfba", "commit_rev":
"c1ef2baa3dc05e2879e28bc22301e6bd713adad3"}

[commit-bot: I haz the power, 2017-04-18 19:12:41]

Description was changed from

==========
Remove DumpWithoutCrashing for HasPermissionsForFileSystemFile/CanCommitURL.

This DumpWithoutCrashing turned out to be non-actionable (see
https://crbug.com/654479#c35).  Furthermore some forbidden accesses seem
unavoidable and in such cases DumpWithoutCrashing causes unnecessary
jank/latency during legitimate drag-and-drop scenarios (in case of
cross-site dragover/dragenter/dragleave that drags filesystem URIs - see
https://crbug.com/705295#c30).

BUG=654479, 705295
==========

to

==========
Remove DumpWithoutCrashing for HasPermissionsForFileSystemFile/CanCommitURL.

This DumpWithoutCrashing turned out to be non-actionable (see
https://crbug.com/654479#c35).  Furthermore some forbidden accesses seem
unavoidable and in such cases DumpWithoutCrashing causes unnecessary
jank/latency during legitimate drag-and-drop scenarios (in case of
cross-site dragover/dragenter/dragleave that drags filesystem URIs - see
https://crbug.com/705295#c30).

BUG=654479, 705295

Review-Url: https://codereview.chromium.org/2826693003
Cr-Commit-Position: refs/heads/master@{#465317}
Committed:
https://chromium.googlesource.com/chromium/src/+/c1ef2baa3dc05e2879e28bc22301...
==========

[commit-bot: I haz the power, 2017-04-18 19:12:45]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/c1ef2baa3dc05e2879e28bc22301...