ivi-shell: Get hmi_controller from resource when before setup time.

> Every one of these calls commit. Wouldn't it be better to call commit
> only once in the end of this function?
> [ntanibata] Exactly, I will fix it.
> It doesn't make too much sense to pass 'resource' as the argument to
> these function calls. Would be better to pass in struct
> hmi_controller, because that is what every single function needs and
> not the wl_resource.
> [ntanibata] I agree. I will fix it.
> ivi_hmi_controller_UI_ready() is the only function that is actually a
> request handler.
> [ntanibata] Right. I will fix it.

Signed-off-by: Nobuhiko Tanibata <NOBUHIKO_TANIBATA@xddp.denso.co.jp>
clients: Change SIGTERM form SIGKILL

> I cannot say, that 'volatile' in this case is wrong. It might even be necessary,
> since you are changing it from a signal handler, which I probably overlooked.
> Another mixup I might had is that this volatile is accessed "directly"
> rather than through a pointer. There was something about variables accessed
> through a pointer behaving slightly differently...
> Anyway, since it's modified from a signal handler, keep the 'volatile'.
> I likely have my own misconceptions about 'volatile'.
> Btw. you cannot catch SIGKILL. Do you need to catch any signals? This code
> makes the program exit with EXIT_SUCCESS even if killed by SIGINT.

Signed-off-by: Nobuhiko Tanibata <NOBUHIKO_TANIBATA@xddp.denso.co.jp>
ivi-shell: Modification to end of function

> Every one of these calls commit. Wouldn't it be better to call commit
> only once in the end of this function?
> [ntanibata] Exactly, I will fix it.

Signed-off-by: Nobuhiko Tanibata <NOBUHIKO_TANIBATA@xddp.denso.co.jp>
ivi-shell: Use error-code defined in ivi-application-protocol.

> > You also need to add the two error codes into ivi-application
> > interface and use those instead of WL_DISPLAY_ERROR_INVALID_OBJECT.
> [ntanibata] I will fix this according to two error codes.
> <enum name="error">
>       <entry name="role" value="0" summary="given wl_surface has
>       another role"/>
>       <entry name="ivi_id" value="1" summary="given ivi_id is assigned
>       to another wl_surface"/>
> </enum>

Signed-off-by: Nobuhiko Tanibata <NOBUHIKO_TANIBATA@xddp.denso.co.jp>