<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK] Add function webkit_dom_element_get_bounding_client_rect"
href="https://bugs.webkit.org/show_bug.cgi?id=163892#c10">Comment # 10</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK] Add function webkit_dom_element_get_bounding_client_rect"
href="https://bugs.webkit.org/show_bug.cgi?id=163892">bug 163892</a>
from <span class="vcard"><a class="email" href="mailto:aidanholm+webkit@gmail.com" title="aidanholm+webkit@gmail.com">aidanholm+webkit@gmail.com</a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=163892#c8">comment #8</a>)
<span class="quote">> > Source/WebKit2/WebProcess/InjectedBundle/API/gtk/DOM/WebKitDOMClientRect.cpp:118
> > +static GObject* webkit_dom_client_rect_constructor(GType type, guint constructPropertiesCount, GObjectConstructParam* constructProperties)
>
> I think you can use constructed for this instead of constructor. It would be
> a lot simpler.</span >
Not sure what you mean by this. I've fixed the other complaints.
<span class="quote">> > Source/WebKit2/WebProcess/InjectedBundle/API/gtk/DOM/WebKitDOMClientRect.cpp:143
> > + "read-only gfloat ClientRect:top",
>
> Ahaha, not a very good property description. :P I guess it's fine, though,
> since it's the style used by existing DOM objects.</span >
When in Rome... I've had the same thought while browsing the docs :)
<span class="quote">> > Source/WebKit2/WebProcess/InjectedBundle/API/gtk/DOM/WebKitDOMElement.h:532
> > + * Returns: (element-type WebKitDOMClientRect) (transfer full): A #GList
>
> Is it really transfer full, or is it actually transfer container (where the
> GList itself is owned by the caller, but the elements are unowned)? It's not
> clear to me from the implementation. i.e., do you free the list with
> g_list_free(list) or with g_list_free_full(list, g_object_unref)?</span >
Actually, I think the way I implemented it currently is transfer full for both functions... I'm still getting used to WebKit's refcounting system. I'll try to change that to transfer none and transfer container, since that seems more user-friendly.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>