[Tizen-submit] 18062: Changes to Tizen:IVI:Release/MediaPlayer

noreply at tizen.org noreply at tizen.org
Wed Nov 20 03:22:18 GMT 2013


Hi,
Graydon, Tracy have made the following changes to MediaPlayer in project Tizen:IVI:Release. Please review and accept ASAP.

Thank You,

[This message was auto-generated]

---

Request: #18062

  submit:       Tizen:IVI:Release:build/MediaPlayer(cleanup) -> Tizen:IVI:Release


Message:
Submitter: Graydon, Tracy <tracy.graydon at intel.com>
Comments: submit/tizen_ivi_release/20131120.032336
Git project: profile/ivi/MediaPlayer
Tag: submit/tizen_ivi_release/20131120.032336
Commit: 5f908ecefe28d6258daa8fc7024242bfcde651d9 Re-organizing the code so that the players are object based. Adding code to remember what page you were on as well as the previous media Signed-off-by: brianjjones <brian.j.jones at intel.com>

State:   new        2013-11-20T03:20:11 tizenrobot
Comment: <no comment>
changes files:
--------------
--- MediaPlayer.changes
+++ MediaPlayer.changes
@@ -1,6 +0,0 @@
-* Tue Oct 15 2013 brianjjones <brian.j.jones at intel.com> submit/tizen/20131015.010041 at 0cdd066
-- Saving of the localStorage now occurs after the file has successfully loaded
-
-* Mon Oct 14 2013 brianjjones <brian.j.jones at intel.com> submit/tizen/20131010.205318 at a65dc75
-- Image player will now remember what image it was last on
-

spec files:
-----------
--- MediaPlayer.spec
+++ MediaPlayer.spec
@@ -1,8 +1,8 @@
 Name:       MediaPlayer
+VCS:        profile/ivi/MediaPlayer#5f908ecefe28d6258daa8fc7024242bfcde651d9
 Summary:    A HTML media player application
 Version:    0.0.1
 Release:    1
-VCS:        profile/ivi/MediaPlayer#submit/tizen/20131015.175409-0-gd76e369374ca1bae63bf2de4fa541f4693a33adf
 Group:      Applications/System
 License:    Apache 2.0
 URL:        http://www.tizen.org2

other changes:
--------------

++++++ MediaPlayer-0.0.1.tar.bz2
--- js/audioPlayer.js
+++ js/audioPlayer.js
@@ -35,6 +35,10 @@
 				this.clearAudioTimeInterval = setInterval(timeoutFunction.bind(this),500);
 			}
 		}
+		else
+		{
+			this.pause();
+		}
 	}
 	else
 		console.log(this.content[this.listIndex].artists[0] + " : " + this.content[this.listIndex].title + " can't play yet, it hasn't loaded");
@@ -87,6 +91,17 @@
 	this.playerControls.pause();
 	this.updateMediaName(this.content[this.listIndex].artists[0], this.content[this.listIndex].title, this.content[this.listIndex].coverArt);
 	$("#audioSrc").attr("src", this.content[this.listIndex].contentURI);
+
+	if (!this.content[this.listIndex].remoteFile)
+	{
+		localStorage.prevAudioTrack = this.content[this.listIndex].contentURI;
+	}
+	else
+	{
+		localStorage.prevAudioTime = undefined;
+		localStorage.prevAudioTrack = undefined;
+	}
+
 	this.playerControls.load();
 }
 
@@ -150,17 +165,6 @@
 	console.log("MediaPlayer in playLoadedMedia");
 	this.currentFileLoaded = true;
 
-	//Only store track to localStorage if it's a local file, and current menu is the Audio menu
-	if (!this.content[this.listIndex].remoteFile && localStorage.prevMenu === "mainMusicButton")
-	{
-		localStorage.prevAudioTrack = this.content[this.listIndex].contentURI;
-	}
-	else
-	{
-		localStorage.prevAudioTime = undefined;
-		localStorage.prevAudioTrack = undefined;
-	}
-
 	this.currentAudioContent = this.content[this.listIndex];
 
 	if (this.loadPrevAudio)
@@ -261,15 +265,18 @@
 					this.listIndex = i;
 					console.log("MediaPlayer: Previous audio found, loading " + this.content[this.listIndex].contentURI);
 					this.loadPrevAudio = true;
+					$("#audioSrc").attr("src", this.content[this.listIndex].contentURI);
+					this.playerControls.load();

Please refer to OBS webUI for more details about this SR.


More information about the Tizen-submit mailing list