131434 2014-04-09 06:07:33 -0700 [WK2] When running in multi-process mode, opening link in new window from context menu creates blank window 2014-04-11 10:25:12 -0700 1 1 1 Unclassified WebKit WebKit2 528+ (Nightly build) Unspecified Unspecified NEW P2 Normal --- 1 m.florek webkit-unassigned ap gyuyoung.kim oldest_to_newest 999152 0 m.florek 2014-04-09 06:07:33 -0700 When running WK2 in multi-process mode (ie Minibrowser with -S switch on EFL port), opening link in new window ("Open link in new window" option in context menu), the newly created window is blank. 999161 1 228960 m.florek 2014-04-09 07:16:42 -0700 Created attachment 228960 Patch proposal to discuss 999162 2 m.florek 2014-04-09 07:17:13 -0700 The problem is in WebChromeClient::createWindow() function. It creates ( by means of WebProcess::shared().createWebPage() ) a new web page in its own WebProcess, which is associated to the current web page (on which the context menu was invoked). So now there are two WebPages in one Webprocess. But few lines before, the message was sent to the WebPageProxy, to create new page. And in UI process a new WebProcessProxy with new WebPageProxy is created, and finally the new WebProcess is launched. Newly created blank window is associated to newly created WebProcess. The first web process tries to pass url to newly created web page, but it sends it to the page in its own process. I have attached a draft of the patch to discuss. Url loading was moved to UI process, and the creation of new page in WebProcess was removed. The problem is, that on "WebChromeClient::createWindow()" function result (which now is null) depends many other functions... 999919 3 ap 2014-04-11 10:24:07 -0700 New windows opened with window.open need to be in the same WebProcess, because they are accessible via JavaScript. So, this is an issue in Efl on UI side, it should know to create a new WebPageProxy in the same process. 999920 4 ap 2014-04-11 10:25:12 -0700 Could also be an issue in MiniBrowser - I know that Safari implements opening in the same process when necessary, but don't know about MiniBrowser. 228960 2014-04-09 07:16:42 -0700 2014-04-09 07:16:42 -0700 Patch proposal to discuss bug131434.patch text/plain 1969 m.florek ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No YW5nZUxvZwppbmRleCA4ZjcwOTVjLi5lYTc5Y2QxIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0 Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUg QEAKKzIwMTQtMDQtMDkgIE1hY2llaiBGbG9yZWsgIDxtLmZsb3Jla0BzYW1zdW5nLmNvbT4KKwor ICAgICAgICBbV0syXSBXaGVuIHJ1bm5pbmcgaW4gbXVsdGktcHJvY2VzcyBtb2RlLCBvcGVuaW5n IGxpbmsgaW4gbmV3IHdpbmRvdyBmcm9tIGNvbnRleHQgbWVudSBjcmVhdGVzIGJsYW5rIHdpbmRv dworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTMxNDM0 CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBVSVBy b2Nlc3MvV2ViUGFnZVByb3h5LmNwcDoKKyAgICAgICAgKFdlYktpdDo6V2ViUGFnZVByb3h5Ojpj cmVhdGVOZXdQYWdlKToKKyAgICAgICAgKiBXZWJQcm9jZXNzL1dlYkNvcmVTdXBwb3J0L1dlYkNo cm9tZUNsaWVudC5jcHA6CisgICAgICAgIChXZWJLaXQ6OldlYkNocm9tZUNsaWVudDo6Y3JlYXRl V2luZG93KToKKwogMjAxNC0wNC0wOSAgQ2FybG9zIEdhcmNpYSBDYW1wb3MgIDxjZ2FyY2lhQGln YWxpYS5jb20+CiAKICAgICAgICAgVW5yZXZpZXdlZC4gRml4IEdUSysgYnVpbGQgYWZ0ZXIgcjE2 Njk3NS4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9XZWJQYWdlUHJveHku Y3BwIGIvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL1dlYlBhZ2VQcm94eS5jcHAKaW5kZXggMTZj OGYwYi4uMjkwZjBmYiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL1dlYlBh Z2VQcm94eS5jcHAKKysrIGIvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL1dlYlBhZ2VQcm94eS5j cHAKQEAgLTI2OTYsNiArMjY5Niw4IEBAIHZvaWQgV2ViUGFnZVByb3h5OjpjcmVhdGVOZXdQYWdl KHVpbnQ2NF90IGZyYW1lSUQsIGNvbnN0IFJlc291cmNlUmVxdWVzdCYgcmVxdWVzCiAgICAgbmV3 UGFnZUlEID0gbmV3UGFnZS0+cGFnZUlEKCk7CiAgICAgbmV3UGFnZVBhcmFtZXRlcnMgPSBuZXdQ YWdlLT5jcmVhdGlvblBhcmFtZXRlcnMoKTsKICAgICBwcm9jZXNzKCkuY29udGV4dCgpLnN0b3Jh Z2VNYW5hZ2VyKCkuY2xvbmVTZXNzaW9uU3RvcmFnZU5hbWVzcGFjZShtX3BhZ2VJRCwgbmV3UGFn ZS0+cGFnZUlEKCkpOworCisgICAgbmV3UGFnZS0+bG9hZFJlcXVlc3QocmVxdWVzdCk7CiB9CiAg ICAgCiB2b2lkIFdlYlBhZ2VQcm94eTo6c2hvd1BhZ2UoKQpkaWZmIC0tZ2l0IGEvU291cmNlL1dl YktpdDIvV2ViUHJvY2Vzcy9XZWJDb3JlU3VwcG9ydC9XZWJDaHJvbWVDbGllbnQuY3BwIGIvU291 cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9XZWJDb3JlU3VwcG9ydC9XZWJDaHJvbWVDbGllbnQuY3Bw CmluZGV4IGY4YzJkOTUuLjc0Nzc3MjUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQyL1dlYlBy b2Nlc3MvV2ViQ29yZVN1cHBvcnQvV2ViQ2hyb21lQ2xpZW50LmNwcAorKysgYi9Tb3VyY2UvV2Vi S2l0Mi9XZWJQcm9jZXNzL1dlYkNvcmVTdXBwb3J0L1dlYkNocm9tZUNsaWVudC5jcHAKQEAgLTIy MCw4ICsyMjAsNyBAQCBQYWdlKiBXZWJDaHJvbWVDbGllbnQ6OmNyZWF0ZVdpbmRvdyhGcmFtZSog ZnJhbWUsIGNvbnN0IEZyYW1lTG9hZFJlcXVlc3QmIHJlcXVlcwogICAgIGlmICghbmV3UGFnZUlE KQogICAgICAgICByZXR1cm4gMDsKIAotICAgIFdlYlByb2Nlc3M6OnNoYXJlZCgpLmNyZWF0ZVdl YlBhZ2UobmV3UGFnZUlELCBwYXJhbWV0ZXJzKTsKLSAgICByZXR1cm4gV2ViUHJvY2Vzczo6c2hh cmVkKCkud2ViUGFnZShuZXdQYWdlSUQpLT5jb3JlUGFnZSgpOworICAgIHJldHVybiAwOwogfQog CiB2b2lkIFdlYkNocm9tZUNsaWVudDo6c2hvdygpCg==