Remove NOTREACHED that gets hit for non well formed xhtml.

Remove NOTREACHED that gets hit for non well formed xhtml.

BUG=627607
Committed: https://crrev.com/e0c18ae2e00183aaad8daaec74dd2e2ca869f673
Cr-Commit-Position: refs/heads/master@{#408766}

[Bryan McQuade, 2016-07-29 20:09:05]

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

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

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

[Bryan McQuade, 2016-07-29 20:10:22]

bmcquade@chromium.org changed reviewers:
+ inferno@chromium.org

[Bryan McQuade, 2016-07-29 20:10:32]

PTAL

[Charlie Harrison, 2016-07-29 20:12:05]

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

[Charlie Harrison, 2016-07-29 20:12:06]

Thank you for the change. Do you mind adding a browser test where we just
navigate to one of these documents?

[Bryan McQuade, 2016-07-29 20:15:02]

bmcquade@chromium.org changed reviewers:
- inferno@chromium.org

[Bryan McQuade, 2016-07-29 20:15:11]

On 2016/07/29 at 20:12:06, csharrison wrote:
> Thank you for the change. Do you mind adding a browser test where we just
navigate to one of these documents?

Coming right up.

[inferno, 2016-07-29 20:15:36]

inferno@chromium.org changed reviewers:
+ inferno@chromium.org

[inferno, 2016-07-29 20:15:36]

lgtm

[Bryan McQuade, 2016-07-29 20:34:46]

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

[Bryan McQuade, 2016-07-29 20:35:00]

On 2016/07/29 at 20:15:11, Bryan McQuade wrote:
> On 2016/07/29 at 20:12:06, csharrison wrote:
> > Thank you for the change. Do you mind adding a browser test where we just
navigate to one of these documents?
> 
> Coming right up.

Added a test. PTAL.

[commit-bot: I haz the power, 2016-07-29 20:35:25]

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-07-29 20:38:43]

lgtm thank you.

[Bryan McQuade, 2016-07-29 20:39:50]

The CQ bit was unchecked by bmcquade@chromium.org

[Bryan McQuade, 2016-07-29 20:39:52]

The CQ bit was checked by bmcquade@chromium.org

[Bryan McQuade, 2016-07-29 20:39:53]

The patchset sent to the CQ was uploaded after l-g-t-m from inferno@chromium.org
Link to the patchset: https://codereview.chromium.org/2194893003/#ps20001
(title: "add test")

[commit-bot: I haz the power, 2016-07-29 20:40:02]

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-07-29 21:26:17]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-29 21:29:13]

Description was changed from

==========
Remove NOTREACHED that gets hit for non well formed xhtml.

BUG=627607
==========

to

==========
Remove NOTREACHED that gets hit for non well formed xhtml.

BUG=627607

Committed: https://crrev.com/e0c18ae2e00183aaad8daaec74dd2e2ca869f673
Cr-Commit-Position: refs/heads/master@{#408766}
==========

[commit-bot: I haz the power, 2016-07-29 21:29:14]

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