summaryrefslogtreecommitdiffstats
path: root/src/com
Commit message (Collapse)AuthorAgeFilesLines
* Add error resolution for missing permissionandroid11-devSimon Dai2020-04-233-13/+30
| | | | | | | | | Prevents LMP from spam requesting permissions, will only request on user action (via error resolution). Bug: 153569299 Test: Manual Change-Id: I327b8b9a0f30d3b8921bd61438bffcd319399cfe
* Set active queue item id when pausedYabin Huang2019-12-201-0/+1
| | | | | | Bug: 144708514 Test: manual Change-Id: I7c48a89da92b2315088d92d1f698663c957e74c6
* Fix a crash when equals() is called on null StringYabin Huang2019-07-311-2/+1
| | | | | | Fixes: 138448649 Test: manual Change-Id: I4eda9f8e1bef72f83959b317fec900a1c243a51f
* DO NOT MERGE: Make LocalMediaPlayer use a same mk file for pi-car-dev and masterYabin Huang2019-03-121-1/+1
| | | | | | | branch Test: manual Change-Id: I697b774dc7c8aa2c15369bc51f9b0e4ac5f0d606
* Removing CATEGORY_LAUNCHER and improving permission handling.android-9.0.0_r47android-9.0.0_r46android-9.0.0_r45android-9.0.0_r44android-9.0.0_r43android-9.0.0_r42android-9.0.0_r41android-9.0.0_r40android-9.0.0_r39android-9.0.0_r38android-9.0.0_r37android-9.0.0_r36android-9.0.0_r35android-9.0.0_r34android-9.0.0_r33android-9.0.0_r32android-9.0.0_r31android-9.0.0_r30android-9.0.0_r22android-9.0.0_r21android-9.0.0_r20android-9.0.0_r19android-9.0.0_r16android-9.0.0_r12android-9.0.0_r11pie-qpr3-s1-releasepie-qpr3-releasepie-qpr3-b-releasepie-qpr2-releasepie-qpr1-s3-releasepie-qpr1-s2-releasepie-qpr1-s1-releasepie-qpr1-releasepie-dr1-releasepie-dr1-devpie-devpie-b4s4-releasepie-b4s4-devRoberto Perez2018-05-174-11/+17
| | | | | | | | | | | Make this application to be treated as a headless app (the UI will be provided by Media Template). Improve icons so they look decent on Media app picker. Remove default media item icons so Media Template can render the metadata correctly. Change-Id: Ibd5d465bea20136ff3d6319bd9c7d0c67a49114d Bug: 79222246 Test: TBD
* Respect Audio Focus during skip, shuffle and new selectionoreo-mr1-devBrad Stenning2017-09-051-3/+10
| | | | | | | | | | | Bug: 63140402 Test: 1. Start music 2. Start bluetooth call 3. Use next/prev/shuffle and song selection 4. confirm playback is not started. Change-Id: Icb6a408cdf2ca2c6e4fdfaaedcd19f456c86b6c3
* Use correct list for song shuffling.Brad Stenning2017-08-291-8/+6
| | | | | | | | | | | | | | | | I incorrectly assumed that the DataModel class held the source to truth for the current state to the play queue. However it holds the results of the last query done from LocalMeidaBrowserService. Thus it is not in the same state as that of the MediaSession queue which is what we are actually shuffling. Once again I'm just patching the errors in this class until a refactor can be planned. Bug: 65161162 Test: After starting to play music browser for a new song but don't select it. Then shuffle the songs again and confirm the same list was shuffled Change-Id: I96df1436fe3674a3d4a07b8b907194aed0caf31b
* Fix song selection after shuffleBrad Stenning2017-08-241-4/+13
| | | | | | | | | | | | | The list of songs shown to the user contain an id that is expected by the rest of the code to match that of their location (index) in a list. Bug: 64987333 Test: 1. Shuffle the list of songs being played 2. Selected a song from the play list 3. Confirm the selected song is now Playing Change-Id: Ia7d0bfacce16693cf8015af3ba17f8e05833a199
* Merge "The onSkipToQueueItem method was as calling play on before updating ↵TreeHugger Robot2017-08-171-9/+2
|\ | | | | | | the current index, the play method calls updatePlaybackStatePlaying which expects the mCurrentQueueIdx to already be set. I updated this to behave like the advance function whre it just sets the current index and calls playCurrentIndex" into oc-mr1-dev
| * The onSkipToQueueItem method was as calling play on before updating theBrad Stenning2017-08-161-9/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | current index, the play method calls updatePlaybackStatePlaying which expects the mCurrentQueueIdx to already be set. I updated this to behave like the advance function whre it just sets the current index and calls playCurrentIndex bug: 32740383 Test: Use the play list (queue) to select different media and not that each time you launch the list it now shows the 3 bars (current song indicator) beside the correct entry Change-Id: I3dcfd3ebdb1b8819b8b350f5a4afc8172c9a2497 (cherry picked from commit ee47297711f062d202f42c9d63eb760de68db1c1)
* | Merge "Internal queue was set to the same queue as the one in the data ↵TreeHugger Robot2017-08-171-1/+1
|\ \ | | | | | | | | | model. This protects the queue from changes by external classes." into oc-mr1-dev
| * | Internal queue was set to the same queue as the one in the data model.Brad Stenning2017-08-161-1/+1
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | This protects the queue from changes by external classes. Bug: 62352340 Test: If using default install you can use first select the the playlist from artist (has 154 elements) and play something from deep in the list (greater than 25) then play from the album list (it's size is 24). It would crash immediately before this fix. Change-Id: I64d9f392a3427ff37480469a22fd4a291ee66560 (cherry picked from commit d95a94eb57f87b30fe993365f4ee3ae0779ae535)
* / Remove the "when" information from the local media notification.Brad Stenning2017-08-161-0/+1
|/ | | | | | | | | | | | Knowing when the card was created is of no value for these controls and it was being interpreted the media duration. Bug: 62708944 Test: 1. Play an media and navigate to overview screen 2. Observe there's no time related info next to the card title Change-Id: I5bd6b4e034920151f7d5d623eb55566f9929fb02 (cherry picked from commit 5a26f65866c011d4a519c563e61dc75b32e70532)
* Save and restore playback position.oreo-dr1-devRakesh Iyer2017-05-161-1/+3
| | | | | | | | | | | | | | Save and restore the playback position in local media player. Earlier we were only saving the queue and current position in the queue, we now also save and restore the current position within a track. Test: 1. Started playing a song, switched away, switched back and verified playback resumed from where it had left off and not from the beginning of the track. 2. Rebooted device and verified that it does the same. Bug: 34682959 Change-Id: I4d5959300e1a278286a877391b33cd86d41b97b8
* Fix divide by zero.Rakesh Iyer2017-04-141-1/+1
| | | | | | | | | | | Local media player seems to have gotten itself into a state where the queue was empty and yet the controls were enabled. While this its not clear how this is even possible, guard against crashing by checking for an empty queue state before dividing by queue size. Bug: 36179863 Test: I looked at the code real hard. Change-Id: I2c4e3991798e17efe9ac1643c89ed963604086ca
* Make LocalMediaPlayer post a notification.Rakesh Iyer2017-03-022-0/+115
| | | | | | | | | | | | | | | | We need local media player to post a notification now since we don't automatically create ongoing action cards for media anymore given that we replaced home with the notification shade. While it would be convenient to post the notification from the media player, it would cause problems with 3rd party applications who post their own notifications, so it's best to post the notification from local media player itself. Having a notification also keeps the service running in the background. Test: Manually verified. Change-Id: Ie63642ca616ad3cb931b07fd93c97125f4b0574c
* Fix up bug in local media player state restoration.Rakesh Iyer2017-01-201-15/+22
| | | | | | | | | | | | | | Once state was restored, the onPlay callback was being invoked, but this was incorrect because it assumes that the media player was already set up. Changing the behavior of this method would break the play/pause functionality so instead put the necessary code inline, it is not a lot of code and it keeps things simple. Bug: 32097748 Test: Saved and restored playlist successfully. Change-Id: Id442270b5bd13c5c5126464a6788a82465711391
* Update package names to work with the proto3 compilerTamas Berghammer2016-11-071-2/+2
| | | | | Bug: b/28974522 Change-Id: Ie2eda765454b5ce5e5eb152fe2a38ae69eeea727
* Save and restore state in local media player.Rakesh Iyer2016-11-012-3/+116
| | | | | | | | | Local media player now saves off the last playing queue and restores it when the media template reconnects. Bug: 32097748 Test: Manual. Change-Id: Ic6c3772700849ac3a59ce93388654027e8167ed4
* Implement shuffle in local media player.Rakesh Iyer2016-10-311-2/+43
| | | | | | | | | | | | This change adds in a shuffle custom action for the local media player. The shuffle algorithm itself is pretty naive it just removes the current playing item from the list, shuffles the rest and puts the current item back on front of the list. Bug: 398865 Test: Manual. Change-Id: Idcdf2c00e508634de9bfff817d5a562a14cdc675
* Move local media player.Rakesh Iyer2016-10-205-0/+1084
Original sha1: f802a6f645c66e914ecfe2c1fd06e4dd1aadc6ef Credits: rni@ Bug: 32118797 Test: Manual. Change-Id: I66c20653be687f10189042cabf84ddbc3aaf949b