Bug 203994

Summary: [GTK] WebDriver: implement new window command
Product: WebKit Reporter: Carlos Garcia Campos <cgarcia>
Component: WebDriverAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bburg, berto, bugs-noreply, clopez, ews-watchlist, gustavo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch clopez: review+, clopez: commit-queue-

Description Carlos Garcia Campos 2019-11-08 06:07:21 PST
11.5 New Window
https://w3c.github.io/webdriver/#new-window
Comment 1 Carlos Garcia Campos 2019-11-08 06:08:31 PST
Adding GTK since this requires new API.
Comment 2 Carlos Garcia Campos 2019-11-08 06:44:22 PST
Created attachment 383122 [details]
Patch

This patch includes unit tests for the new GLib API, but we don'tm have WebDriver tests for the new window command in WebKit tree yet. I've run wpt tests and all pass. I'll update the imported wpt tests in WebKit after landing this.
Comment 3 EWS Watchlist 2019-11-08 06:45:25 PST
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
Comment 4 Carlos Garcia Campos 2019-11-11 02:45:06 PST
Created attachment 383258 [details]
Patch
Comment 5 Carlos Alberto Lopez Perez 2019-11-11 13:33:53 PST
Comment on attachment 383258 [details]
Patch

r=me.
I guess this needs a second review since it adds new public API.
Comment 6 Carlos Garcia Campos 2019-11-12 00:12:47 PST
Committed r252356: <https://trac.webkit.org/changeset/252356>
Comment 7 Radar WebKit Bug Importer 2019-11-12 00:13:15 PST
<rdar://problem/57107742>