PlzNavigate: add DumpWithoutCrashing for invalid main resource request.

PlzNavigate: add DumpWithoutCrashing for invalid main resource request.

This CL is an attempt to solve https://crbug.com/705508
A DumpWithoutCrashing is added on the renderer side to get more
informations.

BUG=705508
Review-Url: https://codereview.chromium.org/2805693002
Cr-Commit-Position: refs/heads/master@{#462517}
Committed: https://chromium.googlesource.com/chromium/src/+/35be0078448bc5bc5198eac37562089682a555c1

[arthursonzogni, 2017-04-06 14:08:46]

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

[commit-bot: I haz the power, 2017-04-06 14:09:02]

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

[arthursonzogni, 2017-04-06 14:43:56]

arthursonzogni@chromium.org changed reviewers:
+ clamy@chromium.org

[arthursonzogni, 2017-04-06 14:43:57]

Hi Camille,
Please take a look. Thanks!

[clamy, 2017-04-06 15:02:50]

Thanks! Lgtm with a comment.

https://codereview.chromium.org/2805693002/diff/20001/content/child/web_url_l...
File content/child/web_url_loader_impl.cc (right):

https://codereview.chromium.org/2805693002/diff/20001/content/child/web_url_l...
content/child/web_url_loader_impl.cc:617: // browser. This should not happen.
Add a
// TODO(arthursonzogni): Remove this when the root cause for crbug.com/705508 is
found.

[arthursonzogni, 2017-04-06 15:09:29]

On 2017/04/06 15:02:50, clamy (slow) wrote:
> Thanks! Lgtm with a comment.
> 
>
https://codereview.chromium.org/2805693002/diff/20001/content/child/web_url_l...
> File content/child/web_url_loader_impl.cc (right):
> 
>
https://codereview.chromium.org/2805693002/diff/20001/content/child/web_url_l...
> content/child/web_url_loader_impl.cc:617: // browser. This should not happen.
> Add a
> // TODO(arthursonzogni): Remove this when the root cause for crbug.com/705508
is
> found.

Thanks for the review.

[arthursonzogni, 2017-04-06 15:09:33]

The CQ bit was checked by arthursonzogni@chromium.org

[arthursonzogni, 2017-04-06 15:09:33]

The patchset sent to the CQ was uploaded after l-g-t-m from clamy@chromium.org
Link to the patchset: https://codereview.chromium.org/2805693002/#ps40001
(title: "Add TODO.")

[commit-bot: I haz the power, 2017-04-06 15:09:52]

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-06 17:04:17]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1491491373116600,
"parent_rev": "4762aebd36e6a5b18565bd9b76e419523e5ac903", "commit_rev":
"35be0078448bc5bc5198eac37562089682a555c1"}

[commit-bot: I haz the power, 2017-04-06 17:05:02]

Description was changed from

==========
PlzNavigate: add DumpWithoutCrashing for invalid main resource request.

This CL is an attempt to solve https://crbug.com/705508
A DumpWithoutCrashing is added on the renderer side to get more
informations.

BUG=705508
==========

to

==========
PlzNavigate: add DumpWithoutCrashing for invalid main resource request.

This CL is an attempt to solve https://crbug.com/705508
A DumpWithoutCrashing is added on the renderer side to get more
informations.

BUG=705508

Review-Url: https://codereview.chromium.org/2805693002
Cr-Commit-Position: refs/heads/master@{#462517}
Committed:
https://chromium.googlesource.com/chromium/src/+/35be0078448bc5bc5198eac37562...
==========

[commit-bot: I haz the power, 2017-04-06 17:05:03]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/35be0078448bc5bc5198eac37562...