From b345ddc3c728359166375af6f19367f547bbd613 Mon Sep 17 00:00:00 2001
From: Marco Nelissen <marcone@google.com>
Date: Thu, 8 Oct 2015 10:00:34 -0700
Subject: Don't send OnCompletion events when looping

AwesomePlayer never did, and neither should NuPlayer.

Bug: 18656047
Change-Id: Ia9abe42cfe6ea643633cec859e9da770092569bb
---
 media/libmediaplayerservice/nuplayer/NuPlayerDriver.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'media')

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 @@ void NuPlayerDriver::notifyListener_l(
                         // 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();
-- 
cgit v1.1