Merge pull request #737 from billdawson/timob-6238

TIMOB-6238 implement videoplayer.start
tidev · Nov 20, 2011 · c3edf26 · c3edf26
2 parents 654677d + 4478948
commit c3edf26
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/android/modules/media/src/java/ti/modules/titanium/media/VideoPlayerProxy.java b/android/modules/media/src/java/ti/modules/titanium/media/VideoPlayerProxy.java
@@ -241,6 +241,15 @@ public void play()
 		control(MSG_PLAY);
 	}
 
+	/**
+	 * Backwards-compatibility
+	 */
+	@Kroll.method
+	public void start()
+	{
+		play();
+	}
+
 
 	@Kroll.method
 	public void pause()

diff --git a/drillbit/tests/media/media.js b/drillbit/tests/media/media.js
@@ -59,7 +59,8 @@ describe("Ti.Media tests", {
 		valueOf(player).shouldNotBeNull();
 		valueOf(player.add).shouldBeFunction();
 		valueOf(player.pause).shouldBeFunction();
-		valueOf(player.start).shouldBeFunction();
+		valueOf(player.play).shouldBeFunction(); // this is the documented way to start playback.
+		valueOf(player.start).shouldBeFunction(); // backwards compat.
 		valueOf(player.stop).shouldBeFunction();
 		if (!isAndroid) valueOf(player.setUrl).shouldBeFunction();
 		valueOf(player.hide).shouldBeFunction();