commit c4ddeeb06a9defc359e99d1ce20882e1d688d1b3
author Insun Kang <insun@google.com> Sat Apr 21 16:03:22 2012 +0900
committer Insun Kang <insun@google.com> Thu Apr 26 13:37:20 2012 +0900
tree 900e010d148a7ff33f3bf9cd1e5b6587a6de1577
parent e370bb62b89d2f2980f519392010ea08a24c558e

Bug fix : enqueue one next subtitle for each subtitle.

o Do not enqueue another subtitle when notifying a null message.
related-to-bug: 6401574

Change-Id: I366e643ffa1601c11a12e46fcf7c00a469c77448

media/libstagefright/timedtext/TimedTextPlayer.cpp

diff --git a/media/libstagefright/timedtext/TimedTextPlayer.cpp b/media/libstagefright/timedtext/TimedTextPlayer.cpp
index dc5f6b8..f855d90 100644
--- a/media/libstagefright/timedtext/TimedTextPlayer.cpp
+++ b/media/libstagefright/timedtext/TimedTextPlayer.cpp
@@ -112,15 +112,14 @@
               break;
             }
             sp<RefBase> obj;
-            msg->findObject("subtitle", &obj);
-            if (obj != NULL) {
+            if (msg->findObject("subtitle", &obj)) {
                 sp<ParcelEvent> parcelEvent;
                 parcelEvent = static_cast<ParcelEvent*>(obj.get());
                 notifyListener(&(parcelEvent->parcel));
+                doRead();
             } else {
                 notifyListener();
             }
-            doRead();
             break;
         }
         case kWhatSetSource: {