Update timedtext unittest
unit test changes to match timedtext parser, which changed to
match the existing consumer. Mismatched use of parcel::write() vs writeByteArray()
Bug: 150915309
Test: atest TimedTextUnitTest, cts
Test: atest CtsMediaTestCases:android.media.cts.MediaPlayerTest#testChangeTimedTextTrack
Change-Id: I91de138f5a177b394e86bb31a5ec294249159ebe
diff --git a/media/libstagefright/timedtext/test/TimedTextUnitTest.cpp b/media/libstagefright/timedtext/test/TimedTextUnitTest.cpp
index 64379d6..d85ae39 100644
--- a/media/libstagefright/timedtext/test/TimedTextUnitTest.cpp
+++ b/media/libstagefright/timedtext/test/TimedTextUnitTest.cpp
@@ -346,6 +346,8 @@
ASSERT_EQ(parcel.readInt32(), fontRecordEntries[i].fontNameLength)
<< "Parcel has invalid value of font name length";
uint8_t fontName[fontRecordEntries[i].fontNameLength];
+ // written with writeByteArray() writes count, then the actual data
+ ASSERT_EQ(parcel.readInt32(), fontRecordEntries[i].fontNameLength);
status = parcel.read((void *)fontName, fontRecordEntries[i].fontNameLength);
ASSERT_EQ(status, 0) << "Failed to read the font name from parcel";
ASSERT_EQ(memcmp(fontName, fontRecordEntries[i].font, fontRecordEntries[i].fontNameLength),