Bug 38763

Summary: ResourceHandleSoup should not depend on GTK+
Product: WebKit Reporter: Leandro Pereira <leandro>
Component: WebCore Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
ResourceHandleSoup should not depend on GTK+
gns: review-
ResourceHandleSoup should not depend on GTK+ none

Description Leandro Pereira 2010-05-07 11:46:39 PDT
ResourceHandleSoup.cpp includes gtk/gtk.h, while it should include only glib.h. Attached patches fixes that.
Comment 1 Leandro Pereira 2010-05-07 11:47:37 PDT
Created attachment 55401 [details]
ResourceHandleSoup should not depend on GTK+
Comment 2 Gustavo Noronha (kov) 2010-05-07 12:03:17 PDT
Comment on attachment 55401 [details]
ResourceHandleSoup should not depend on GTK+

Change the CString include to be #include "CString.h", please, and leave it at the first block.
Comment 3 Gustavo Noronha (kov) 2010-05-07 12:05:16 PDT
Comment on attachment 55401 [details]
ResourceHandleSoup should not depend on GTK+

461461     g_object_set(session,
462462                  SOUP_SESSION_MAX_CONNS, MAX_CONNECTIONS,
463463                  SOUP_SESSION_MAX_CONNS_PER_HOST, MAX_CONNECTIONS_PER_HOST,
464                   NULL);
 464                  0);

We use NULL for GObject calls, because they generate compiler warnings otherwise.

596      // Used to set the authentication dialog toplevel; may be NULL
 596     // Used to set the authentication dialog toplevel; may be 0

This is a bug in the checker, just leave this as NULL.
Comment 4 Leandro Pereira 2010-05-07 12:13:15 PDT
Created attachment 55403 [details]
ResourceHandleSoup should not depend on GTK+

Reviewed patch according to gns' comments.
Comment 5 Gustavo Noronha (kov) 2010-05-07 12:47:08 PDT
Comment on attachment 55403 [details]
ResourceHandleSoup should not depend on GTK+

Looks good!
Comment 6 WebKit Commit Bot 2010-05-08 10:53:51 PDT
Comment on attachment 55403 [details]
ResourceHandleSoup should not depend on GTK+

Clearing flags on attachment: 55403

Committed r59028: <http://trac.webkit.org/changeset/59028>
Comment 7 WebKit Commit Bot 2010-05-08 10:53:56 PDT
All reviewed patches have been landed.  Closing bug.