Bug 53866

Summary: [chromium] Remove unused "external" case from PopupMenuChromium
Product: WebKit Reporter: Jay Civelli <jcivelli>
Component: WebKit Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, fishd
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
Patch
none
Patch for landing none

Description Jay Civelli 2011-02-05 23:34:59 PST 
In the PopupMenuChromium.[h,cpp] files, we still have the code for the "external" case, which was used previously on the Mac.
That code is not used anymore (see https://bugs.webkit.org/show_bug.cgi?id=46016) and should be removed.
Comment 1 Jay Civelli 2011-02-05 23:43:36 PST 
Created attachment 81392 [details]
Patch
Comment 2 Jay Civelli 2011-02-07 13:01:33 PST 
Created attachment 81510 [details]
Patch for landing
Comment 3 WebKit Commit Bot 2011-02-07 22:46:03 PST 
Comment on attachment 81510 [details]
Patch for landing

Clearing flags on attachment: 81510

Committed r77904: <http://trac.webkit.org/changeset/77904>
Comment 4 WebKit Commit Bot 2011-02-07 22:46:09 PST 
All reviewed patches have been landed.  Closing bug.