diff options
| author |  fmalita <fmalita@chromium.org> | 2015-10-13 08:59:23 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2015-10-13 08:59:23 -0700 |
| commit | 9ae8fe1c601ecb7fef9962f9eb1adf11032378e4 (patch) | |
| tree | 94b30174c895385a93279dee3ee6d66d928ddf55 /src | |
| parent | 598f3ff4b342fe0d57158ccea5059dfc981e8e3b (diff) | |
[SkTextBlob] Remove incorrect builder assert
At the end of TightRunBounds, glyphPosX cannot exceed the start of the
next run. But glyphPosY is running ahead of glyphPosX (for
kFull_Positioning) => the glyphPosY assert is incorrect.
Since the two pointers advance in lock-step, there isn't much value in
the glyphPosY assert anyway - we might as well remove it.
BUG=chromium:542643
R=reed@google.com,bungeman@google.com
Review URL: https://codereview.chromium.org/1405463004
Diffstat (limited to 'src')
| -rw-r--r-- | src/core/SkTextBlob.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/core/SkTextBlob.cpp b/src/core/SkTextBlob.cpp index 2ccf26b1c6..7fd96ad8b2 100644 --- a/src/core/SkTextBlob.cpp +++ b/src/core/SkTextBlob.cpp @@ -403,8 +403,6 @@ SkRect SkTextBlobBuilder::TightRunBounds(const SkTextBlob::RunRecord& run) { } SkASSERT((void*)glyphPosX <= SkTextBlob::RunRecord::Next(&run)); - SkASSERT(run.positioning() == SkTextBlob::kHorizontal_Positioning || - (void*)glyphPosY <= SkTextBlob::RunRecord::Next(&run)); return bounds.makeOffset(run.offset().x(), run.offset().y()); } |