commit 63e7162df610c05794ae88ea67440e64bd96e591
author Yoshifumi Inoue <yosin@chromium.org> Wed May 22 03:14:54 2019
committer Commit Bot <commit-bot@chromium.org> Wed May 22 03:14:54 2019
tree 790c72cfd973c132200fc3650cdac665734e0be1
parent a0ca6599dbb9cc7e635ccc87a1134e19cf409ccb

Validate NGOffsetMapping at construction

This patch introduces |SECURITY_DCHECK()| in |NGOffsetMapping| constructor and
|NGOffsetMappingUnit| constructor to detect wrong behavior of
|NGOffsetMappingBuider| for ease of debugging.

This patch also remove |CHECK| for offsets in |NGOffsetMapping|[1] in
|Element#innerText| because of we check them at construction time.

[1] http://crrev.com/c/1619510 Make Element#innerText to use CHECK at StringView
construction

Bug: 964871, 965702
Change-Id: Ie03709d4eaddea6696f3d1bc009fd5184675ce8b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1621803
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#662048}