Chromium Code Reviews
DescriptionFix wrench menu spacing regression
Fix a mistake I made in http://crrev.com/1003853004.
Before that CL, the "More tools" submenu was incorrectly using IDC_ZOOM_MENU
command, and I fixed that. But then I incorrectly added checks for
IDC_MORE_TOOLS_MENU wherever it was checking for IDC_ZOOM_MINUS, causing this
incorrect spacing bug. This removes those checks, but we still need a check
to make sure it is enabled, because when it was incorrectly using IDC_ZOOM_MENU,
it was getting enabled by BrowserCommandController.
BUG=468644
Committed: https://crrev.com/24ac1f518e765a9f07310123b98df4329ed845f2
Cr-Commit-Position: refs/heads/master@{#321413}
Patch Set 1 #
Messages
Total messages: 7 (2 generated)
|
|||||||||||||||||||