142967 2015-03-23 08:46:16 -0700 [EFL][WK2] EWK2ContextTestMultipleProcesses.ewk_context_network_process_model fails to pass 2015-07-22 07:54:53 -0700 1 1 1 Unclassified WebKit WebKit EFL 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 132813 1 gyuyoung.kim gyuyoung.kim commit-queue lucas.de.marchi ossy piorkowskiprzemyslaw oldest_to_newest 1079287 0 gyuyoung.kim 2015-03-23 08:46:16 -0700 In test_ewk2_context, EWK2ContextTestMultipleProcesses.ewk_context_network_process_model is often failed. The failure reason is that webView1WebProcessID is same with webView2WebProcessID. Need to investigate. PlatformProcessIdentifier webView1WebProcessID = toImpl(EWKViewGetWKView(webView1))->page()->process().processIdentifier(); PlatformProcessIdentifier webView2WebProcessID = toImpl(EWKViewGetWKView(webView2))->page()->process().processIdentifier(); PlatformProcessIdentifier webView1NetworkProcessID = toImpl(EWKViewGetWKView(webView1))->page()->process().processPool().networkProcess()->processIdentifier(); PlatformProcessIdentifier webView2NetworkProcessID = toImpl(EWKViewGetWKView(webView2))->page()->process().processPool().networkProcess()->processIdentifier(); ASSERT_NE(webView1WebProcessID, webView2WebProcessID); 1079288 1 gyuyoung.kim 2015-03-23 09:02:18 -0700 This test result looks like flaky. 1085891 2 gyuyoung.kim 2015-04-16 05:41:17 -0700 I disable this test until fixing it in order to maintain EFL bot. Bug 143824 is filed to disable it. 1109063 3 256687 gyuyoung.kim 2015-07-12 22:43:04 -0700 Created attachment 256687 Patch 1109064 4 gyuyoung.kim 2015-07-12 22:46:44 -0700 EWK2ContextTestMultipleProcesses.ewk_context_network_process_model test is flaky in EFL API test. When failure happens, process id is 0 as below, ../../Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_context.cpp:254: Failure Expected: (webView1WebProcessID) != (webView2WebProcessID), actual: 0 vs 0 [ FAILED ] EWK2ContextTestMultipleProcesses.ewk_context_network_process_model (8 ms) [----------] 2 tests from EWK2ContextTestMultipleProcesses (24 ms total) It looks that 0 can be returned when page is not closed. PlatformProcessIdentifier WebPageProxy::processIdentifier() const { if (m_isClosed) return 0; return m_process->processIdentifier(); } Unfortunately I don't know yet why a page is closed during the test. So I add a workaround condition which doesn't test it when process id is 0 for now. 1109071 5 256690 gyuyoung.kim 2015-07-13 01:20:46 -0700 Created attachment 256690 Patch 1109549 6 gyuyoung.kim 2015-07-14 19:39:56 -0700 Ossy, ping ? 1110072 7 256955 gyuyoung.kim 2015-07-16 20:16:38 -0700 Created attachment 256955 Patch 1111260 8 256955 ossy 2015-07-22 06:59:43 -0700 Comment on attachment 256955 Patch rs=me 1111274 9 256955 commit-queue 2015-07-22 07:54:50 -0700 Comment on attachment 256955 Patch Clearing flags on attachment: 256955 Committed r187156: <http://trac.webkit.org/changeset/187156> 1111275 10 commit-queue 2015-07-22 07:54:53 -0700 All reviewed patches have been landed. Closing bug. 256687 2015-07-12 22:43:04 -0700 2015-07-13 01:20:39 -0700 Patch bug-142967-20150713144159.patch text/plain 2068 gyuyoung.kim U3VidmVyc2lvbiBSZXZpc2lvbjogMTg2NzQ1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggMzk1Yzc0NWQzOTZiYmE0 ZjgzOGQ5YWU3NmUwNDgzZWQwOWQ2MzI0My4uNTU1ZjZiNjM2ZmE2MGFmNTkxNDVlZGQ3N2Q3MTg1 MGFjZWMwYWI0YSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDE1LTA3LTEyICBHeXV5 b3VuZyBLaW0gIDxneXV5b3VuZy5raW1Ad2Via2l0Lm9yZz4KKworICAgICAgICBbRUZMXVtXSzJd IEVXSzJDb250ZXh0VGVzdE11bHRpcGxlUHJvY2Vzc2VzLmV3a19jb250ZXh0X25ldHdvcmtfcHJv Y2Vzc19tb2RlbCBmYWlscyB0byBwYXNzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn L3Nob3dfYnVnLmNnaT9pZD0xNDI5NjcKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9P UFMhKS4KKworICAgICAgICBUaG91Z2ggcHJvY2Vzc0lkZW50aWZpZXIoKSBjYW4gcmV0dXJuIDAg d2hlbiBhIHBhZ2UgaXMgbm90IGNsb3NlZCwgZXdrX2NvbnRleHRfbmV0d29ya19wcm9jZXNzX21v ZGVsIGhhcworICAgICAgICBub3QgY29uc2lkZXJlZCBwcm9jZXNzIGlkIGNhbiBiZSAwLiBUaGlz IHBhdGNoIGFkZHMgYSBwcm90ZWN0aW9uIGxvZ2ljIHdoaWNoIGRvZXNuJ3QgdGVzdCB3aGVuIHBy b2Nlc3MgaWQgaXMgMAorICAgICAgICBhdCB0aGUgbW9tZW50LgorCisgICAgICAgICogVUlQcm9j ZXNzL0FQSS9lZmwvdGVzdHMvdGVzdF9ld2syX2NvbnRleHQuY3BwOgorICAgICAgICAoVEVTVF9G KToKKwogMjAxNS0wNy0xMSAgTmlraXRhIFZhc2lseWV2ICA8bnZhc2lseWV2QGFwcGxlLmNvbT4K IAogICAgICAgICBXZWIgSW5zcGVjdG9yOiBJbnNwZWN0b3Igc2hvdWxkIGJlIGFibGUgdG8gYmUg ZG9ja2VkIHRvIHRoZSBib3R0b20gb2YgYSBuYXJyb3cgd2luZG93CmRpZmYgLS1naXQgYS9Tb3Vy Y2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL2VmbC90ZXN0cy90ZXN0X2V3azJfY29udGV4dC5jcHAg Yi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL2VmbC90ZXN0cy90ZXN0X2V3azJfY29udGV4 dC5jcHAKaW5kZXggNDRhMGUzZWMyMjVlNWJhNTM0ZjEyMGI2ZjY2NjJmZWY0ZDg1MmQwNC4uMWMz NzNjN2Q1MmM1ZTNiZWNjMTE0YjQyMzVlNjRhYWQxMTIxYjc0OCAxMDA2NDQKLS0tIGEvU291cmNl L1dlYktpdDIvVUlQcm9jZXNzL0FQSS9lZmwvdGVzdHMvdGVzdF9ld2syX2NvbnRleHQuY3BwCisr KyBiL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZWZsL3Rlc3RzL3Rlc3RfZXdrMl9jb250 ZXh0LmNwcApAQCAtMjUxLDYgKzI1MSw5IEBAIFRFU1RfRihFV0syQ29udGV4dFRlc3RNdWx0aXBs ZVByb2Nlc3NlcywgZXdrX2NvbnRleHRfbmV0d29ya19wcm9jZXNzX21vZGVsKQogICAgIFBsYXRm b3JtUHJvY2Vzc0lkZW50aWZpZXIgd2ViVmlldzFOZXR3b3JrUHJvY2Vzc0lEID0gdG9JbXBsKEVX S1ZpZXdHZXRXS1ZpZXcod2ViVmlldzEpKS0+cGFnZSgpLT5wcm9jZXNzKCkucHJvY2Vzc1Bvb2wo KS5uZXR3b3JrUHJvY2VzcygpLT5wcm9jZXNzSWRlbnRpZmllcigpOwogICAgIFBsYXRmb3JtUHJv Y2Vzc0lkZW50aWZpZXIgd2ViVmlldzJOZXR3b3JrUHJvY2Vzc0lEID0gdG9JbXBsKEVXS1ZpZXdH ZXRXS1ZpZXcod2ViVmlldzIpKS0+cGFnZSgpLT5wcm9jZXNzKCkucHJvY2Vzc1Bvb2woKS5uZXR3 b3JrUHJvY2VzcygpLT5wcm9jZXNzSWRlbnRpZmllcigpOwogCisgICAgaWYgKCF3ZWJWaWV3MVdl YlByb2Nlc3NJRCB8fCAhd2ViVmlldzJXZWJQcm9jZXNzSUQpCisgICAgICAgIHJldHVybjsKKwog ICAgIEFTU0VSVF9ORSh3ZWJWaWV3MVdlYlByb2Nlc3NJRCwgd2ViVmlldzJXZWJQcm9jZXNzSUQp OwogICAgIEFTU0VSVF9ORSh3ZWJWaWV3MVdlYlByb2Nlc3NJRCwgd2ViVmlldzFOZXR3b3JrUHJv Y2Vzc0lEKTsKICAgICBBU1NFUlRfTkUod2ViVmlldzFXZWJQcm9jZXNzSUQsIHdlYlZpZXcyTmV0 d29ya1Byb2Nlc3NJRCk7Cg== 256690 2015-07-13 01:20:46 -0700 2015-07-16 20:16:33 -0700 Patch bug-142967-20150713171941.patch text/plain 2068 gyuyoung.kim U3VidmVyc2lvbiBSZXZpc2lvbjogMTg2NzQ1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggMzk1Yzc0NWQzOTZiYmE0 ZjgzOGQ5YWU3NmUwNDgzZWQwOWQ2MzI0My4uNTU1ZjZiNjM2ZmE2MGFmNTkxNDVlZGQ3N2Q3MTg1 MGFjZWMwYWI0YSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDE1LTA3LTEyICBHeXV5 b3VuZyBLaW0gIDxneXV5b3VuZy5raW1Ad2Via2l0Lm9yZz4KKworICAgICAgICBbRUZMXVtXSzJd IEVXSzJDb250ZXh0VGVzdE11bHRpcGxlUHJvY2Vzc2VzLmV3a19jb250ZXh0X25ldHdvcmtfcHJv Y2Vzc19tb2RlbCBmYWlscyB0byBwYXNzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn L3Nob3dfYnVnLmNnaT9pZD0xNDI5NjcKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9P UFMhKS4KKworICAgICAgICBUaG91Z2ggcHJvY2Vzc0lkZW50aWZpZXIoKSBjYW4gcmV0dXJuIDAg d2hlbiBhIHBhZ2UgaXMgbm90IGNsb3NlZCwgZXdrX2NvbnRleHRfbmV0d29ya19wcm9jZXNzX21v ZGVsIGhhcworICAgICAgICBub3QgY29uc2lkZXJlZCBwcm9jZXNzIGlkIGNhbiBiZSAwLiBUaGlz IHBhdGNoIGFkZHMgYSBwcm90ZWN0aW9uIGxvZ2ljIHdoaWNoIGRvZXNuJ3QgdGVzdCB3aGVuIHBy b2Nlc3MgaWQgaXMgMAorICAgICAgICBhdCB0aGUgbW9tZW50LgorCisgICAgICAgICogVUlQcm9j ZXNzL0FQSS9lZmwvdGVzdHMvdGVzdF9ld2syX2NvbnRleHQuY3BwOgorICAgICAgICAoVEVTVF9G KToKKwogMjAxNS0wNy0xMSAgTmlraXRhIFZhc2lseWV2ICA8bnZhc2lseWV2QGFwcGxlLmNvbT4K IAogICAgICAgICBXZWIgSW5zcGVjdG9yOiBJbnNwZWN0b3Igc2hvdWxkIGJlIGFibGUgdG8gYmUg ZG9ja2VkIHRvIHRoZSBib3R0b20gb2YgYSBuYXJyb3cgd2luZG93CmRpZmYgLS1naXQgYS9Tb3Vy Y2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL2VmbC90ZXN0cy90ZXN0X2V3azJfY29udGV4dC5jcHAg Yi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL2VmbC90ZXN0cy90ZXN0X2V3azJfY29udGV4 dC5jcHAKaW5kZXggNDRhMGUzZWMyMjVlNWJhNTM0ZjEyMGI2ZjY2NjJmZWY0ZDg1MmQwNC4uMWMz NzNjN2Q1MmM1ZTNiZWNjMTE0YjQyMzVlNjRhYWQxMTIxYjc0OCAxMDA2NDQKLS0tIGEvU291cmNl L1dlYktpdDIvVUlQcm9jZXNzL0FQSS9lZmwvdGVzdHMvdGVzdF9ld2syX2NvbnRleHQuY3BwCisr KyBiL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZWZsL3Rlc3RzL3Rlc3RfZXdrMl9jb250 ZXh0LmNwcApAQCAtMjUxLDYgKzI1MSw5IEBAIFRFU1RfRihFV0syQ29udGV4dFRlc3RNdWx0aXBs ZVByb2Nlc3NlcywgZXdrX2NvbnRleHRfbmV0d29ya19wcm9jZXNzX21vZGVsKQogICAgIFBsYXRm b3JtUHJvY2Vzc0lkZW50aWZpZXIgd2ViVmlldzFOZXR3b3JrUHJvY2Vzc0lEID0gdG9JbXBsKEVX S1ZpZXdHZXRXS1ZpZXcod2ViVmlldzEpKS0+cGFnZSgpLT5wcm9jZXNzKCkucHJvY2Vzc1Bvb2wo KS5uZXR3b3JrUHJvY2VzcygpLT5wcm9jZXNzSWRlbnRpZmllcigpOwogICAgIFBsYXRmb3JtUHJv Y2Vzc0lkZW50aWZpZXIgd2ViVmlldzJOZXR3b3JrUHJvY2Vzc0lEID0gdG9JbXBsKEVXS1ZpZXdH ZXRXS1ZpZXcod2ViVmlldzIpKS0+cGFnZSgpLT5wcm9jZXNzKCkucHJvY2Vzc1Bvb2woKS5uZXR3 b3JrUHJvY2VzcygpLT5wcm9jZXNzSWRlbnRpZmllcigpOwogCisgICAgaWYgKCF3ZWJWaWV3MVdl YlByb2Nlc3NJRCB8fCAhd2ViVmlldzJXZWJQcm9jZXNzSUQpCisgICAgICAgIHJldHVybjsKKwog ICAgIEFTU0VSVF9ORSh3ZWJWaWV3MVdlYlByb2Nlc3NJRCwgd2ViVmlldzJXZWJQcm9jZXNzSUQp OwogICAgIEFTU0VSVF9ORSh3ZWJWaWV3MVdlYlByb2Nlc3NJRCwgd2ViVmlldzFOZXR3b3JrUHJv Y2Vzc0lEKTsKICAgICBBU1NFUlRfTkUod2ViVmlldzFXZWJQcm9jZXNzSUQsIHdlYlZpZXcyTmV0 d29ya1Byb2Nlc3NJRCk7Cg== 256955 2015-07-16 20:16:38 -0700 2015-07-22 07:54:50 -0700 Patch bug-142967-20150717121528.patch text/plain 2056 gyuyoung.kim U3VidmVyc2lvbiBSZXZpc2lvbjogMTg2OTIyCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggMTIxOWZhODg1ZmE1MjBm NWZmZWIxNGRiYWUzZjEwNzI1NzU3ZTdjYS4uNzc2ZWY4MWYzNTI3ODYyNTFmYzk2MzMzYjAwMDAz ZjUzZDllYjVkZiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDE1LTA3LTE2ICBHeXV5 b3VuZyBLaW0gIDxneXV5b3VuZy5raW1Ad2Via2l0Lm9yZz4KKworICAgICAgICBbRUZMXVtXSzJd IEVXSzJDb250ZXh0VGVzdE11bHRpcGxlUHJvY2Vzc2VzLmV3a19jb250ZXh0X25ldHdvcmtfcHJv Y2Vzc19tb2RlbCBmYWlscyB0byBwYXNzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn L3Nob3dfYnVnLmNnaT9pZD0xNDI5NjcKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9P UFMhKS4KKworICAgICAgICBUaG91Z2ggcHJvY2Vzc0lkZW50aWZpZXIoKSBjYW4gcmV0dXJuIDAg d2hlbiBhIHBhZ2UgaXMgbm90IGNsb3NlZCwgZXdrX2NvbnRleHRfbmV0d29ya19wcm9jZXNzX21v ZGVsIGhhcworICAgICAgICBub3QgYmVlbiBjb25zaWRlcmVkIHByb2Nlc3MgaWQgY2FuIGJlIDAu IFRoaXMgcGF0Y2ggYWRkcyBhIHByb3RlY3Rpb24gbG9naWMgd2hpY2ggZG9lc24ndCB0ZXN0IHdo ZW4gcHJvY2VzcworICAgICAgICBpZCBpcyAwIGF0IHRoZSBtb21lbnQuCisKKyAgICAgICAgKiBV SVByb2Nlc3MvQVBJL2VmbC90ZXN0cy90ZXN0X2V3azJfY29udGV4dC5jcHA6CisgICAgICAgIChU RVNUX0YpOgorCiAyMDE1LTA3LTE2ICBKb3NlcGggUGVjb3Jhcm8gIDxwZWNvcmFyb0BhcHBsZS5j b20+CiAKICAgICAgICAgV2ViIEluc3BlY3RvcjogW01hY10gU2F2ZSBkaWFsb2cgbm90IHdvcmtp bmcgd2hlbiBpbnNwZWN0b3IgaXMgZG9ja2VkCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9V SVByb2Nlc3MvQVBJL2VmbC90ZXN0cy90ZXN0X2V3azJfY29udGV4dC5jcHAgYi9Tb3VyY2UvV2Vi S2l0Mi9VSVByb2Nlc3MvQVBJL2VmbC90ZXN0cy90ZXN0X2V3azJfY29udGV4dC5jcHAKaW5kZXgg NDRhMGUzZWMyMjVlNWJhNTM0ZjEyMGI2ZjY2NjJmZWY0ZDg1MmQwNC4uMWMzNzNjN2Q1MmM1ZTNi ZWNjMTE0YjQyMzVlNjRhYWQxMTIxYjc0OCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvVUlQ cm9jZXNzL0FQSS9lZmwvdGVzdHMvdGVzdF9ld2syX2NvbnRleHQuY3BwCisrKyBiL1NvdXJjZS9X ZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZWZsL3Rlc3RzL3Rlc3RfZXdrMl9jb250ZXh0LmNwcApAQCAt MjUxLDYgKzI1MSw5IEBAIFRFU1RfRihFV0syQ29udGV4dFRlc3RNdWx0aXBsZVByb2Nlc3Nlcywg ZXdrX2NvbnRleHRfbmV0d29ya19wcm9jZXNzX21vZGVsKQogICAgIFBsYXRmb3JtUHJvY2Vzc0lk ZW50aWZpZXIgd2ViVmlldzFOZXR3b3JrUHJvY2Vzc0lEID0gdG9JbXBsKEVXS1ZpZXdHZXRXS1Zp ZXcod2ViVmlldzEpKS0+cGFnZSgpLT5wcm9jZXNzKCkucHJvY2Vzc1Bvb2woKS5uZXR3b3JrUHJv Y2VzcygpLT5wcm9jZXNzSWRlbnRpZmllcigpOwogICAgIFBsYXRmb3JtUHJvY2Vzc0lkZW50aWZp ZXIgd2ViVmlldzJOZXR3b3JrUHJvY2Vzc0lEID0gdG9JbXBsKEVXS1ZpZXdHZXRXS1ZpZXcod2Vi VmlldzIpKS0+cGFnZSgpLT5wcm9jZXNzKCkucHJvY2Vzc1Bvb2woKS5uZXR3b3JrUHJvY2Vzcygp LT5wcm9jZXNzSWRlbnRpZmllcigpOwogCisgICAgaWYgKCF3ZWJWaWV3MVdlYlByb2Nlc3NJRCB8 fCAhd2ViVmlldzJXZWJQcm9jZXNzSUQpCisgICAgICAgIHJldHVybjsKKwogICAgIEFTU0VSVF9O RSh3ZWJWaWV3MVdlYlByb2Nlc3NJRCwgd2ViVmlldzJXZWJQcm9jZXNzSUQpOwogICAgIEFTU0VS VF9ORSh3ZWJWaWV3MVdlYlByb2Nlc3NJRCwgd2ViVmlldzFOZXR3b3JrUHJvY2Vzc0lEKTsKICAg ICBBU1NFUlRfTkUod2ViVmlldzFXZWJQcm9jZXNzSUQsIHdlYlZpZXcyTmV0d29ya1Byb2Nlc3NJ RCk7Cg==