ivi-shell: don't check view is rendered before adding surface

If a surface is removed from a layer's render order list, but the
changes are not commited. Then, the surface could not be added to the
same layer. Because ivi_view of the surface is still in the render order
list of the layer.

It is not possible to change the order of surfaces in a layer without
clearing and commiting the changes before. This patch fixes this issue.

After this patch, the pending render order list of a layer is always
modified regardless of its active render order list.

Signed-off-by: Emre Ucan <eucan@de.adit-jv.com>
Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>

ivi-shell/ivi-layout.c

@@ -1773,8 +1773,6 @@ ivi_layout_layer_add_surface(struct ivi_layout_layer *ivilayer,
 	ivi_view = get_ivi_view(ivilayer, addsurf);
 	if (!ivi_view)
 		ivi_view = ivi_view_create(ivilayer, addsurf);
-	else if (ivi_view_is_rendered(ivi_view))
-		return IVI_SUCCEEDED;
 
 	wl_list_remove(&ivi_view->pending_link);
 	wl_list_insert(&ivilayer->pending.view_list, &ivi_view->pending_link);