[EFL][WK2] test_ewk2_view is failed
authorryuan.choi@gmail.com <ryuan.choi@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 2 Oct 2014 11:14:43 +0000 (11:14 +0000)
committerryuan.choi@gmail.com <ryuan.choi@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 2 Oct 2014 11:14:43 +0000 (11:14 +0000)
https://bugs.webkit.org/show_bug.cgi?id=132983

Reviewed by Gyuyoung Kim.

Enable test_ewk2_view because test cases of test_ewk2_view are passed except
for few tests related to device_pixel_ratio.
This patch adds macro guard for ailed test case to CAIRO_SURFACE_SET_DEVICE_SCALE macro.
In addition, fixed some warnings while building with test_ewk2_view.

* PlatformEfl.cmake: Enables test_ewk2_view.
* UIProcess/API/efl/tests/UnitTestUtils/EWK2UnitTestBase.cpp: Added const keyword for argument to fix the warnings.
(EWK2UnitTest::EWK2UnitTestBase::keyDown):
(EWK2UnitTest::EWK2UnitTestBase::keyUp):
* UIProcess/API/efl/tests/UnitTestUtils/EWK2UnitTestBase.h:
* UIProcess/API/efl/tests/test_ewk2_view.cpp:
(TEST_F):

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

Source/WebKit2/ChangeLog
Source/WebKit2/PlatformEfl.cmake
Source/WebKit2/UIProcess/API/efl/tests/UnitTestUtils/EWK2UnitTestBase.cpp
Source/WebKit2/UIProcess/API/efl/tests/UnitTestUtils/EWK2UnitTestBase.h
Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_view.cpp

index 282fc50..fca9c20 100644 (file)
@@ -1,3 +1,23 @@
+2014-10-02  Ryuan Choi  <ryuan.choi@gmail.com>
+
+        [EFL][WK2] test_ewk2_view is failed
+        https://bugs.webkit.org/show_bug.cgi?id=132983
+
+        Reviewed by Gyuyoung Kim.
+
+        Enable test_ewk2_view because test cases of test_ewk2_view are passed except
+        for few tests related to device_pixel_ratio.
+        This patch adds macro guard for ailed test case to CAIRO_SURFACE_SET_DEVICE_SCALE macro.
+        In addition, fixed some warnings while building with test_ewk2_view.
+
+        * PlatformEfl.cmake: Enables test_ewk2_view.
+        * UIProcess/API/efl/tests/UnitTestUtils/EWK2UnitTestBase.cpp: Added const keyword for argument to fix the warnings.
+        (EWK2UnitTest::EWK2UnitTestBase::keyDown):
+        (EWK2UnitTest::EWK2UnitTestBase::keyUp):
+        * UIProcess/API/efl/tests/UnitTestUtils/EWK2UnitTestBase.h:
+        * UIProcess/API/efl/tests/test_ewk2_view.cpp:
+        (TEST_F):
+
 2014-10-02  JinYoung Hur  <hurims@gmail.com>
 
         Minor bug in UIClient::exceededDatabaseQuota()
index f07fb63..1982610 100644 (file)
@@ -488,6 +488,7 @@ set(EWK2UnitTests_BINARIES
     test_ewk2_settings
     test_ewk2_ssl
     test_ewk2_storage_manager
+    test_ewk2_view
     test_ewk2_window_features
 )
 
@@ -497,7 +498,6 @@ set(EWK2UnitTests_BINARIES
 # webkit.org/b/132980: test_ewk2_context_menu
 # webkit.org/b/132981: test_ewk2_download_job
 # webkit.org/b/132982: test_ewk2_text_checker
-# webkit.org/b/132983: test_ewk2_view
 
 if (ENABLE_API_TESTS)
     foreach (testName ${EWK2UnitTests_BINARIES})
index 3d0cd8a..15f36a0 100644 (file)
@@ -323,7 +323,7 @@ void EWK2UnitTestBase::multiMove(int id, int x, int y)
     evas_event_feed_multi_move(evas_object_evas_get(m_webView), id, x, y, 0, 0, 0, 0, 0, 0, 0, 0, 0);
 }
 
-void EWK2UnitTestBase::keyDown(char* keyname, char* key, char* string, char* modifier)
+void EWK2UnitTestBase::keyDown(const char* keyname, const char* key, const char* string, const char* modifier)
 {
     Evas* evas = evas_object_evas_get(m_webView);
     ASSERT(evas);
@@ -338,7 +338,7 @@ void EWK2UnitTestBase::keyDown(char* keyname, char* key, char* string, char* mod
     evas_event_feed_key_down(evas, keyname, key, string, 0, 0, 0);
 }
 
-void EWK2UnitTestBase::keyUp(char* keyname, char* key, char* string)
+void EWK2UnitTestBase::keyUp(const char* keyname, const char* key, const char* string)
 {
     evas_event_feed_key_up(evas_object_evas_get(m_webView), keyname, key, string, 0, 0, 0);
 }
index c28cb87..fb5c9f5 100644 (file)
@@ -63,8 +63,8 @@ protected:
     void multiDown(int id, int x, int y);
     void multiUp(int id, int x, int y);
     void multiMove(int id, int x, int y);
-    void keyDown(char* keyname, char* key, char* string, char* modifier);
-    void keyUp(char* keyname, char* key, char* string);
+    void keyDown(const char* keyname, const char* key, const char* string, const char* modifier);
+    void keyUp(const char* keyname, const char* key, const char* string);
 
 private:
     Evas_Object* m_webView;
index be34b23..302d8eb 100644 (file)
@@ -21,6 +21,7 @@
 
 #include "UnitTestUtils/EWK2UnitTestBase.h"
 #include "UnitTestUtils/EWK2UnitTestServer.h"
+#include <WebCore/CairoUtilities.h>
 #include <wtf/text/CString.h>
 #include <wtf/text/WTFString.h>
 
@@ -278,6 +279,7 @@ TEST_F(EWK2ViewTest, ewk_view_url_get)
     EXPECT_STREQ(environment->defaultTestPageUrl(), ewk_view_url_get(webView()));
 }
 
+#if HAVE(CAIRO_SURFACE_SET_DEVICE_SCALE)
 TEST_F(EWK2ViewTest, ewk_view_device_pixel_ratio)
 {
     ASSERT_TRUE(loadUrlSync(environment->defaultTestPageUrl()));
@@ -291,6 +293,7 @@ TEST_F(EWK2ViewTest, ewk_view_device_pixel_ratio)
     ASSERT_TRUE(ewk_view_device_pixel_ratio_set(webView(), 1));
     ASSERT_FLOAT_EQ(1, ewk_view_device_pixel_ratio_get(webView()));
 }
+#endif
 
 TEST_F(EWK2ViewTest, ewk_view_html_string_load)
 {
@@ -1207,12 +1210,14 @@ TEST_F(EWK2ViewTest, ewk_view_contents_size_get)
     EXPECT_EQ(environment->defaultWidth(), contentsWidth);
     EXPECT_EQ(environment->defaultHeight(), contentsHeight);
 
+#if HAVE(CAIRO_SURFACE_SET_DEVICE_SCALE)
     ewk_view_device_pixel_ratio_set(webView(), 2);
     ASSERT_TRUE(loadUrlSync(environment->defaultTestPageUrl()));
     ewk_view_contents_size_get(webView(), &contentsWidth, &contentsHeight);
 
     EXPECT_EQ(environment->defaultWidth() / 2, contentsWidth);
     EXPECT_EQ(environment->defaultHeight() / 2, contentsHeight);
+#endif
 
     const char fixedContentsSize[] =
         "<!DOCTYPE html>"