Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CCB: SDL forw OnButtonEvent of CUSTOM_BTN to BACKGROUND #967

Closed
KhrystynaDubovyk opened this issue Nov 7, 2016 · 14 comments
Closed

CCB: SDL forw OnButtonEvent of CUSTOM_BTN to BACKGROUND #967

KhrystynaDubovyk opened this issue Nov 7, 2016 · 14 comments
Assignees
Labels

Comments

@KhrystynaDubovyk
Copy link

KhrystynaDubovyk commented Nov 7, 2016

Bug Report

SDL not forwards OnButtonEvent notification of CUSTOM_BUTTON to BACKGROUND App

Reproduction Steps
  1. Make sure app is in BACKGROUND HMI level
  2. Send OnButtonEvent(CUSTOM_BUTTON, SHORT/LONG) from HMI
Expected Behavior

Notification is re-sent to App

Observed Behavior

SDL didn't resend notification to App

OS & Version Information

commit:

  • OS/Version: Ubuntu 16.04
  • SDL Core Version: a4c2702
  • Testing Against: ATF
Test Case, Sample Code, and / or Example App
@LitvinenkoIra
Copy link
Contributor

Priority Medium.
Loss of functionality on SDL side with standard use-case.

@robinmk
Copy link

robinmk commented May 16, 2018

We do not think the same requirement should apply to CUSTOM-BUTTON and hence think this bug is invalid.
Eg: Lets say we provide the ability for apps to populate softbuttons on the app list (like a short cut to the most widely used buttons). Then we would not want to limit the button presses being sent to the app in BACKGROUND.

@GetmanetsIrina
Copy link

@robinmk , please confirm the Expected result:

Expected result:
In case OnButtonEvent(CUSTOM_BUTTON, SHORT/LONG) is allowed by Policy SDL must resend it to BACKGROUND App.

Requirement

In case there is a media application in FULL HMILevel and several other media applications in BACKGROUND and-or NONE HMILevel connected to SDL and subscribed for Tune Up/Down, Seek Right/Left HW buttons, SDL must transfer the OnButtonEvent notification from HMI to the media application in FULL only.

will be still actual for Tune Up/Down, Seek Right/Left HW buttons.

@robinmk
Copy link

robinmk commented Jul 10, 2018

@GetmanetsIrina @AStasiuk

Requirement
In case there is a media application in FULL HMILevel and several other media applications in BACKGROUND and-or NONE HMILevel connected to SDL and subscribed for Tune Up/Down, Seek Right/Left HW buttons, SDL must transfer the OnButtonEvent notification from HMI to the media application in FULL only.
will be still actual for Tune Up/Down, Seek Right/Left HW buttons

This still holds true for Tune Up/Down, Seek Left/Right buttons. The rule should be expanded to include Media applications that are in LIMITED as well. The behavior for LIMITED should be similar to applications in FULL.

Regarding CUSTOM_BUTTON, I would like to refer you to the description provided in HMI_API for the OnButtonEvent RPC. The description states rules that define the correct behavior.
OnButtonEvent

@GetmanetsIrina
Copy link

Defect is reproduced.
SDL branch: develop(af97446)
ATF: develop
Script: smartdevicelink/sdl_atf_test_scripts#1921

@ghost
Copy link

ghost commented Jul 16, 2018

The next link to PR which provided fix this issue:
#2365

@ghost ghost removed their assignment Jul 30, 2018
@KrupenichOleg KrupenichOleg changed the title SDL forwards OnButtonEvent notification of CUSTOM_BUTTON to BACKGROUND App. SDL not forwards OnButtonEvent notification of CUSTOM_BUTTON to BACKGROUND App. Nov 12, 2018
@KrupenichOleg
Copy link

  1. Description and summary updated according to comments written by @robinmk, this defect is valid now
  2. Uploaded fresh script to reproducing this case (updated script from PR#1921)

@KrupenichOleg
Copy link

Defect is reproduced.
SDL branch: develop (d363167)
ATF: develop
Script: smartdevicelink/sdl_atf_test_scripts#1921

@LitvinenkoIra LitvinenkoIra changed the title SDL not forwards OnButtonEvent notification of CUSTOM_BUTTON to BACKGROUND App. CCB: SDL forw OnButtonEvent of CUSTOM_BTN to BACKGROUND Dec 9, 2019
@LitvinenkoIra
Copy link
Contributor

Defect is reproduced.
SDL branch: develop(e137556)
ATF: develop
Script: smartdevicelink/sdl_atf_test_scripts#1921

@LitvinenkoIra
Copy link
Contributor

#2365 - rebased to the latest develop
Fix is valid

@LitvinenkoIra
Copy link
Contributor

Full regression successfully passed
SDL: fix/sdl_forwards_onbuttoneven c1b6be3
Scripts: develop 6f43bfd
ATF: develop 37da7aba

@LitvinenkoIra
Copy link
Contributor

@theresalech Please triage this issue

@theresalech
Copy link
Contributor

@LitvinenkoIra thank you. We are currently working on finalizing the roadmap with the Steering Committee, so will keep this PR in consideration for upcoming Core releases.

@LitvinenkoIra LitvinenkoIra self-assigned this Dec 12, 2019
iCollin pushed a commit that referenced this issue Jan 10, 2020
…GROUND` App. (#2365)

* Bugfix `SDL forwards OnButtonEvent notification`

Bugfix `SDL forwards OnButtonEvent notification
of CUSTOM_BUTTON to BACKGROUND App.`
Issue number #967

* Check that SDL forwards OnButtonEvent notification of CUSTOM_BUTTON to BACKGROUND App

* fixup! Bugfix `SDL forwards OnButtonEvent notification`

Co-authored-by: Ira Lytvynenko (GitHub) <ILytvynenko@luxoft.com>
Co-authored-by: Igor Gapchuk (GitHub) <41586842+IGapchuk@users.noreply.github.com>
@iCollin
Copy link
Collaborator

iCollin commented Jan 10, 2020

Closed via #2365

@iCollin iCollin closed this as completed Jan 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

10 participants