Fix DCHECK on SetBookmark.
authorvogelheim <vogelheim@chromium.org>
Wed, 27 May 2015 15:43:30 +0000 (08:43 -0700)
committerCommit bot <commit-bot@chromium.org>
Wed, 27 May 2015 15:43:42 +0000 (15:43 +0000)
commit21e6831ee0526449affa3ef9232e8890fa2617ad
tree4e4ca3ce74fff40c6a634d809c364fdcb6d3aef0
parent2a058de88fa6cfa6d7a76783269f39043f0d8afc
Fix DCHECK on SetBookmark.

The DCHECK was a lie. The idea was that - when a bookmark is set -
the scanner must clearly be at a character boundary and hence the
bookmark does not need to save a 'partial' UTF-8 code point. The
first part is true - the Scanner is always at a character boundary -
but the 'partial' UTF-8 code point is at the end of a block, not at
the current character position of the Scanner.
Hence, the 'partial' character needs to be saved as well.

jkummerow: Thanks for noticing.

BUG=chromium:470930
R=jochen@chromium.org, jkummerow@chromium.org
LOG=N

Review URL: https://codereview.chromium.org/1154773004

Cr-Commit-Position: refs/heads/master@{#28661}
src/scanner-character-streams.cc
src/scanner-character-streams.h