DCHECK that the document url is valid in HTMLPreloadScanner

DCHECK that the document url is valid in HTMLPreloadScanner

This patch asserts that the document url passed into the scanner is valid.
This fixes a fuzzer issue where the fuzzer was passing in an empty (invalid)
url.

BUG=644667
Committed: https://crrev.com/42754d4bc93465269b460b5912a3d7f30549e53c
Cr-Commit-Position: refs/heads/master@{#417092}

[Charlie Harrison, 2016-09-07 19:06:32]

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

[commit-bot: I haz the power, 2016-09-07 19:07:16]

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

[Charlie Harrison, 2016-09-07 20:21:39]

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

[commit-bot: I haz the power, 2016-09-07 20:23:07]

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

[Charlie Harrison, 2016-09-07 20:59:03]

csharrison@chromium.org changed reviewers:
+ kouhei@chromium.org

[Charlie Harrison, 2016-09-07 20:59:03]

kouhei@, PTAL?

[commit-bot: I haz the power, 2016-09-07 21:42:43]

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

[commit-bot: I haz the power, 2016-09-07 21:42:44]

Dry run: This issue passed the CQ dry run.

[kouhei (in TOK), 2016-09-07 22:16:18]

The CQ bit was checked by kouhei@chromium.org

[kouhei (in TOK), 2016-09-07 22:16:19]

lgtm

[commit-bot: I haz the power, 2016-09-07 22:16:51]

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-09-07 22:22:23]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-07 22:24:24]

Description was changed from

==========
DCHECK that the document url is valid in HTMLPreloadScanner

This patch asserts that the document url passed into the scanner is valid.
This fixes a fuzzer issue where the fuzzer was passing in an empty (invalid)
url.

BUG=644667
==========

to

==========
DCHECK that the document url is valid in HTMLPreloadScanner

This patch asserts that the document url passed into the scanner is valid.
This fixes a fuzzer issue where the fuzzer was passing in an empty (invalid)
url.

BUG=644667

Committed: https://crrev.com/42754d4bc93465269b460b5912a3d7f30549e53c
Cr-Commit-Position: refs/heads/master@{#417092}
==========

[commit-bot: I haz the power, 2016-09-07 22:24:25]

Patchset 2 (id:??) landed as
https://crrev.com/42754d4bc93465269b460b5912a3d7f30549e53c
Cr-Commit-Position: refs/heads/master@{#417092}

[Charlie Harrison, 2016-09-07 22:55:17]

Thanks!