[Webkit-unassigned] [Bug 96769] [GTK] Don't use the C API internally in WebKitBackForwardList

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Oct 3 01:35:20 PDT 2012


https://bugs.webkit.org/show_bug.cgi?id=96769





--- Comment #5 from Carlos Garcia Campos <cgarcia at igalia.com>  2012-10-03 01:35:44 PST ---
(In reply to comment #4)
> (From update of attachment 166481 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=166481&action=review
> 
> This patch is pretty straightforward and I have not much to comment here. LGTM
> 
> > Source/WebKit2/UIProcess/API/gtk/WebKitBackForwardList.cpp:58
> > +    WebBackForwardList* webList;
> 
> I would probably name it backForwardList instead of webList. But I'm not genius choosing names either, so take it as a subjective opinion.

This was wkList before, I think, I renamed to webList to avoid confusion with C API variables.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list