fixup! fix(cdk-experimental/menu) make trigger items behave like normal items when pressing left/right

Author: mmalerba

src/cdk-experimental/menu/menu-bar.ts

@@ -26,9 +26,6 @@ import {PointerFocusTracker} from './pointer-focus-tracker';
 import {MENU_AIM, MenuAim} from './menu-aim';
 import {CdkMenuBase} from './menu-base';
 
-// TODO(mmalerba): Keyboard controls should match
-//   https://www.w3.org/TR/wai-aria-practices-1.1/examples/menubar/menubar-1/menubar-1.html#
-
 /**
  * Directive applied to an element which configures it as a MenuBar by setting the appropriate
  * role, aria attributes, and accessible keyboard and mouse handling logic. The component that

src/cdk-experimental/menu/menu.ts

@@ -39,9 +39,6 @@ import {MENU_AIM, MenuAim} from './menu-aim';
 import {MENU_TRIGGER, MenuTrigger} from './menu-trigger';
 import {CdkMenuBase} from './menu-base';
 
-// TODO(mmalerba): Keyboard controls should match
-//   https://www.w3.org/TR/wai-aria-practices-1.1/examples/menubar/menubar-1/menubar-1.html#
-
 /**
  * Directive which configures the element as a Menu which should contain child elements marked as
  * CdkMenuItem or CdkMenuGroup. Sets the appropriate role and aria-attributes for a menu and
@@ -68,10 +65,9 @@ import {CdkMenuBase} from './menu-base';
 })
 export class CdkMenu extends CdkMenuBase implements AfterContentInit, OnDestroy {
   /** Event emitted when the menu is closed. */
-  @Output() readonly closed: EventEmitter<void | 'click' | 'tab' | 'escape'> = new EventEmitter();
+  @Output() readonly closed: EventEmitter<void> = new EventEmitter();
 
   /** List of nested CdkMenuGroup elements */
-  // TODO(mmalerba): Does it make sense for menubars to have groups as well?
   @ContentChildren(CdkMenuGroup, {descendants: true})
   private readonly _nestedGroups: QueryList<CdkMenuGroup>;
 
@@ -84,8 +80,6 @@ export class CdkMenu extends CdkMenuBase implements AfterContentInit, OnDestroy
     @Optional() dir?: Directionality,
   ) {
     super(elementRef, menuStack, dir);
-    // TODO(mmalerba): This is matching what we were already doing, but judging
-    //  by the signature of `closed`, it looks like maybe we want to emit a reason?
     this.destroyed.subscribe(this.closed);
     this.menuStack?.push(this);
     this._parentTrigger?.registerChildMenu(this);