MD WebUI: fix action menu initial focus to be more like context menus

chrome/test/data/webui/cr_elements/cr_action_menu_test.js

Index: chrome/test/data/webui/cr_elements/cr_action_menu_test.js
diff --git a/chrome/test/data/webui/cr_elements/cr_action_menu_test.js b/chrome/test/data/webui/cr_elements/cr_action_menu_test.js
index 15dd8e0da2cc483f4da159d6918634e5d4040342..635bd09c53790da842d12813214dc175c48d99f5 100644
--- a/chrome/test/data/webui/cr_elements/cr_action_menu_test.js
+++ b/chrome/test/data/webui/cr_elements/cr_action_menu_test.js
@@ -36,38 +36,43 @@ suite('CrActionMenu', function() {
       menu.close();
   });
 
-  test('focus after showing', function() {
+  function down() {
+    MockInteractions.keyDownOn(menu, 'ArrowDown', [], 'ArrowDown');
+  }
+
+  test('hidden or disabled items', function() {
     menu.showAt(document.querySelector('#dots'));
+    down();
     assertEquals(menu.root.activeElement, items[0]);
 
     menu.close();
     items[0].hidden = true;
     menu.showAt(document.querySelector('#dots'));
+    down();
     assertEquals(menu.root.activeElement, items[1]);
 
     menu.close();
-    items[1].hidden = true;
+    items[1].disabled = true;
     menu.showAt(document.querySelector('#dots'));
+    down();
     assertEquals(menu.root.activeElement, items[2]);
-
-    menu.close();
-    items[2].disabled = true;
-    menu.showAt(document.querySelector('#dots'));
-    assertEquals(null, menu.root.activeElement);
   });
 
   test('focus after down/up arrow', function() {
-    function down() {
-      MockInteractions.keyDownOn(menu, 'ArrowDown', [], 'ArrowDown');
-    }
-
     function up() {
       MockInteractions.keyDownOn(menu, 'ArrowUp', [], 'ArrowUp');
     }
 
     menu.showAt(document.querySelector('#dots'));
-    assertEquals(items[0], menu.root.activeElement);
 
+    // The menu should be focused when shown, but not on any of the items.
+    assertEquals(menu, document.activeElement);
+    assertNotEquals(items[0], menu.root.activeElement);

[dpapad, 2017/03/18 01:03:05]

Optional nit: What is the value of menu.root.activeElement at this point? Is it
null? Perhaps you can replace all 3 assertions with a single assertion.

assertEquals(null, menu.root.activeElement);

[Dan Beam, 2017/03/18 17:55:06]

it's not null, it's a wrapper in the shadow DOM and I don't think the test
should know about it (or at least it doesn't seem useful)

+    assertNotEquals(items[1], menu.root.activeElement);
+    assertNotEquals(items[2], menu.root.activeElement);
+
+    down();
+    assertEquals(items[0], menu.root.activeElement);
     down();
     assertEquals(items[1], menu.root.activeElement);
     down();