ivi-shell: ivi-shell.c, Set NULL to resource of ivisurface when shell_destroy_shell_surface called

> /* FIXME: Need to set ivisurf->resource = NULL. */

Signed-off-by: Nobuhiko Tanibata <NOBUHIKO_TANIBATA@xddp.denso.co.jp>
ivi-shell: ivi-shell.c, Remove unnecessary check for weston_surface of application_surface_create()

> FIXME: This is not checked in desktop_shell, required to check?
> - Not required to check this, because libwayland-server will raise
> a protocol error automatically, if the passed in object is null
> but the protocol XML does not specify allow-null="true" for the argument.

Signed-off-by: Nobuhiko Tanibata <NOBUHIKO_TANIBATA@xddp.denso.co.jp>
ivi-shell: ivi-shell.c, Add NULL check of ivisurf->resource

> FIXME: shell_surf->resource might be NULL,
> if controller decides to reconfigure a surface after a client
> has destroyed it.

Signed-off-by: Nobuhiko Tanibata <NOBUHIKO_TANIBATA@xddp.denso.co.jp>
ivi-shell: ivi-shell.c, Move down function call for wl_resource creation

> FIXME: this shall be done below:TAG_1, if I use the same order of
> calling in  create_common_surface
> FIXME: not required if move res to below:TAG_1
> FIXME: Move below:TAG_1

Signed-off-by: Nobuhiko Tanibata <NOBUHIKO_TANIBATA@xddp.denso.co.jp>
ivi-shell: ivi-shell.c,Move ivi_layout_interface into struct ivi_shell

> FIXME: move this static variable into struct ivi_shell.

Signed-off-by: Nobuhiko Tanibata <NOBUHIKO_TANIBATA@xddp.denso.co.jp>
hmi-controller.c: indent with tabs

For the stray changes, see 'git show -b <sha1>'.
hmi-controller.c: remove unneeded braces from if-statements
hmi-controller.c: whitespace fixes
hmi-controller.c: remove unneeded casts
hmi-controller.c: split long lines
clients/ivi-shell-user-interface.c: whitespace fixes
clients/ivi-shell-user-interface.c: split long lines

hmi_homescreen_setting_create() lost one level of ident through a little
refactoring.
clients/ivi-shell-user-interface.c: remove unnecessary braces