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

noreply at tizen.org noreply at tizen.org
Tue Oct 15 17:51:40 GMT 2013


Hi,
brianjjones 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: #16133

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


Message:
Submitter: brianjjones <brian.j.jones at intel.com>
Comments: Saving of the localStorage now occurs after the file has successfully loaded
Git project: profile/ivi/MediaPlayer
Tag: submit/tizen/20131015.175409
Commit: d76e369374ca1bae63bf2de4fa541f4693a33adf Saving of the localStorage now occurs after the file has successfully loaded

State:   new        2013-10-15T17:50:15 tizenrobot
Comment: <no comment>
changes files:
--------------
--- MediaPlayer.changes
+++ MediaPlayer.changes
@@ -0,0 +1,6 @@
+* 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
@@ -2,7 +2,7 @@
 Summary:    A HTML media player application
 Version:    0.0.1
 Release:    1
-VCS:        profile/ivi/MediaPlayer#submit/tizen/20131010.205318-0-g5f908ecefe28d6258daa8fc7024242bfcde651d9
+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,10 +35,6 @@
 				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");
@@ -91,17 +87,6 @@
 	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();
 }
 
@@ -165,6 +150,17 @@
 	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)
@@ -265,18 +261,15 @@
 					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();
 					this.fillMediaList();

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


More information about the Tizen-submit mailing list