[EFL] Use EINA_LOG instead of fprintf
authorgyuyoung.kim@samsung.com <gyuyoung.kim@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 23 Feb 2013 04:14:00 +0000 (04:14 +0000)
committergyuyoung.kim@samsung.com <gyuyoung.kim@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 23 Feb 2013 04:14:00 +0000 (04:14 +0000)
https://bugs.webkit.org/show_bug.cgi?id=110313

Reviewed by Laszlo Gombos.

Replace fprintf with EINA_LOG.

Source/WebCore:

* platform/efl/NetworkInfoProviderEfl.cpp:
(WebCore::NetworkInfoProviderEfl::startUpdating):
* platform/graphics/efl/CairoUtilitiesEfl.cpp:
(WebCore::evasObjectFromCairoImageSurface):

Source/WebKit/efl:

* ewk/ewk_frame.cpp:
(_ewk_frame_debug):

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

Source/WebCore/ChangeLog
Source/WebCore/platform/efl/NetworkInfoProviderEfl.cpp
Source/WebCore/platform/graphics/efl/CairoUtilitiesEfl.cpp
Source/WebKit/efl/ChangeLog
Source/WebKit/efl/ewk/ewk_frame.cpp

index 79421b9..cd41b61 100644 (file)
@@ -1,3 +1,17 @@
+2013-02-22  Gyuyoung Kim  <gyuyoung.kim@samsung.com>
+
+        [EFL] Use EINA_LOG instead of fprintf 
+        https://bugs.webkit.org/show_bug.cgi?id=110313
+
+        Reviewed by Laszlo Gombos.
+
+        Replace fprintf with EINA_LOG.
+
+        * platform/efl/NetworkInfoProviderEfl.cpp:
+        (WebCore::NetworkInfoProviderEfl::startUpdating):
+        * platform/graphics/efl/CairoUtilitiesEfl.cpp:
+        (WebCore::evasObjectFromCairoImageSurface):
+
 2013-02-22  Tony Gentilcore  <tonyg@chromium.org>
 
         Threaded HTML parser fails resources/plain-text-unsafe.dat
index e51e967..eb830de 100644 (file)
@@ -44,7 +44,7 @@ NetworkInfoProviderEfl::NetworkInfoProviderEfl()
 void NetworkInfoProviderEfl::startUpdating()
 {
     if (!eeze_init())
-        fprintf(stderr, "Fail to start network information client.\n");
+        EINA_LOG_ERR("Fail to start network information client.");
 }
 
 void NetworkInfoProviderEfl::stopUpdating()
index aa4d749..679f59e 100644 (file)
@@ -32,20 +32,20 @@ PassRefPtr<Evas_Object> evasObjectFromCairoImageSurface(Evas* canvas, cairo_surf
 
     cairo_status_t status = cairo_surface_status(surface);
     if (status != CAIRO_STATUS_SUCCESS) {
-        fprintf(stderr, "cairo surface is invalid: %s", cairo_status_to_string(status));
+        EINA_LOG_ERR("cairo surface is invalid: %s", cairo_status_to_string(status));
         return 0;
     }
 
     cairo_surface_type_t type = cairo_surface_get_type(surface);
     if (type != CAIRO_SURFACE_TYPE_IMAGE) {
-        fprintf(stderr, "unknown surface type %d, required %d (CAIRO_SURFACE_TYPE_IMAGE).",
+        EINA_LOG_ERR("unknown surface type %d, required %d (CAIRO_SURFACE_TYPE_IMAGE).",
             type, CAIRO_SURFACE_TYPE_IMAGE);
         return 0;
     }
 
     cairo_format_t format = cairo_image_surface_get_format(surface);
     if (format != CAIRO_FORMAT_ARGB32 && format != CAIRO_FORMAT_RGB24) {
-        fprintf(stderr, "unknown surface format %d, expected %d or %d.",
+        EINA_LOG_ERR("unknown surface format %d, expected %d or %d.",
             format, CAIRO_FORMAT_ARGB32, CAIRO_FORMAT_RGB24);
         return 0;
     }
@@ -54,19 +54,19 @@ PassRefPtr<Evas_Object> evasObjectFromCairoImageSurface(Evas* canvas, cairo_surf
     int height = cairo_image_surface_get_height(surface);
     int stride = cairo_image_surface_get_stride(surface);
     if (width <= 0 || height <= 0 || stride <= 0) {
-        fprintf(stderr, "invalid image size %dx%d, stride=%d", width, height, stride);
+        EINA_LOG_ERR("invalid image size %dx%d, stride=%d", width, height, stride);
         return 0;
     }
 
     void* data = cairo_image_surface_get_data(surface);
     if (!data) {
-        fprintf(stderr, "could not get source data.");
+        EINA_LOG_ERR("could not get source data.");
         return 0;
     }
 
     RefPtr<Evas_Object> image = adoptRef(evas_object_image_filled_add(canvas));
     if (!image) {
-        fprintf(stderr, "could not add image to canvas.");
+        EINA_LOG_ERR("could not add image to canvas.");
         return 0;
     }
 
@@ -75,7 +75,7 @@ PassRefPtr<Evas_Object> evasObjectFromCairoImageSurface(Evas* canvas, cairo_surf
     evas_object_image_alpha_set(image.get(), format == CAIRO_FORMAT_ARGB32);
 
     if (evas_object_image_stride_get(image.get()) != stride) {
-        fprintf(stderr, "evas' stride %d diverges from cairo's %d.",
+        EINA_LOG_ERR("evas' stride %d diverges from cairo's %d.",
             evas_object_image_stride_get(image.get()), stride);
         return 0;
     }
index 19d7df9..6521119 100644 (file)
@@ -1,3 +1,15 @@
+2013-02-22  Gyuyoung Kim  <gyuyoung.kim@samsung.com>
+
+        [EFL] Use EINA_LOG instead of fprintf 
+        https://bugs.webkit.org/show_bug.cgi?id=110313
+
+        Reviewed by Laszlo Gombos.
+
+        Replace fprintf with EINA_LOG.
+
+        * ewk/ewk_frame.cpp:
+        (_ewk_frame_debug):
+
 2013-02-19  Tony Chang  <tony@chromium.org>
 
         Convert 3 settings to use Settings.in
index 1237346..1d94de6 100644 (file)
@@ -135,13 +135,13 @@ static inline void _ewk_frame_debug(Evas_Object* ewkFrame)
     evas_object_color_get(clip, &contentRed, &contentGreen, &contentBlue, &contentAlpha);
     evas_object_geometry_get(clip, &contentX, &contentY, &contentWidth, &contentHeight);
 
-    fprintf(stderr, "%p: type=%s name=%s, visible=%d, color=%02x%02x%02x%02x, %d,%d+%dx%d, clipper=%p (%d, %02x%02x%02x%02x, %d,%d+%dx%d)\n",
+    EINA_LOG_DBG("%p: type=%s name=%s, visible=%d, color=%02x%02x%02x%02x, %d,%d+%dx%d, clipper=%p (%d, %02x%02x%02x%02x, %d,%d+%dx%d)\n",
             ewkFrame, evas_object_type_get(ewkFrame), evas_object_name_get(ewkFrame), evas_object_visible_get(ewkFrame),
             red, green, blue, alpha, x, y, width, height,
             clip, evas_object_visible_get(clip), contentRed, contentGreen, contentBlue, contentAlpha, contentX, contentY, contentWidth, contentHeight);
     parent = evas_object_smart_parent_get(ewkFrame);
     if (!parent)
-        fprintf(stderr, "\n");
+        EINA_LOG_ERR("could not get parent object.\n");
     else
         _ewk_frame_debug(parent);
 }