summaryrefslogtreecommitdiff
path: root/src/components/media_manager/src/streamer_adapter.cc
Commit message (Collapse)AuthorAgeFilesLines
* fix: reduce logs during streaming (#1778)Sho Amano2021-04-141-1/+1
| | | | | | | | | | | * Reduce some logs to TRACE level Each time SDL Core receives a video packet from mobile, it generates a large amount of logs. Instead of completely removing them, this commit decreases their levels to TRACE. Head unit systems with video streaming feature can set log output level to DEBUG (or higher) to get rid of the logs. Co-authored-by: Jacob Keeler <jacob.keeler@livioradio.com>
* [SDL-0296] Possibility to update video streaming capabilities during ↵Iryna Lytvynenko (GitHub)2021-02-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ignition cycle (#3468) * Temporary commit to change URL of rpc_spec repository * Change rpc_spec reference according to the new changes in the rpc_spec repo * Add the new OnAppCapabilityUpdated RPC with appropriate types to the HMI_API * Add the smart objects keys for the new parameters * Add OnAppCapabilityUpdatedNotification and BCOnAppCapabilityUpdatedNotification * Add Unit tests for the OnAppCapabilityUpdated RPCs * Fix code generation for recursive structures Currently InterfaceGenerator is not able to generate recursive structures i.e. one or several structure attributes have its own type e.g.: struct VideoStreamingCapability { ... VideoStreamingCapability additionalVideoStreamingCapabilities; } because structure type fully defined only when all of it attributes are defined. Otherwise param with type of structure will have AlwaysFalseSchema for validation so this param will never be validated successfully. With this commit InterfaceGenerator is able to handle such problem. * Update OnBCSystemCapabilityUpdatedNotificationFromHMITest Unit tests Add the checking of the SystemCapabilityType::VIDEO_STREAMING in the message and setting it into the HMICapabilities. * Add additionalVideoStreamingCapabilities to default hmi_capabilities * Add Processing OnSystemCapabilityUpdated with videoStreamingCapabilities * Update Unit tests for OnSystemCapabilityUpdated notifications * Add app_id to OnAppCapabilityUpdated notification to HMI * Updates during code review * Update logger * Update link to rpc_spec * Return back the driverDistractionCapability that was missed during conflicts resolving * Fix functionality that was broken during conflicts resolving * Updates during code review * Update link to rpc_spec * Revert "Fix streamer activity in case of suspend (#3488)" This reverts commit 42721d5c56dfc26e3d69f7d80d7806110e570cac. * Adjust old fix for all streaming transports * Change commit to rpc_spec * Update rpc_spec * Mark function as deprecated * Updates during code review * Remove trailing whitespaces * Update link to rpc_spec * Update rpc_spec Co-authored-by: Igor Gapchuk <igapchuck@luxoft.com> Co-authored-by: YarikMamykin <ymamykin@gmail.com> Co-authored-by: Andrii Kalinich <AKalinich@luxoft.com> Co-authored-by: Dmitriy Boltovskiy <dboltovskyi@luxoft.com>
* Fix streamer activity in case of suspend (#3488)Andrii Kalinich (GitHub)2020-10-051-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fix streamer activity in case of suspend The problem of the current implementation is that SDL kills streaming thread responsible for sending a/v streaming data to HMI when streaming timeout expires. This issue is observed when mobile app dumps 10-seconds audio file to SDL during 2 seconds and after that app does not send any data. In that case HMI will play audio file during 2 seconds + timeout = 5 seconds. At the 5th second SDL just kills streaming thread with all pending messages, however audio service is still open. As a result not a whole audio file is played. The correct behavior from SDL side in that case is not kill streaming thread when streaming timeout was expired. SDL should kill streaming thread only when service is actually stopped. Current SDL behavior was updated to align with a correct behavior described above. * Adjust fix to work with audio socket streaming * Revert "[WIP] initial fix of socket streaming by guessing playback time (#3057)" This reverts commit 7c442abce73c462e7724a64b3ce037bcb8e9093e. * Remove unused code after revert * Fix style issues
* [SDL 0046] Implement logger abstraction (#3472)VladSemenyuk2020-09-011-21/+17
| | | | | | | | | | | | | | | | | | | | | | | | * Implement logger abstraction. Integrate it to SDL. Create one logger instance for shared libraries * Rename all logger macros. * Update src/components/application_manager/rpc_plugins/rc_rpc_plugin/src/resource_allocation_manager_impl.cc Co-authored-by: Shobhit Adlakha <ShobhitAd@users.noreply.github.com> * Fix review comments(errors in log messages) * Fix review coments * Fix code style * Add logger variable creation to new commads * Fix review comment * Fixe review comment(class imblemtation moved to .cc file) Co-authored-by: Shobhit Adlakha <ShobhitAd@users.noreply.github.com>
* Fix avoid race conditions (#3471)Serhii Niukalov (GitHub)2020-08-211-4/+4
| | | | | | | | | | | | | | * Introduced enums instead multiple bool flags Update threadFunc logic Update Start logic Update Stop logic Update Join logic * Update of the affected code * UTs update Co-authored-by: sniukalov <sniukaov@luxoft.com>
* Fixed pip audio streaming cutoff issueShobhitAd2019-06-211-0/+4
|
* fix: invalid memory access in MediaManager when stopping CoreSho Amano2018-07-061-1/+1
| | | | | | | Since ThreadDelegate calls set_delegate() inside its destructor, we need to delete ThreadDelegate first, then Thread. This is detected by valgrind.
* Fix/1701 (#1702)fronneburg2017-10-031-1/+4
| | | | proper socket streamer cleanup
* Fix invalid memory access.Vladislav Antonov2016-07-121-1/+1
| | | | | | | | Fixed access to deleted fields in Timer and StreamerAdapter. Fixed uninitialized values in request_controller. Related Issue: APPLINK-25098
* Format all code in projectKozoriz2016-04-261-19/+18
| | | | | | | | | | Formated all code in appMain, components, plugins to correct coding-style Used clang-format-3.6 Used 2 commands : find src/appMain/ -name "*.h" -o -name "*.cc" -o -name "*.hpp" -o -name "*.cpp" | xargs clang-format-3.6 -i -style=file find src/components/ -name "*.h" -o -name "*.cc" -o -name "*.hpp" -o -name "*.cpp" | xargs clang-format-3.6 -i -style=file find src/plugins/ -name "*.h" -o -name "*.cc" -o -name "*.hpp" -o -name "*.cpp" | xargs clang-format-3.6 -i -style=file
* Check for empty queue in MessageQueue::pop() methodArtem Nosach2015-10-041-1/+5
| | | | | | | | Return bool value from MessageQueue::pop() method that shows if pop operation was successful. Add default constructors to all classes used as message for message queue. Closes-bug: APPLINK-14928
* Fix review commentsArtem Nosach2015-10-041-21/+20
| | | | Related-issues: APPLINK-15198
* Create streamer adapter base classArtem Nosach2015-10-041-0/+170
Create base class for all streamer adapters. Streamer adapter holds message queue for stream data messages and thread that sends theese messages to listeners. Related-issues: APPLINK-15198