Chromium Code Reviews| Index: ui/gfx/render_text_harfbuzz.cc |
| diff --git a/ui/gfx/render_text_harfbuzz.cc b/ui/gfx/render_text_harfbuzz.cc |
| index 39f00409700ab1cabd97b6a76e9bbd569bc78e9a..87ce7bab1965c7be32b8cf2498e8c8d7f6ccf8bb 100644 |
| --- a/ui/gfx/render_text_harfbuzz.cc |
| +++ b/ui/gfx/render_text_harfbuzz.cc |
| @@ -404,9 +404,6 @@ class HarfBuzzLineBreaker { |
| // Merge segments that belong to the same run. |
| if (last_segment.run == segment.run) { |
| DCHECK_EQ(last_segment.char_range.end(), segment.char_range.start()); |
| - DCHECK_LE( |
| - std::abs(last_segment.x_range.end() - segment.x_range.start()), |
| - std::numeric_limits<float>::epsilon()); |
|
msw
2016/05/04 22:03:59
It seems like there's likely floating math roundin
xdai1
2016/05/04 22:49:46
I agree with you. Thanks for your suggestion!
|
| last_segment.char_range.set_end(segment.char_range.end()); |
| last_segment.x_range.set_end(SkScalarToFloat(text_x_) + |
| segment.width()); |