e_client: Ensure `e_client_view_get` never fails

The following pattern has been widely used:

`e_view_client_view_get(e_client_view_get(ec));`

If the given parameter to `e_view_client_view_get` is null, it will
abort.
To maintain the above pattern, the inner call as a parameter of
`e_view_client_view` must not return null.
This patch ensures that `e_client_view_get` never returns null.

Change-Id: Ie5aa7a7adc3bef2311cbf40cf15b8fa3f84c43e7

diff --git a/src/bin/core/e_client.c b/src/bin/core/e_client.c
index c9726be99abf6c699b04dc77008533f2c62b0c71..890920f4a74d52ee9734287e4c6d77bbe4e94831 100644 (file)
--- a/src/bin/core/e_client.c
+++ b/src/bin/core/e_client.c
@@ -5927,9 +5927,6 @@ e_client_base_output_resolution_desk_useful_geometry_get(E_Client *ec, int *x, i
 E_API E_View_Client *
 e_client_view_get(E_Client *ec)
 {
-   E_OBJECT_CHECK_RETURN(ec, NULL);
-   EINA_SAFETY_ON_NULL_RETURN_VAL(PRI(ec)->view_client, NULL);
-
    return PRI(ec)->view_client;
 }