Closed Bug 949016 Opened 7 years ago Closed 7 years ago
[B2G][Bluetooth File Transfer][Music] Playback slider unmovable and causes audio issues while playing MP3 recieved by bluetooth transfer,
Description: While playing MP3 sent over bluetooth playback slider does not move when interacted with and causes other parts of song to play simultaneusly. Repro Steps: 1) Updated Buri to Build ID: 20131211004003 2) Turn on bluetooth and pair to another device 3) Have paired device send .mp3 file via bluetooth transfer 4) Accept transfer request 5) Once transfer is complete, click on notification to open file 6) While file is playing move playback slider repeatedly Actual: Moving playback slider on bluetooth transferred .mp3 does not move from starting position and causes multiple parts of song to be played at once. Expected: Playback slider can be used on bluetooth transferred .mp3 without incident. Environmental Variables Device: Buri v 1.3 Mozilla RIL Build ID: 20131211004003 Gecko: http://hg.mozilla.org/releases/mozilla-aurora/rev/6bb84d0bc170 Gaia: 7a2ccae2a546ac4d981d250272dafa630c926422 Platform Version: 28.0a2 Firmware Version: V1.2_US_20131115 Notes: Repro frequency: 3/3, 100% See attached: screenshot and video clip
Issue occurs in today's 1.4 Buri build Result: Moving playback slider on bluetooth transferred .mp3 does not move from starting position and causes multiple parts of song to be played at once. Environmental Variables Device: Buri v 1.4 Mozilla RIL Build ID: 20131211040203 Gecko: http://hg.mozilla.org/mozilla-central/rev/12ea03a70243 Gaia: 6415b8b44068596404c10365394544e94edd5ce5 Platform Version: 29.0a1 Firmware Version: V1.2_US_20131115
Does this reproduce on 1.2?
Issue does NOT reproduce in 1.2 on Buri Result: Playback slider can be used on bluetooth transferred .mp3 without incident. Environmental Variables Device: Buri v 1.2 COM RIL Build ID: 20131211004007 Gecko: http://hg.mozilla.org/releases/mozilla-b2g26_v1_2/rev/43d7b300241a Gaia: 096722a9e2510ecdfe45ba7382d7d50826b82feb Platform Version: 26.0 RIL Version: 01.02.00.019.102 Firmware Version: V1.2_US_20131115
blocking-b2g: --- → 1.3?
Component: Gaia::Bluetooth File Transfer → Gaia::Music
This issue started to occur on the Buri 1.3 Build ID: 20131122123502 Gaia 2cc78d696482e0434b584f5645af55e3105e59a2 SourceStamp 3c4fc4279e6a BuildID 20131122123502 Version 28.0a1 Last working Buri 1.3 Build ID: 20131121040202 Gaia 71063dd91bc8cbb15ba335236ed67a1c5058bd58 SourceStamp cf378dddfac8 BuildID 20131121040202 Version 28.0a1
Tried the same issue on 2 Nexus 4. On Transferring image/audio(mp3) file from Phone 1 to Phone 2 , notification is received to accept the transfer. Click accept and Phone 2 restarts. Also I noticed, that "Visible to all" turns off when pairing. Its a random behavior. OS Version : 184.108.40.206-prerelease Platform version: 29.0a1 Build Identifier : 20131213153944 Git Commit Info : 2013-12-13 03:54:27 a6c2af8d
(In reply to amitav.anand from comment #5) > Tried the same issue on 2 Nexus 4. On Transferring image/audio(mp3) file > from Phone 1 to Phone 2 , notification is received to accept the transfer. > Click accept and Phone 2 restarts. > > Also I noticed, that "Visible to all" turns off when pairing. Its a random > behavior. > > OS Version : 220.127.116.11-prerelease > Platform version: 29.0a1 > Build Identifier : 20131213153944 > Git Commit Info : 2013-12-13 03:54:27 a6c2af8d There really isn't a reason for the needinfo request here - clearing flag.
Bad experience if multiple parts of the song is being played when you move the slider. Marking this a blocker.
blocking-b2g: 1.3? → 1.3+
This issue is reproducible on ZTE device also. On receiving the mp3 file via BT, 2 instance on the music player is being created. The function showCurrentView is being called on receiving file via BT from musicdb.oncreated = function(event) in music.js file. Also saw the traces and saw 2 prints from OMX codec being created "Attempting to allocate OMX node 'OMX.qcom.audio.decoder.mp3'" Analyzing more on this.
Okay, I found the root cause and it's another regression that caused by bug 939139, the width style of the progress bar was removed. This issue is reproduced by tapping on notification to open a received audio, so it's an open activity which uses the open.html in music app. The open.html includes different css files from index.html, and the width style of the progress bar is in "shared/style_unstable/progress_activity.css", which was not included in open.html and that's why the width of progress bar in open activity is 0. The patch should be simply including "shared/style_unstable/progress_activity.css" in open.html, or assign width: 100% to #player-seek-bar-progress.
Depends on: 939139
Jim, This is the patch that includes progress_activity.css in the open.html to fix this issue, would you please review it? thanks.
Attachment #8355437 - Flags: review?(squibblyflabbetydoo)
(In reply to Dominic Kuo [:dkuo] from comment #10) > Created attachment 8355437 [details] [review] > include the missing css file in open.html > > Jim, > > This is the patch that includes progress_activity.css in the open.html to > fix this issue, would you please review it? thanks. Hi, I tested the patch,it solves the progress bar issue but it seems there is one more issue with this bug. Same song is being played twice. After Pausing one instance of the song from UI, we can hear one more instance of the song still being played in background. I observed that, after the file is received via BT, when we click on the notification, control goes through bt_openReceivedFile in apps/system/js/bluetooth_transfer.js and this function is being called twice. Even the window.addEventListener in apps/music/js/open.js is called twice which according to the function prototype should be called only on "localized" events. And handleOpenActivity in music/open.js is being called 4 times. Still Analyzing it...
I believe that's another issue because this patch only modified the css and html files, js files are not involved. Please file a new bug if the issue you mentioned in comment 11 still reproducible, let's trace and fix in the new bug, thanks.
The issue that comment 11 mentioned is bug 956276, it's unrelated to my patch. Jim, while you are testing the patch, please note bug 956276 can cause 2-4 open activities at the same time, so probably you are unable to verify the patch if the open activity are not launched normally, let me know if you encounter that issue, thanks.
jim, any update on the review?
Comment on attachment 8355437 [details] [review] include the missing css file in open.html Sorry for the delay; this looks good! r=me
Attachment #8355437 - Flags: review?(squibblyflabbetydoo) → review+
Thanks Jim! Landed on master: c75a943bb4c2ca329c01406211c4fe77186d49cb
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Uplifted c75a943bb4c2ca329c01406211c4fe77186d49cb to: v1.3: ca0d5cd8715c0f2782e926da318fc0a396282266
You need to log in before you can comment on or make changes to this bug.