GitLab

The ExtensionContextMenuModel looks at the action type in order to
determine if context menu items match. Instead of using a bespoke enum,
use ActionInfo::Type directly (with a base::Optional<> to indicate no
action).

This also:
- Removes a ternary that treated TYPE_ACTION the same as TYPE_BROWSER
- Removes the action type as a member variable, since its only used in
  one place.

Finally, this helps pave the way for introducing an "action" context
for menus.

Bug: 893373
Change-Id: Ibef23864c8a849f7248cd6b73ef1dfe728962a58
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2132531Reviewed-by: Istiaque Ahmed <lazyboy@chromium.org>
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#757595}