| Summary: | [GTK] Add API to create a webview related to another one | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> | ||||
| Component: | WebKit2 | Assignee: | Nobody <webkit-unassigned> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | berto, bunhere, cdumez, commit-queue, gustavo, gyuyoung.kim, mrobinson, sergio, svillar | ||||
| Priority: | P2 | Keywords: | Gtk | ||||
| Version: | 528+ (Nightly build) | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Attachments: |
|
||||||
|
Description
Carlos Garcia Campos
2014-02-09 03:32:08 PST
Created attachment 223626 [details]
Patch
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 on attachment 223626 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=223626&action=review > Source/WebKit2/UIProcess/API/gtk/WebKitWebView.cpp:653 > + * > + * The related #WebKitWebView used when creating the view to share the > + * same web process. > + * We should explain here that you can't obtain the related view after the creation, or is the gtk-doc note for a write-only, consctruct-only property enough, you think (I don't recall seeing one)? (In reply to comment #3) > (From update of attachment 223626 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=223626&action=review > > > Source/WebKit2/UIProcess/API/gtk/WebKitWebView.cpp:653 > > + * > > + * The related #WebKitWebView used when creating the view to share the > > + * same web process. > > + * > > We should explain here that you can't obtain the related view after the creation, or is the gtk-doc note for a write-only, consctruct-only property enough, you think (I don't recall seeing one)? I assumed that being writable/construct only property and the lack of public getter was clear enough, but I'll add a comment in the docs. Committed r163781: <http://trac.webkit.org/changeset/163781> |