tag fadc3996a671d0d6027a3853f9e4b955c9ac6130
tagger The Android Open Source Project <initial-contribution@android.com> Fri Sep 06 15:10:59 2019 -0700
object 205355ee59dc5d5ef225a024c3bc66272428c0d8

Android 8.1.0 release 68

commit 205355ee59dc5d5ef225a024c3bc66272428c0d8
author Mihai Popa <popam@google.com> Tue Jun 25 11:15:18 2019 +0100
committer Arjun Garg <arjgarg@google.com> Thu Jul 11 12:16:39 2019 -0700
tree 9124c2d77d370c1db04d3ce858c1b385a9d25eac
parent 7193aefe1e5d065f8dcf1948e5a48eb55afa7c67

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)
(cherry picked from commit 42a6af7a11842cea3faa97d24dba209a97101bb3)