Bug 148461 - [Win] Simplify menu handling code in WinLauncher
Summary: [Win] Simplify menu handling code in WinLauncher
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: PC All
: P2 Normal
Assignee: Brent Fulgham
Depends on: 148451
  Show dependency treegraph
Reported: 2015-08-25 20:35 PDT by Brent Fulgham
Modified: 2015-08-26 10:34 PDT (History)
3 users (show)

See Also:

Patch (3.63 KB, patch)
2015-08-25 20:46 PDT, Brent Fulgham
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Brent Fulgham 2015-08-25 20:35:20 PDT
Said pointed out in a recent code review that the menu handler for WinLauncher duplicates a bunch of menu item checks with the main WinProc. This patch attempts to follow-up on that suggestion.
Comment 1 Brent Fulgham 2015-08-25 20:46:56 PDT
Created attachment 259917 [details]
Comment 2 Said Abou-Hallawa 2015-08-26 09:46:50 PDT
Looks good to me. Unofficial r=me. Thanks for cleaning this part of the code.
Comment 3 WebKit Commit Bot 2015-08-26 10:34:06 PDT
Comment on attachment 259917 [details]

Clearing flags on attachment: 259917

Committed r188969: <http://trac.webkit.org/changeset/188969>
Comment 4 WebKit Commit Bot 2015-08-26 10:34:09 PDT
All reviewed patches have been landed.  Closing bug.