[Tizen-submit] 16132: Changes to Tizen:IVI/MediaPlayer

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


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

Thank You,

[This message was auto-generated]

---

Request: #16132

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


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:13 tizenrobot
Comment: <no comment>
changes files:
--------------
--- MediaPlayer.changes
+++ MediaPlayer.changes
@@ -0,0 +1,3 @@
+* 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
+

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/20131015.010041-0-g5af223ce00eaa4a7f2f55b87a4a24d4fff19754f
+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
@@ -87,18 +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);
-
-	//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.playerControls.load();
 }
 
@@ -162,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)
--- js/imagePlayer.js
+++ js/imagePlayer.js
@@ -93,6 +93,9 @@
 {
 	this.listIndex = index;
 
+	$("#imagePlayer").css("background", "url(" + this.content[this.listIndex].contentURI + ") no-repeat center");
+	$("#imagePlayer").css("background-size", "contain");
+
 	if (!this.content[this.listIndex].remoteFile)
 	{
 		localStorage.prevImage = this.content[this.listIndex].contentURI;
@@ -101,9 +104,6 @@
 	{
 		localStorage.prevImage = undefined;
 	}
-
-	$("#imagePlayer").css("background", "url(" + this.content[this.listIndex].contentURI + ") no-repeat center");
-	$("#imagePlayer").css("background-size", "contain");
 }

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


More information about the Tizen-submit mailing list