diff options
author | Mihai Popa <popam@google.com> | 2019-06-25 11:15:18 +0100 |
---|---|---|
committer | Bryan Ferris <bferris@google.com> | 2019-07-10 18:02:10 +0000 |
commit | 209c0e824980630c7738c219eb4fe7d9504b9bf0 (patch) | |
tree | b86dbd363cce0216a8fdbd0305c6497e4cd06d8a /rs/java/android/renderscript/ProgramFragmentFixedFunction.java | |
parent | 1bc24d187715db74a40c7dabbb83507321ac53aa (diff) |
Fix Layout.primaryIsTrailingPreviousAllLineOffsets
The CL fixes a crash in Layout.primaryIsTrailingPreviousAllLineOffsets.
The crash was happening when the method was called for a line beginning
with an empty bidi run. This could happen, for example, for empty text -
I was unable to find any other case. The CL improves the existing test
for the method with this case, which was previously crashing.
The CL also fixes a potential crash in getLineHorizontals. However, this
bug could never happen as in the current code path clamped is always
false (and kept as parameter for parity with getHorizontal).
Bug: 135444178
Bug: 78464361
Test: atest FrameworksCoreTests:android.text.LayoutTest\#testPrimaryIsTrailingPrevious
Change-Id: I47157abe1d74675884734e3810628a566e40c1b4
(cherry picked from commit 7ad499d00716f45fffdf7331493ed21d1b8d9b77)
Diffstat (limited to 'rs/java/android/renderscript/ProgramFragmentFixedFunction.java')
0 files changed, 0 insertions, 0 deletions