With #secondary-ui-md turned on, or with MacViews, the Page Info bubble on desktop uses |Combobox|es instead of |MenuButton|s. There is currently a bug in the code used for |Combobox|es that causes any permission which has a different number to the default three options (Ask, Allow, and Block) to display the incorrect menu item string for the corresponding |ContentSetting| that is currently selected. For example, this previously caused the Ads content setting to show up as 'Allow' even though it was set to 'Block' (see issue 789587 for more context).
Since there are currently no content settings which have a different number of menu items in their |Combobox| drop-down, this is OK because this bug will never get hit. Obviously if this ever changes, this should be fixed.
Probably the fix should involve relying on the |command_id|s (which would be the ContentSetting enum values) given to the Combobox instead of the currently selected index. ComboboxModelAdapter::GetCheckedIndex() would probably be a good place to do this since it checks |IsCommandIdChecked()| (but assumes that |command_id| is the same as the combobox checked index).
Comment 1 by robliao@chromium.org
, Sep 13