[EFL] Fix failing WebKit1 API test of ewk_view_setting_tiled_backing_store
authorjinwoo7.song@samsung.com <jinwoo7.song@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 6 Jan 2014 07:54:02 +0000 (07:54 +0000)
committerjinwoo7.song@samsung.com <jinwoo7.song@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 6 Jan 2014 07:54:02 +0000 (07:54 +0000)
https://bugs.webkit.org/show_bug.cgi?id=126509

Reviewed by Gyuyoung Kim.

In WebKit1 EFL port, unit test was using a webview based on tiled backing store,
so ewk_view_setting_tiled_backing_store_enabled_set() was set as true as default.
As we do not need to use tiled backing store, we disable tiled backing store
and remove unnecessary EwkViewType in unit test webview.

* tests/UnitTestUtils/EWKTestView.cpp:
(EWKUnitTests::EWKTestView::init):
* tests/UnitTestUtils/EWKTestView.h:
* tests/test_ewk_view.cpp: Fix wrong comment in the unit test.

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

Source/WebKit/efl/ChangeLog
Source/WebKit/efl/tests/UnitTestUtils/EWKTestView.cpp
Source/WebKit/efl/tests/UnitTestUtils/EWKTestView.h
Source/WebKit/efl/tests/test_ewk_view.cpp

index 98c7e11..a24461e 100644 (file)
@@ -1,3 +1,20 @@
+2014-01-05  Jinwoo Song  <jinwoo7.song@samsung.com>
+
+        [EFL] Fix failing WebKit1 API test of ewk_view_setting_tiled_backing_store
+        https://bugs.webkit.org/show_bug.cgi?id=126509
+
+        Reviewed by Gyuyoung Kim.
+
+        In WebKit1 EFL port, unit test was using a webview based on tiled backing store,
+        so ewk_view_setting_tiled_backing_store_enabled_set() was set as true as default.
+        As we do not need to use tiled backing store, we disable tiled backing store
+        and remove unnecessary EwkViewType in unit test webview.
+
+        * tests/UnitTestUtils/EWKTestView.cpp:
+        (EWKUnitTests::EWKTestView::init):
+        * tests/UnitTestUtils/EWKTestView.h:
+        * tests/test_ewk_view.cpp: Fix wrong comment in the unit test.
+
 2014-01-02  Gavin Barraclough  <barraclough@apple.com>
 
         Merge didMoveOnscreen / page visibility to isVisible
index ef14f6a..db2a882 100644 (file)
@@ -29,7 +29,7 @@ EWKTestView::EWKTestView()
 {
 }
 
-bool EWKTestView::init(EwkViewType testViewType, int width, int height)
+bool EWKTestView::init(int width, int height)
 {
     m_webView = nullptr;
 
@@ -46,9 +46,6 @@ bool EWKTestView::init(EwkViewType testViewType, int width, int height)
     if (!m_webView)
         return false;
 
-    if (testViewType == TiledView)
-        ewk_view_setting_tiled_backing_store_enabled_set(m_webView.get(), EINA_TRUE);
-
     ewk_view_theme_set(m_webView.get(), Config::defaultThemePath);
 
     evas_object_resize(m_webView.get(), width, height);
index f792a38..d6a12c5 100644 (file)
@@ -29,16 +29,11 @@ namespace EWKUnitTests {
 
 class EWKTestView {
 public:
-    enum EwkViewType {
-        SingleView = 0,
-        TiledView
-    };
-
     EWKTestView();
 
     Evas_Object* webView() { return m_webView.get(); }
 
-    bool init(EwkViewType testViewType = TiledView, int width = Config::defaultViewWidth, int height = Config::defaultViewHeight);
+    bool init(int width = Config::defaultViewWidth, int height = Config::defaultViewHeight);
 private:
     EWKTestView(const EWKTestView&);
     EWKTestView& operator=(const EWKTestView&);
index a3d73af..9f43037 100644 (file)
@@ -72,7 +72,7 @@ TEST_F(EWKTestBase, ewk_view_setting_enable_fullscreen)
 }
 
 /**
-* @brief Checking whether function properly get/set fullscreen setting value.
+* @brief Checking whether function properly get/set tiled backing store setting value.
 */
 TEST_F(EWKTestBase, ewk_view_setting_tiled_backing_store)
 {