Changeset 185696 in webkit

Timestamp:

Jun 18, 2015, 1:57:09 AM (11 years ago)

Author:

Gyuyoung Kim

Message:

[EFL] test_ewk2_context has been failed since r185529
​https://bugs.webkit.org/show_bug.cgi?id=146046

Reviewed by Csaba Osztrogonác.

ewk_context_network_process_model test has been failed since r185529.
This patch modifies the test which is similar to ewk_context_network_process_model
based on multiple process model to fix the failure. Besides ewk_context_network_process_model
seems to be fixed indirectly. So we enable it again.

• UIProcess/API/efl/tests/test_ewk2_context.cpp:

(TEST_F):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/API/efl/tests/test_ewk2_context.cpp (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2015-06-18  Gyuyoung Kim  <gyuyoung.kim@webkit.org>
+
+        [EFL] test_ewk2_context has been failed since r185529
+        https://bugs.webkit.org/show_bug.cgi?id=146046
+
+        Reviewed by Csaba Osztrogonác.
+
+        ewk_context_network_process_model test has been failed since r185529.
+        This patch modifies the test which is similar to ewk_context_network_process_model
+        based on multiple process model to fix the failure. Besides ewk_context_network_process_model
+        seems to be fixed indirectly. So we enable it again.
+
+        * UIProcess/API/efl/tests/test_ewk2_context.cpp:
+        (TEST_F):
+
 2015-06-17  Yongjun Zhang  <yongjun_zhang@apple.com>
 

trunk/Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_context.cpp

@@ -211 +211 @@
     ASSERT_EQ(webView1WebProcessID, webView2WebProcessID);
 
-    ASSERT_TRUE(toImpl(EWKViewGetWKView(webView1))->page()->process().processPool().networkProcess() == nullptr);
-    ASSERT_TRUE(toImpl(EWKViewGetWKView(webView2))->page()->process().processPool().networkProcess() == nullptr);
-}
-
-
-TEST_F(EWK2ContextTestMultipleProcesses, DISABLED_ewk_context_network_process_model)
+    PlatformProcessIdentifier webView1NetworkProcessID = toImpl(EWKViewGetWKView(webView1))->page()->process().processPool().networkProcess()->processIdentifier();
+    PlatformProcessIdentifier webView2NetworkProcessID = toImpl(EWKViewGetWKView(webView2))->page()->process().processPool().networkProcess()->processIdentifier();
+
+    ASSERT_EQ(webView1NetworkProcessID, webView2NetworkProcessID);
+}
+
+TEST_F(EWK2ContextTestMultipleProcesses, ewk_context_network_process_model)
 {
     Ewk_Context* context = ewk_view_context_get(webView());