Bug 48816

Summary: [GTK] PopupMenuGtk has a very confusing if-else block
Product: WebKit Reporter: Martin Robinson <mrobinson>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Severity: Normal CC: dbates
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Description Flags
Code cleanup dbates: review+

Description Martin Robinson 2010-11-01 18:31:13 PDT
There is some very confusing code in PopupMenuGtk. This bug tracks cleaning it up.
Comment 1 Martin Robinson 2010-11-01 18:34:17 PDT
Created attachment 72615 [details]
Code cleanup
Comment 2 Daniel Bates 2010-11-01 18:35:32 PDT
Comment on attachment 72615 [details]
Code cleanup

Looks good to me. Yay! for the cleanup.
Comment 3 Martin Robinson 2010-11-01 18:46:14 PDT
Comment on attachment 72615 [details]
Code cleanup

Removing this from the commit-queue since dbates warned me that it could land uncleanly.
Comment 4 Martin Robinson 2010-11-01 21:34:47 PDT
Committed r71099: <http://trac.webkit.org/changeset/71099>