GitLab

Follow up to the recent
https://chromium-review.googlesource.com/c/chromium/src/+/1899374 where
one caller (extensions) was fixed and the DCHECK introduced.

Since there are many direct or indirect callers to
BookmarkModel::Remove() and is difficult to audit them all, this patch
upgrades the DCHECK into a CHECK to enforce the precondition.

Bug: 1021829
Change-Id: I6590650059ddf94b054ad0b6d22224f0ec531654
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911212
Auto-Submit: Mikel Astiz <mastiz@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714584}