Bug 48816

Summary: [GTK] PopupMenuGtk has a very confusing if-else block
Product: WebKit Reporter: Martin Robinson <mrobinson>
Component: WebKitGTKAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: dbates
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
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.
r=me
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>