After detach, there's no document to notify parsing is finished.

After detach, there's no document to notify parsing is finished.

HTMLViewSourceParser was tripping the DCHECK in
DocumentParser::document when it recieved the last bytes of the
document it was parsing after it was detached.

BUG=664915
Committed: https://crrev.com/8e59e2d6feaa38b37836837e996b9ad91c0b3745
Cr-Commit-Position: refs/heads/master@{#434937}

[dominicc (has gone to gerrit), 2016-11-29 06:40:04]

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

[commit-bot: I haz the power, 2016-11-29 06:40:14]

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

[dominicc (has gone to gerrit), 2016-11-29 06:42:44]

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

[commit-bot: I haz the power, 2016-11-29 06:42:50]

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

[dominicc (has gone to gerrit), 2016-11-29 06:43:02]

dominicc@chromium.org changed reviewers:
+ yoichio@chromium.org

[dominicc (has gone to gerrit), 2016-11-29 06:43:09]

PTAL

[yoichio, 2016-11-29 07:02:49]

yoichio@chromium.org changed reviewers:
+ tkent@chromium.org, yosin@chromium.org

[tkent, 2016-11-29 07:07:37]

lgtm

https://codereview.chromium.org/2537043002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/html/parser/HTMLViewSourceParserTest.cpp
(right):

https://codereview.chromium.org/2537043002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/html/parser/HTMLViewSourceParserTest.cpp:26: }
Because there are no EXPECT_op()s in the test, please add expected result of the
test.  'no crash' in this case.

[dominicc (has gone to gerrit), 2016-11-29 07:32:45]

The CQ bit was checked by dominicc@chromium.org

[dominicc (has gone to gerrit), 2016-11-29 07:32:45]

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

[commit-bot: I haz the power, 2016-11-29 07:32: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, 2016-11-29 09:23:51]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1480404765423540,
"parent_rev": "7c23ef4c4b3b579fcf94672d8991de705fecd094", "commit_rev":
"271ac4c7ddc0a92c2551f1222c72af06dba427fb"}

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

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-29 09:34:44]

Description was changed from

==========
After detach, there's no document to notify parsing is finished.

HTMLViewSourceParser was tripping the DCHECK in
DocumentParser::document when it recieved the last bytes of the
document it was parsing after it was detached.

BUG=664915
==========

to

==========
After detach, there's no document to notify parsing is finished.

HTMLViewSourceParser was tripping the DCHECK in
DocumentParser::document when it recieved the last bytes of the
document it was parsing after it was detached.

BUG=664915

Committed: https://crrev.com/8e59e2d6feaa38b37836837e996b9ad91c0b3745
Cr-Commit-Position: refs/heads/master@{#434937}
==========

[commit-bot: I haz the power, 2016-11-29 09:34:45]

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

[battre, 2016-11-29 10:47:17]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/2538473004/ by battre@chromium.org.

The reason for reverting is: Speculative revert for crbug.com/669435.