summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Add `remoteControl` to HMICapability struct in Mobile APIfix/remote_control_supportedJacob Keeler2017-09-2611-0/+172
|
* Merge pull request #1775 from smartdevicelink/fix/update_mobile_api_4.4.0Jacob Keeler2017-09-263-6151/+6054
|\ | | | | Update MOBILE_API.xml to match `rpc_spec` repo
| * Update MOBILE_API.xml to match `rpc_spec` repofix/update_mobile_api_4.4.0jacobkeeler2017-09-223-6151/+6054
| | | | | | Includes fixes to build issues resulting from this update.
* | Merge pull request #1784 from smartdevicelink/fix/video_capabilities_validationJacob Keeler2017-09-261-3/+28
|\ \ | | | | | | Validate outgoing video params
| * | Validate outgoing video paramsfix/video_capabilities_validationjacobkeeler2017-09-251-3/+28
| | | | | | | | | Reverts this section back to https://github.com/shoamano83/sdl_core/commit/f6ea1262c4b6a3d1e10dc89fa570720a6c85ea9d#diff-4a7a19ae471dc741759dcc42796f9fc3R1513
* | | Merge pull request #1781 from APCVSRepo/fix/core-cannot-find-app-via-btJacob Keeler2017-09-261-0/+1
|\ \ \ | | | | | | | | Fix/core cannot find app via bt
| * \ \ Merge pull request #7 from ilikegithubcom/fix/core-cannot-find-app-via-btBrandon Shuliang He2017-09-121-0/+1
| |\ \ \ | | | | | | | | | | fix core cannot find app via bt.
| | * | | fix core cannot find app via bt.panliang012017-09-121-0/+1
| |/ / /
* | | | Merge pull request #1774 from ↵JackLivio2017-09-262-2/+8
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | AKalinich-Luxoft/hotfix/fix_cache_manager_hmi_types_getter Fix DISALLOWED response when app has specifically defined policies
| * | | Fix DISALLOWED response when app has specifically defined policiesAKalinich-Luxoft2017-09-222-2/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This problem mostly related to RC functionality. When SDL receives any RC request, it checks is sending app have REMOTE_CONTROL HMI type. If app does not have such HMI type, SDL disallows such RPC for it. The problem was in CacheManager::GetHMITypes() function, which returns appHMIType array even if it was not initialized(param is not present in policy) so SDL tries to find HMI type in an empty array and that always evaluates to false so SDL decides that current app is not RC app. If appHMITypes is not specified in policies, SDL should search HMI type in array of HMI types, provided by application on its registration. This check works correctly if application has default policies. To fix that problem there was added check in GetHMITypes() to avoid returning of uninitialized array.
* | | | Merge pull request #1776 from JackLivio/fix/haptic_data_nav_onlyJackLivio2017-09-252-1/+37
|\ \ \ \ | | | | | | | | | | Require nav or projection app for SendHapticData
| * | | | Require nav or projection app for SendHapticDataJackLivio2017-09-252-1/+37
| | |_|/ | |/| |
* | | | Merge pull request #1763 from ↵JackLivio2017-09-255-0/+122
|\ \ \ \ | |/ / / |/| | | | | | | | | | | shoamano83/fix/enable_system_capabilities_from_file Fix/enable system capabilities from file
| * | | Remove unnecessary set_xxx_supported() callsSho Amano2017-09-222-15/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These calls are not needed unless load_capabilities_from_file() is called multiple times. Unit test cases are also updated so that the function is called only once. Reflecting review comment.
| * | | fix: update supported flags of system capabilities when reading from fileSho Amano2017-09-131-0/+12
| | | |
| * | | Update unit tests to check system capability support flagsSho Amano2017-09-134-0/+81
| | |/ | |/|
* | | Merge pull request #1767 from smartdevicelink/fix/coverity_issues_4.4.0Jacob Keeler2017-09-216-23/+22
|\ \ \ | |_|/ |/| | Coverity fixes for 4.4.0
| * | Fix style issuesfix/coverity_issues_4.4.0jacobkeeler2017-09-213-9/+12
| | |
| * | Fix resource leaks introduced in RC baselinejacobkeeler2017-09-212-9/+4
| | | | | | | | | Fixed CIDs 174941 and 174934
| * | Fix coverity issues related to protocol changesjacobkeeler2017-09-213-7/+8
|/ / | | | | Fixes CIDs 174942, 174939, 174936, 174936
* | Merge pull request #1760 from ↵JackLivio2017-09-1916-48/+82
|\ \ | | | | | | | | | | | | AKalinich-Luxoft/fix/fix_core_crash_on_nonrc_app_exit Fix SDL crash on exit/unregistering of non-RC application
| * | Fix affected mocks and unit tests logicAKalinich-Luxoft2017-09-185-10/+23
| | | | | | | | | | | | | | | After adding new method to few classes some of unit tests and mocks have to be updated.
| * | Fix possible NULL pointer assignment in Resource Allocation ManagerAKalinich-Luxoft2017-09-181-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | One of the reasons of this defect is that shared_ptr was initied with NULL pointer so it causes DCHECK fatal error. ResourceAllocationManagerImpl::GetApplicationExtention() returns NULL pointer in some cases what is not correct. It was replaced with default constructed shared pointers.
| * | Fix SDL crash on exit/unregistering of non-RC applicationAKalinich-Luxoft2017-09-1811-35/+54
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SDL plugin manager is receiving notifications from application manager when any application was deactivated or unregistered. This notification will be transfered from plugin manager to RC plugin. However RC plugin processes events from any application so when it try to process event from non-RC application, SDL crashes. RC plugin should process events only from applications with RC functionality. Also there is another problem when application is unregistering its app_id() is removing from applications list in AM before OnApplicationEvent() call so when some of plugins will try to get shared_ptr using app_id, it will receive null pointer, however in AM still exists shared_ptr to this app. So it will be better to pass shared_ptr of application instead of accessing to it every time from plugins using app_id parameter. Moreover, some of plugins could require more information about application in future. Following changes were done: - Updated OnApplicationEvent() with application shared_ptr param for plugin manager and all related plugins - RC plugin now triggers OnApplicationEvent only for events from RC applications
* | Merge pull request #1761 from JackLivio/fix/metadata_unit_test_fixJackLivio2017-09-181-0/+7
|\ \ | | | | | | Fix unit test failure
| * | Fix unit test failureJackLivio2017-09-181-0/+7
|/ / | | | | | | This unit test was seen to fail on some machines. This fix makes sure that HMIToMobileResult does not return an older cached value defined earlier in the show_test class.
* | Merge pull request #1752 from ↵Jacob Keeler2017-09-111-1/+1
|\ \ | |/ |/| | | | | LuxoftAKutsan/hotfix/extended_policy_tests_build_issue Replace Functional module library
| * Replace Functional module libraryAlexander Kutsan2017-09-081-1/+1
|/ | | | | Replace Adding functional module libaries before application manager library in message helper test It fixes linkage issue.
* Merge remote-tracking branch ↵jacobkeeler2017-09-082-12/+21
|\ | | | | | | 'LuxoftAKutsan/fix/SDL_respond_ACK_with_protocol_version_4_for_video_and_audio_services_instead_of_negotiatied_version_protocol' into develop
| * fixup! Fix incorrect behaviour of SDL during respose ACK whenr video and ↵Alex Kutsan2017-06-232-5/+8
| | | | | | | | audio services starts
| * Fix incorrect behaviour of SDL during respose ACK whenr video and audio ↵okozlovlux2017-06-201-2/+5
| | | | | | | | | | | | | | | | | | | | services starts - added condition in SendStartSessionAck to avoid sending max supported protocol version when video or audio services start. SDL must send negotiated protocol_version from message from mobile app Related to Issue-1365
* | Merge pull request #1750 from JackLivio/fix/remove_equipment_rc_artifactJackLivio2017-09-071-1/+0
|\ \ | | | | | | Remove RC artifact
| * | Remove RC artifactJackLivio2017-09-071-1/+0
|/ /
* | Merge pull request #1624 from ↵Jacob Keeler2017-09-074-6/+19
|\ \ | | | | | | | | | | | | AKalinich-Luxoft/fix/fix_messages_default_language_checking Fix Consumer friendly message default language checking
| * | Fix Consumer friendly message default language checkingAKalinich-Luxoft2017-06-194-6/+19
| | | | | | | | | | | | | | | | | | | | | | | | If “en-us” is not present for any message, the table shall be rejected, PM should log error and shut SDL down. However this works only for external policies. Added missed checks for regular policies. kMandatoryLanguage was renamed to default_language_
* | | Merge pull request #1746 from ChrisB-Elektrobit/feature/metadata_taggingJacob Keeler2017-09-077-2/+358
|\ \ \ | | | | | | | | Feature/metadata tagging
| * | | - adding a test to cover the case where metadata tagging is provided but no ↵ChrisB-Elektrobit2017-08-211-12/+61
| | | | | | | | | | | | | | | | accompanying field data exists; this should result in a 'WARNINGS' result code
| * | | - performed renamings suggested in smartdevicelink/sdl_evolution#239ChrisB-Elektrobit2017-08-176-31/+30
| | | |
| * | | - changing Show Request metadata but no data warning result code from ↵ChrisB-Elektrobit2017-08-171-1/+1
| | | | | | | | | | | | | | | | IGNORED to WARNINGS
| * | | - adding warning result code and info message to ShowResponse to handleChrisB-Elektrobit2017-08-142-4/+21
| | | | | | | | | | | | | | | | | | | | the case where metadata tags are provided without a corresponding mainField entry.
| * | | - removed some extraneous logging to clean up the code a bitChrisB-Elektrobit2017-08-111-12/+4
| | | |
| * | | - added some additional unit tests to show_test.cc to test translation ofChrisB-Elektrobit2017-08-091-0/+130
| | | | | | | | | | | | | | | | metadata tags from input format to the HMI output format
| * | | - added new ShowRequest::HandleMetadata function for the Show RPC toChrisB-Elektrobit2017-08-084-0/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | handle mapping metadata values from incoming mobile protocol to outgoing HMI protocol - hooked HandleMetadata() into ShowRequest::run() for the four mainFields - added new metadata json string entries to smart_object_keys.h/.cc
| * | | - modified HMI_API.xml to support the changes needed for metadata taggingChrisB-Elektrobit2017-08-041-0/+42
| | | |
| * | | - modified MOBILE_API.xml to support the changes needed for metadataChrisB-Elektrobit2017-08-041-0/+62
| | | | | | | | | | | | | | | | tagging
* | | | Merge pull request #1615 from ↵Jacob Keeler2017-09-072-63/+70
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | AKalinich-Luxoft/fix/fix_consumer_friendly_messages_saving Fix Consumer Friendly Messages removing from PT
| * | | | Fixed logger messages contextAKalinich-Luxoft2017-06-162-8/+8
| | | | |
| * | | | Refactoring in SaveConsumerFriendlyMessages functionAKalinich-Luxoft2017-06-152-65/+70
| | | | | | | | | | | | | | | | | | | | Removed nested if's.
| * | | | Fix consumer friendly messages removing from PTAKalinich-Luxoft2017-06-151-3/+5
| | |/ / | |/| | | | | | | | | | | | | | | | | | In case PTU comes with omitted consumer_friendly_messages param SDL should maintain current consumer_friendly_messages section in Local PT.
* | | | Merge pull request #1620 from AByzhynar/fix/FixFactoryDefaultsJacob Keeler2017-09-078-13/+57
|\ \ \ \ | | | | | | | | | | Clear persistent data of all registered applications after FACTORY_DEFAULTS and MASTER_RESET