Bug 86005

Summary: [GTK] Missing field initializers for WKPageLoaderClient
Product: WebKit Reporter: Simon Pena <simon.pena>
Component: WebKit2Assignee: Nobody <webkit-unassigned>
Severity: Normal CC: cgarcia, gns, mrobinson, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Patch none

Description Simon Pena 2012-05-09 10:54:14 PDT
Building WK2 currently outputs the following warnings:

../../Source/WebKit2/UIProcess/API/gtk/WebKitLoaderClient.cpp: In function ‘void attachLoaderClientToView(WebKitWebView*)’:
../../Source/WebKit2/UIProcess/API/gtk/WebKitLoaderClient.cpp:149:5: warning: missing initializer for member ‘WKPageLoaderClient::willGoToBackForwardListItem’ [-Wmissing-field-initializers]
../../Source/WebKit2/UIProcess/API/gtk/WebKitLoaderClient.cpp:149:5: warning: missing initializer for member ‘WKPageLoaderClient::interactionOccurredWhileProcessUnresponsive’ [-Wmissing-field-initializers]

The attached patch initializes these members.
Comment 1 Simon Pena 2012-05-09 10:59:14 PDT
Created attachment 140977 [details]
Comment 2 WebKit Review Bot 2012-05-09 11:01:05 PDT
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 3 WebKit Review Bot 2012-05-09 11:11:55 PDT
Comment on attachment 140977 [details]

Rejecting attachment 140977 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 1

ERROR: /mnt/git/webkit-commit-queue/Source/WebKit2/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive).

Full output: http://queues.webkit.org/results/12656723
Comment 4 Simon Pena 2012-05-09 11:15:31 PDT
I'll check if I messed up the changelog, one sec.
Comment 5 Simon Pena 2012-05-09 11:18:51 PDT
Created attachment 140983 [details]
Comment 6 WebKit Review Bot 2012-05-10 09:32:44 PDT
Comment on attachment 140983 [details]

Clearing flags on attachment: 140983

Committed r116651: <http://trac.webkit.org/changeset/116651>
Comment 7 WebKit Review Bot 2012-05-10 09:32:48 PDT
All reviewed patches have been landed.  Closing bug.