commit 0e4bd4db6072472de7b7fca7c91f8ac854b2d24c
author Marco Nelissen <marcone@google.com> Thu Oct 08 10:00:34 2015 -0700
committer Marco Nelissen <marcone@google.com> Thu Oct 08 10:00:34 2015 -0700
tree 7bf0f4fd9bac69bf0ccee553e68fe6f8a4097e29
parent 1d00430722b24499373c40c6521be4eb41916f0e

Don't send OnCompletion events when looping

AwesomePlayer never did, and neither should NuPlayer.

Bug: 18656047
Change-Id: Ia9abe42cfe6ea643633cec859e9da770092569bb

media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp

diff --git a/media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp b/media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp
index 7370224..6419be0 100644
--- a/media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp
+++ b/media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp
@@ -747,7 +747,8 @@
                         // the last little bit of audio. If we're looping, we need to restart it.
                         mAudioSink->start();
                     }
-                    break;
+                    // don't send completion event when looping
+                    return;
                 }
 
                 mPlayer->pause();