From 161da40b068cdf7d9daca0762f0180789bfe3ec5 Mon Sep 17 00:00:00 2001 From: "Ucan, Emre \\(ADITG/SW1\\)" Date: Fri, 4 Mar 2016 12:50:43 +0000 Subject: [PATCH] ivi-shell: remove ivi_layout_surface_set_position API Signed-off-by: Emre Ucan Acked-by: wataru_natsume Reviewed-by: Bryce Harrington --- ivi-shell/ivi-layout-export.h | 9 --------- ivi-shell/ivi-layout.c | 24 ------------------------ tests/ivi_layout-internal-test.c | 11 ----------- tests/ivi_layout-test-plugin.c | 16 +++------------- tests/ivi_layout-test.c | 1 - 5 files changed, 3 insertions(+), 58 deletions(-) diff --git a/ivi-shell/ivi-layout-export.h b/ivi-shell/ivi-layout-export.h index 86315f69..5e5b305f 100644 --- a/ivi-shell/ivi-layout-export.h +++ b/ivi-shell/ivi-layout-export.h @@ -311,15 +311,6 @@ struct ivi_layout_interface { int32_t x, int32_t y, int32_t width, int32_t height); - /** - * \brief Sets the horizontal and vertical position of the surface. - * - * \return IVI_SUCCEEDED if the method call was successful - * \return IVI_FAILED if the method call was failed - */ - int32_t (*surface_set_position)(struct ivi_layout_surface *ivisurf, - int32_t dest_x, int32_t dest_y); - /** * \brief Set the horizontal and vertical dimension of the surface. * diff --git a/ivi-shell/ivi-layout.c b/ivi-shell/ivi-layout.c index 74f09484..e60ee6d7 100644 --- a/ivi-shell/ivi-layout.c +++ b/ivi-shell/ivi-layout.c @@ -2076,29 +2076,6 @@ ivi_layout_surface_set_dimension(struct ivi_layout_surface *ivisurf, return IVI_SUCCEEDED; } -static int32_t -ivi_layout_surface_set_position(struct ivi_layout_surface *ivisurf, - int32_t dest_x, int32_t dest_y) -{ - struct ivi_layout_surface_properties *prop = NULL; - - if (ivisurf == NULL) { - weston_log("ivi_layout_surface_set_position: invalid argument\n"); - return IVI_FAILED; - } - - prop = &ivisurf->pending.prop; - prop->dest_x = dest_x; - prop->dest_y = dest_y; - - if (ivisurf->prop.dest_x != dest_x || ivisurf->prop.dest_y != dest_y) - ivisurf->event_mask |= IVI_NOTIFICATION_POSITION; - else - ivisurf->event_mask &= ~IVI_NOTIFICATION_POSITION; - - return IVI_SUCCEEDED; -} - static int32_t ivi_layout_surface_set_orientation(struct ivi_layout_surface *ivisurf, enum wl_output_transform orientation) @@ -2622,7 +2599,6 @@ static struct ivi_layout_interface ivi_layout_interface = { .surface_set_opacity = ivi_layout_surface_set_opacity, .surface_set_source_rectangle = ivi_layout_surface_set_source_rectangle, .surface_set_destination_rectangle = ivi_layout_surface_set_destination_rectangle, - .surface_set_position = ivi_layout_surface_set_position, .surface_set_dimension = ivi_layout_surface_set_dimension, .surface_set_orientation = ivi_layout_surface_set_orientation, .surface_set_content_observer = ivi_layout_surface_set_content_observer, diff --git a/tests/ivi_layout-internal-test.c b/tests/ivi_layout-internal-test.c index 7ec29482..2c924e53 100644 --- a/tests/ivi_layout-internal-test.c +++ b/tests/ivi_layout-internal-test.c @@ -100,16 +100,6 @@ test_surface_bad_dimension(struct test_context *ctx) lyt->commit_changes(); } -static void -test_surface_bad_position(struct test_context *ctx) -{ - const struct ivi_layout_interface *lyt = ctx->layout_interface; - - iassert(lyt->surface_set_position(NULL, 20, 30) == IVI_FAILED); - - lyt->commit_changes(); -} - static void test_surface_bad_source_rectangle(struct test_context *ctx) { @@ -1041,7 +1031,6 @@ run_internal_tests(void *data) test_surface_bad_destination_rectangle(ctx); test_surface_bad_orientation(ctx); test_surface_bad_dimension(ctx); - test_surface_bad_position(ctx); test_surface_bad_source_rectangle(ctx); test_surface_bad_properties(ctx); diff --git a/tests/ivi_layout-test-plugin.c b/tests/ivi_layout-test-plugin.c index 8097563c..00b1e87d 100644 --- a/tests/ivi_layout-test-plugin.c +++ b/tests/ivi_layout-test-plugin.c @@ -458,8 +458,9 @@ RUNNER_TEST(surface_position) runner_assert(prop->dest_x == 0); runner_assert(prop->dest_y == 0); - runner_assert(lyt->surface_set_position( - ivisurf, 20, 30) == IVI_SUCCEEDED); + runner_assert(lyt->surface_set_destination_rectangle( + ivisurf, 20, 30, + prop->dest_width, prop->dest_height) == IVI_SUCCEEDED); runner_assert(prop->dest_x == 0); runner_assert(prop->dest_y == 0); @@ -631,17 +632,6 @@ RUNNER_TEST(commit_changes_after_dimension_set_surface_destroy) ivisurf, 200, 300) == IVI_SUCCEEDED); } -RUNNER_TEST(commit_changes_after_position_set_surface_destroy) -{ - const struct ivi_layout_interface *lyt = ctx->layout_interface; - struct ivi_layout_surface *ivisurf; - - ivisurf = lyt->get_surface_from_id(IVI_TEST_SURFACE_ID(0)); - runner_assert(ivisurf != NULL); - runner_assert(lyt->surface_set_position( - ivisurf, 20, 30) == IVI_SUCCEEDED); -} - RUNNER_TEST(commit_changes_after_source_rectangle_set_surface_destroy) { const struct ivi_layout_interface *lyt = ctx->layout_interface; diff --git a/tests/ivi_layout-test.c b/tests/ivi_layout-test.c index 6c739dae..196ef8fb 100644 --- a/tests/ivi_layout-test.c +++ b/tests/ivi_layout-test.c @@ -205,7 +205,6 @@ const char * const surface_property_commit_changes_test_names[] = { "commit_changes_after_opacity_set_surface_destroy", "commit_changes_after_orientation_set_surface_destroy", "commit_changes_after_dimension_set_surface_destroy", - "commit_changes_after_position_set_surface_destroy", "commit_changes_after_source_rectangle_set_surface_destroy", "commit_changes_after_destination_rectangle_set_surface_destroy", };