Commit 908f1466 authored by fukino's avatar fukino Committed by Commit bot

Conform selector used in focus manager of history to <cr-menu> element.

The issue was caused by a oversight of crrev.com/805713003.
As the menu was not treated as focus parent, 'focus' event was dispatched with the text-field as an target. This make the menu hidden just after the menu is shown.
This CL corrects the selector to make the menu focus parent.

BUG=444480
TEST=go through repro steps written in the issue

Review URL: https://codereview.chromium.org/822663003

Cr-Commit-Position: refs/heads/master@{#309610}
parent 6fab03eb
......@@ -22,7 +22,7 @@ HistoryFocusManager.prototype = {
/** @override */
getFocusParent: function() {
return document.querySelector('#overlay .showing') ||
document.querySelector('menu:not([hidden])') ||
document.querySelector('cr-menu:not([hidden])') ||
$('history-page');
},
};
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment