[EFL] Change data type of parameter in ewk_view_viewport_attributes_get().
authorgyuyoung.kim@samsung.com <gyuyoung.kim@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 7 Oct 2011 02:23:03 +0000 (02:23 +0000)
committergyuyoung.kim@samsung.com <gyuyoung.kim@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 7 Oct 2011 02:23:03 +0000 (02:23 +0000)
https://bugs.webkit.org/show_bug.cgi?id=69505

Reviewed by Andreas Kling.

Source/WebKit/efl:

ewk_view_viewport_attributes_get() has used implicit casting for width and height of viewport.
However, it is better to use *int* data type as parameter data type instead of *float* casting.

* ewk/ewk_view.cpp:
(ewk_view_viewport_attributes_get):
* ewk/ewk_view.h:

Tools:

* EWebLauncher/main.c: Use *int* variables for width and height of viewport meta tag.
(on_viewport_changed):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@96890 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit/efl/ChangeLog
Source/WebKit/efl/ewk/ewk_view.cpp
Source/WebKit/efl/ewk/ewk_view.h
Tools/ChangeLog
Tools/EWebLauncher/main.c

index 9970f60..bb85741 100755 (executable)
@@ -1,3 +1,17 @@
+2011-10-06  Gyuyoung Kim  <gyuyoung.kim@samsung.com>
+
+        [EFL] Change data type of parameter in ewk_view_viewport_attributes_get().
+        https://bugs.webkit.org/show_bug.cgi?id=69505
+
+        Reviewed by Andreas Kling.
+
+        ewk_view_viewport_attributes_get() has used implicit casting for width and height of viewport.
+        However, it is better to use *int* data type as parameter data type instead of *float* casting.
+
+        * ewk/ewk_view.cpp:
+        (ewk_view_viewport_attributes_get):
+        * ewk/ewk_view.h:
+
 2011-10-04  Gyuyoung Kim  <gyuyoung.kim@samsung.com>
 
         [EFL] Only set when the custom encoding is different from existing value
index acc75a4..de8d8d3 100644 (file)
@@ -3456,7 +3456,7 @@ void ewk_view_viewport_attributes_set(Evas_Object* o, const WebCore::ViewportArg
     evas_object_smart_callback_call(o, "viewport,changed", 0);
 }
 
-void ewk_view_viewport_attributes_get(const Evas_Object* o, float* w, float* h, float* init_scale, float* max_scale, float* min_scale, float* device_pixel_ratio, Eina_Bool* user_scalable)
+void ewk_view_viewport_attributes_get(const Evas_Object* o, int* w, int* h, float* init_scale, float* max_scale, float* min_scale, float* device_pixel_ratio, Eina_Bool* user_scalable)
 {
     WebCore::ViewportAttributes attributes = _ewk_view_viewport_attributes_compute(o);
 
index 27d6a6b..ac55dc6 100644 (file)
@@ -2062,7 +2062,7 @@ EAPI Eina_Bool ewk_view_paint_contents(Ewk_View_Private_Data *priv, cairo_t *cr,
  * @param device_pixel_ratio the pointer to store the device pixel ratio value
  * @param user_scalable the pointer to store if user can scale viewport
  */
-EAPI void ewk_view_viewport_attributes_get(const Evas_Object *o, float *w, float *h, float *init_scale, float *max_scale, float *min_scale, float *device_pixel_ratio , Eina_Bool *user_scalable);
+EAPI void ewk_view_viewport_attributes_get(const Evas_Object *o, int *w, int *h, float *init_scale, float *max_scale, float *min_scale, float *device_pixel_ratio , Eina_Bool *user_scalable);
 
 /**
  * Sets the zoom range.
index 7c6c2cc..2d54488 100644 (file)
@@ -1,3 +1,13 @@
+2011-10-06  Gyuyoung Kim  <gyuyoung.kim@samsung.com>
+
+        [EFL] Change data type of parameter in ewk_view_viewport_attributes_get().
+        https://bugs.webkit.org/show_bug.cgi?id=69505
+
+        Reviewed by Andreas Kling.
+
+        * EWebLauncher/main.c: Use *int* variables for width and height of viewport meta tag.
+        (on_viewport_changed):
+
 2011-10-06  Brent Fulgham  <bfulgham@webkit.org>
 
         [WinCairo] Unreviewed test correction.
index 6ae9f3d..413f146 100644 (file)
@@ -440,7 +440,8 @@ on_viewport_changed(void* user_data, Evas_Object* webview, void* event_info)
 {
     ELauncher *app = (ELauncher *)user_data;
 
-    float w, h, initScale, minScale, maxScale, devicePixelRatio;
+    int w, h;
+    float initScale, minScale, maxScale, devicePixelRatio;
     Eina_Bool userScalable;
 
     ewk_view_viewport_attributes_get(webview, &w, &h, &initScale, &maxScale, &minScale, &devicePixelRatio, &userScalable);
@@ -448,9 +449,9 @@ on_viewport_changed(void* user_data, Evas_Object* webview, void* event_info)
     /**
      * If there is no argument in viewport tag, argument's value is -1.
      */
-    if ((int)w == -1)
+    if (w == -1)
         w = DEFAULT_WIDTH;
-    if ((int)h == -1)
+    if (h == -1)
         h = DEFAULT_HEIGHT;
     if ((int)initScale == -1)
         initScale = DEFAULT_ZOOM_INIT; // There's no scale separated from zooming in webkit-efl.
@@ -463,8 +464,8 @@ on_viewport_changed(void* user_data, Evas_Object* webview, void* event_info)
     if ((int)userScalable == -1)
         userScalable = EINA_TRUE;
 
-    app->viewport.w = (int)w;
-    app->viewport.h = (int)h;
+    app->viewport.w = w;
+    app->viewport.h = h;
     app->viewport.initScale = initScale;
     app->viewport.minScale = minScale;
     app->viewport.maxScale = maxScale;