Fix potential memory leak

json structure allocating code should be placed in where it is used, if
there is a condition check which may cause a return before the
before-mentioned place in the function.

Bug-AGL: SPEC-3584

(cherry-picked from f19cfe46d7172b512f06f5acf657b8d56e69a0b3)

Change-Id: I5f88c7ce0b9257b2782144548f11a0e1d7ab388a
Signed-off-by: Li Xiaoming <lixm.fnst@cn.fujitsu.com>
Signed-off-by: Marius Vlad <marius.vlad@collabora.com>

src/homescreen, hs-client, hs-proxy: Fix trivial unused warnings

As we're now using Werror, need clean-up some trivial bits.

Bug-AGL: SPEC-3843

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: I17c7e61c0fd3b420a4c76fa40fe6486d2bad0c5e

conf.d/cmake/config.cmake: Fail on warnings

Fail the compilation on warnings, so we avoid silly mistakes like found
in SPEC-3843.

Bug-AGL: SPEC-3843

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: Ib7905cdfb9a6012c4a5306255779293f3ba38778

hs-clientmanager: Pass arguments to printf() wrappers

Avoid crashing on various platforms, by not setting any argument at all.

Bug-AGL: SPEC-3843

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: Ifb082b95fd6ebc71e863b843c9fd895f5f7efdde

hs-proxy,hs-clientmanager: Handle correctly the shutdown of apps

The fake subscribe mechanism failed to account for the client context,
which is bound to the afb_req_t of the client itself, and only
dealing with the client list.  This effectively means we can't
really register (a/an automated) callback function to remove the
appid from the client list once the application has been legally
terminated/stopped.

This adds a check to verify, for the showWindow verb, if the application
is still found to be running, and return the appropriate value in case
that is not case. This should determine to start the application and fix
the issue.

Bug-AGL: SPEC-3796

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: Ia828e1ec374bf3bed21c52814721074c01f16691

hs-clientmanager: Do not store always the client context

This incorrectly assumed that by re-using the same client context we
could gain access to the *current* client's context. But as we have
seen, the fake subscribe mechanism isn't really capable of doing that,
requiring a direct connection from the client to have that in.

Storing the client context make sense to happen only when we can create
it (with a real subscribe verb) and not *re-use* it, and assign it to
other applications. This basically reverts it to the way it was
previously.

Bug-AGL: SPEC-3796

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: I643dfe91049efa8937c9a8a21eb6fbb366b3fdcc

hs-proxy: Avoid creating a client context

The subscribe mechanism requires that the client itself performs the
subscribe (it assumes a unique, distinct afb_req_t type in order to
create the client context) and attempting to handle one,
automatically would not really work, so only use it (the fake sub mechanism)
the keep track of application w/ the help of the client_list.

Bug-AGL: SPEC-3796

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: I783614e1db83280878b77b786e4f80fccd1a74a1

Fix pushAppListChangedEvent json-c object reference count error

The agl-service-homescreen has been shut down due to an error in this part when installing or uninstalling the application.

Bug-AGL: SPEC-3679
Signed-off-by: Yevgeny Hong <yev@drimaes.com>
Change-Id: I276d57ade7db96dd2a3aaa59d54a1d259c17bef7
(cherry picked from commit a06ae44353446b2684a4a43f4332ad93a43feadb)

Add gitreview file for jellyfish branch

Update the .gitreview file.

Change-Id: Ica57a7f15a54b50872d009ad8dcf629b90addd51
Bug-AGL: SPEC-3581
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

hs-proxy: Keep track of clientCtx and client when starting application

The subscribe verb was used in the past to keep track of applications,
which was done in background with the help of event handlers. As
that no longer happens we keep track of the client and its clientCtx
when starting directly from the homescreen binding, faking the subscribe
logic.

As the afb_req_t object is no longer in scope when the async callback
handler returns, we do it before doing the call to af-main, and assume
that we'll succeed.

We use the async callback handler to  verify if the application was
indeed started but any application crashing afterwards will require an
additional check performed, as it will found in the hash table of
clients.

Bug-AGL: SPEC-3524

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: I0e417dc5f2953947cd27ca551c0d2a2af5c57e6d

hs-clientmanager: Store the appidClientContext directly

Make client related methods public to be able to call them from
other callsites.

Bug-AGL: SPEC-3524

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: I7e13b83564bb6e5d49b760c27665b44d09b0ee94

hs-proxy: Hand over the hs_instance to AfmMainProxy

Bug-AGL: SPEC-3524

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: I80a7bac4244e62f49ab4bf068476ad4e888085f1

src: Split hs_instance into its own header file

We need to gain a hold of the clientManager class instance, which this
hs_instance acts as a container.

Bug-AGL: SPEC-3524

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: I3ff67fee1a652e8e85857a32352593ac9a2dcca2

config.xml.in: Remove windowmanager dependency

Bug-AGL: SPEC-3447

Signed-off-by: Marius Vlad <marius.vlad@collabora.com>
Change-Id: I958e0ffda5f3ffd491518ae8e0277ac34f6b085b

Update autobuild scripts

Update autobuild scripts with reworked version that fixes building
outside of the source tree.

Bug-AGL: SPEC-2049, SPEC-3300

Signed-off-by: Scott Murray <scott.murray@konsulko.com>
Change-Id: I646c8315b41d337ec422a73358503a11e6e58a68

Fix CMake files and non-debug crashes

Changes include:
- Reworked src/CMakeLists.txt to use app template macros.
- Removed now unnecessary linker script, symbol exports are handled
  in the app template macros.
- Set missing required entries in conf.d/cmake/config.cmake, and
  update to match latest template version in cmake-apps-module,
  as well as some naming fixes in the PROJECT* definitions.
- Move project definition to top-level CMakeLists.txt to make
  tracking conf.d/cmake/config.cmake changes easier.  This quiets
  the warning from newer versions of CMake about needing a project
  definition in the top-level CMakeLists.txt.
- Moved icon file to default location for app template.
- Added config.xml.in file to work with app template, replacing
  previous hard-coded version.
- Fixed some issues in src/hs-clientmanager.{h,cpp} to address
  crashes seen in the release widget now that it is actually built
  without debug flags, and to quiet some warnings.

Bug-AGL: SPEC-3343

Signed-off-by: Scott Murray <scott.murray@konsulko.com>
Change-Id: I3741e6a307297272d5c22e894dfa9ff8b0dd1f52

Update autobuild script to create new targets

Update autobuild script to create multiple widget target debug, coverage and all
Add config.cmake to match with the cmake-app-module
Change CMakeLists following those changes

Bug-AGL: SPEC-2049

Signed-off-by: Frederic Marec <frederic.marec@iot.bzh>
Change-Id: Iad8261dbd7e02b1cafcfd29db82015d2db5445c1

Relax computation of appid

The returned appid is now either the part up to
the first @ or the entire string if no @ is present.

This evolution is needed to able the removal
of handling different versions.

Bug-AGL: SPEC-2538

Change-Id: Ibe8cfbb60333702ba33d59d3e89f33688d2a0f51
Signed-off-by: José Bollo <jose.bollo@iot.bzh>

docs(ApplicationGuide.md): Fix typo

Bug-AGL: SPEC-2714

Change-Id: Ia2697a2becaee2d13d6d58cfb2cb04af286d69ec
Signed-off-by: Li Xiaoming <lixm.fnst@cn.fujitsu.com>

Change document

1. change contents refer to doxygen files to fit with markdown syntax.
2. remove destination in api-services-book.yml.
3. fix some issues.

Bug-AGL: SPEC-2326

Change-Id: I277326a1ef0205aa690db5398c72f74f6fc0e04a
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

Modify markdown file

1. delete html tags in markdown file
2. unify contents format
3. change supported environment to GG

Bug-AGL: SPEC-2096

Change-Id: I41f0fcc9b0aab9ace177199aadd51336aea6570b
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

Impove event process

1. manager event list in hs_instance.
2. add setEventHook interface to hook event.

Change-Id: I65a64f2d16343eb68d22fd1ad9d5fbf565f5967a
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

Change log macro to AFB_XXX

1. using binder log macro instead of HMI_LOG.
2. unify log using level as below:
  AFB_ERROR:   fatal error or serious error may occur.
  AFB_WARNING: input parameter error or not serious error.
  AFB_NOTICE:  alert sth.
  AFB_INFO:    print input argument or sth else.
  AFB_DEBUG:   prompt for calling location.
3. delete hmi-debug.h.

Change-Id: I203864ced39c418d2f792faa50ab2c009deb9d02
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

Modify project name in config.xml

Fix issue, modify project name to homescreen-service.

Change-Id: Ia578e32d1acd141e38e8dad39144c0cb0a1f5b61
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

Start app and get runnables list by homescreen

1.start application in showWindow.
2.add "getRunnables" verb.
3.handle "application-list-changed" event from afm-main and add "application-list-changed" event.

Bug-AGL: SPEC-2188

Change-Id: I619b97424d20af373a945ff502a8133339916923
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

Define func_list as static

For sharing func_list between all client instance,
define func_list as static.

Change-Id: I8a0a131128e057cc55a2d3efb76b66033dd20a04
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

autobuild: add autobuild scripts

 - Add agl autobuild script to silence build
   warnings on yocto.
 - Add linux autobuild script to build with
   the sdk
 - These  scripts are based on the latest
   version of the autobuild script, but they've
   been customized to match the existin cmake
   (the update and package-test targets have been
   removed).

Change-Id: I6f8bd89ee0dfe770557e9a9b6d7729f30735ce43
Signed-off-by: Raquel Medina <raquel.medina@konsulko.com>

Improve homescreen-service

1.Improve class cohesion and reduce public interface number
2.Unified event pattern

BUG-AGL: SPEC-2120

Change-Id: I4c5066be2deaf2b809af66cb5a8acdc5bffa5a23
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

docs: add yaml book

Bug-AGL: SPEC-1988

Change-Id: I877c8935449095f3dd8f531282f009538cfc7175
Signed-off-by: Frederic Marec <frederic.marec@iot.bzh>

fix issue,not free string

After called afb_req_get_application_id api needed to
free the string.

Change-Id: Ia59a3f4984854d61f4c5a8d9206f44988594ebf3
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

update doc

add new sequence and update doxygen files.

Change-Id: I6737939eb2628577d58b2d1d65086e46e7cb14a4
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

change project name

change homescreen-service-2017 to homescreen-service.

Change-Id: I440864ec5dc940bbbaf34aa09c6c1da4b8b9be57
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

Improvement:a client instance only call afb_req_subscribe once

when call subscribe every time, hs_client instance will call afb_req_subscribe once,
this is needless, every hs_client always only has one event, so only need to call afb_req_subscribe once.

Change-Id: I91417b749201fc378287e03f08dfd4dfd5df9dc7
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

add new features in homescreen-service and homescreen

homescreen-service: add five verbs.
1.showWindow: instead of tap_shortcut and show onscreen.
2.hideWindow: used when want to hide onscreen.
3.replyShowWindow: used when post onscreen reply information to application.
4.showNotification: used by application who want to display notification on homescreen top area.
5.showInformation: used by application who want to display information on homescreen botton area.

homescreen:
1.add fullscreen transfer button.
2.display notification and information.

Bug-AGL: SPEC-1931

Change-Id: I612e541243ee6502eb90ff1aa2ab4d99bfbc7156
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

use appid instead of appname in "tap_shortcut"

Now In homescreen-service used application_id to identify
different application, so use appid instead of appname
in "tap_shortcut" parameter.

Bug-AGL: SPEC-1764

Change-Id: I8493331f37977a30d6d88c0d222da4c2db32e727
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

Improve HS_ClientManager and fix issue

1.No longer operate HS_Client in homescreen.cpp,move to HS_ClientManager class.
2.In hs_client.cpp,afb_event_push return the count of clients that received the event,
  not push event result, in tap_shortcut/on_screen_message/on_screen_reply directly
  return zero.

Bug-AGL: SPEC-1764

Change-Id: Ief17d3fe3a9c80937fbda05d1c158eeed3924a75
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

Migration to binding V3

migration homescreen-service from v2 to v3

Change-Id: I5e6d42c3dff528e46d0ca407e09eb1d05bacea80
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

emit event to one application

changed agl-service-homescreen and libhomescreen to make
agl-service-homescreen emit event to one application only.

BUG-AGL: SPEC-1764

Change-Id: I87e4fc8fe900fdf9d7fde04852077c7174b8a3ba
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

Use feature 'required-binding'

The feature "urn:AGL:widget:required-binding" is now
preferred to the feature "urn:AGL:widget:required-api"
for requiring a local binding.

Bug-AGL: SPEC-1800

Change-Id: Ic8ba6b048e6ba7b573dac2f2f1833a4df8eaa0dc
Signed-off-by: Jose Bollo <jose.bollo@iot.bzh>

Init afb_daemon event with loop

use loop to make afb_daemon event automatically.

Change-Id: I599d333fea7feb18db91ed3bbe43faad405c41c1
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

Add some comments for hs-helper

    Add some comments.

Change-Id: I08e4f2050f5107d538a4622dc5462d11bbd5078d
Signed-off-by: zheng_wenlong <wenlong_zheng@nexty-ele.com>

Update lastest code from agl-service-homescreen-2017

    Update the lastest code from agl-service-homescreen-2017.
    To impove homescreen-service's extensibility.
    "Change homescreen-service from c project to c++ project"

Relative Commit: https://gerrit.automotivelinux.org/gerrit/#/c/16351/
Change-Id: Id7e50fa61fdfcff30f69588aa1da4d93695edfc1
Signed-off-by: zheng_wenlong <wenlong_zheng@nexty-ele.com>
Signed-off-by: wang_zhiqiang <wang_zhiqiang@dl.cn.nexty-ele.com>

Change gitreview to new location

Change-Id: If2690f0524871dff14f8409e9cdd011408f83428
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Revert "Use appid between homescreen-service and apps"

This reverts commit b5702d25b7b5386fabd6a81a748ea2d6fe647fcd.

Bug-AGL: SPEC-1685

Change-Id: If5b526a5a5406ff0c3e59cc4a578b977386bda14
Signed-off-by: Tadao Tanikawa <tanikawa.tadao@jp.panasonic.com>

Use appid between homescreen-service and apps

    Use appid between hss and apps, and check event destination in libhomescreen.
    So these is no need compare code when recived Event_TapShortcut Event.

BUG-AGL: SPEC-1645

Change-Id: I3609a157ffdf31aa19cad6ae0f809f52e2129d0d
Signed-off-by: zheng_wenlong <wenlong_zheng@nexty-ele.com>

Set default debug level to error

    Default enable HMI_ERROR output.
    Remove afb-binding.h and add necessary header files.

    [Patch Set 2]
    Change LOG_LEVEL_MAX value equal LOG_LEVEL_DEBUG.

Change-Id: Ie50e267b9ff9ad107bb38162ad90acc410c79cbc
Signed-off-by: zheng_wenlong <wenlong_zheng@nexty-ele.com>

Add a new OnScreenReply api to agl-service-homescreen

    Add a new event named OnScreenReply. Applications
    can use this event to reply to the homescreen.
    Also update the documentation for this event.

    [Patch Sets 8,9]
    Update ApplicationGuide.md. For details:
        Use master source build agl-service-homescreen-2017.
        Change agl version to EE.
        Add libhomescreen sample code path.

    [Patch Sets 10]
    Delete tailing space.

BUG-AGL: SPEC-985
Change-Id: Iea0c321731fa62cd94fc08fc5e303c20847fd525
Signed-off-by: zheng_wenlong <wenlong_zheng@nexty-ele.com>

Move sample to libhomescreen

    The sample folder is using libhomescren with
    agl-service-homescreen-2017, So better move to libhomescreen.

BUG-AGL: SPEC-988
Change-Id: I165bd3baa67ae2909f5ac699faf4f1f620f78747
Signed-off-by: zheng_wenlong <wenlong_zheng@nexty-ele.com>

Add debug message macros controlled by environment variable

    Add a HMI_DEBUG macro to print debug messages.
    It is controlled by the USE_HMI_DEBUG environment variable.

BUG-AGL: SPEC-998
Change-Id: Ibb0a5c119a3a627538e7ec2728c64007f21af215
Signed-off-by: zheng_wenlong <wenlong_zheng@nexty-ele.com>

Delete libhomescreen from this gerrit repository

    Relocate libhomescreen into its own repository as src/libhomescreen.

BUG-AGL: SPEC-993
Change-Id: I481c9b2322a1a112f23facdd930cb36e594919bb
Signed-off-by: zheng_wenlong <wenlong_zheng@nexty-ele.com>

Modify the library name in CmakeLists.txt

    A wrong name is used in CMakeLists.txt (SM = soundmanager
    instead of HS = homescreen). Fix CMakeLists.txt by
    replacing SM with HS.

BUG-AGL: SPEC-995
Change-Id: Id6273295982106705da0d63be89f50c05f4d6567
Signed-off-by: zheng_wenlong <wenlong_zheng@nexty-ele.com>

Change comments from Japanese to English

    There are some Japanese comments in RC1,
    change these to English.

    [Patch Sets 2]
    Split CMakeLists.txt due to modification in another commit.

    [Patch Sets 3]
    Update commit message.

    [Patch Sets 4]
    Return to Patch Sets 1 to rebase for this commit.
    <Delete libhomescreen from this gerrit repository>

    [Patch Sets 5]
    Return to Patch Sets 3 to rebase for this commit.
    <Modify library name in CMakeLists.txt>

Change-Id: Ib520d39ede23cb0059f303c6e38090c9ed65a45b
Signed-off-by: zheng_wenlong <wenlong_zheng@nexty-ele.com>

Add gitreview file for master

Change-Id: I91a3830de36d61f46303274f39bfe2ace528a69d
Signed-off-by: Jan-Simon Möller <jsmoeller@linuxfoundation.org>

Add agl-service-homescreen-2017

    Add a new binding agl-service-homescreen-2017 for homescreen-2017.
    There is a gles sample how to use this binding in sample/simple-egl.
    A image about this see JIRA SPEC-871.

    [PatchSets2]
    Use aglwgt make package.

Change-Id: I9ae89188f9eb2de4c98ec8baa2f05fccca8b2372
Signed-off-by: zheng_wenlong <wenlong_zheng@nexty-ele.com>

Initial empty repository