[webkit-reviews] review denied: [Bug 14812] [gtk] [request] add
webkit_gtk_frame_get_title() function : [Attachment 16044]
Updated to trunk and following naming comments
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Aug 27 03:30:16 PDT 2007
Mark Rowe (bdash) <mrowe at apple.com> has denied Diego Escalante Urrelo
<diegoe at gnome.org>'s request for review:
Bug 14812: [gtk] [request] add webkit_gtk_frame_get_title() function
http://bugs.webkit.org/show_bug.cgi?id=14812
Attachment 16044: Updated to trunk and following naming comments
http://bugs.webkit.org/attachment.cgi?id=16044&action=edit
------- Additional Comments from Mark Rowe (bdash) <mrowe at apple.com>
Inside webkit_gtk_frame_title_changed_cb g_strdup is called but I don't see any
corresponding calls to free the memory.
There are some minor coding style issues present:
+gchar*
+webkit_gtk_frame_get_title (WebKitGtkFrame* frame)
should be one line, for example.
You should also include a ChangeLog entry. You can use
WebKitTools/Scripts/prepare-ChangeLog to generate a template which you can then
fill in.
More information about the webkit-reviews
mailing list