Print Preview: Mac: Do not blur focused item while opening the hamburger menu.

chrome/browser/resources/print_preview/search/destination_list.js

Index: chrome/browser/resources/print_preview/search/destination_list.js
diff --git a/chrome/browser/resources/print_preview/search/destination_list.js b/chrome/browser/resources/print_preview/search/destination_list.js
index 9b9647a00bf39eb3414b75b50c2300473f3cf6c5..d92cc07bedefb6b4479245f91a79d6b702621c8a 100644
--- a/chrome/browser/resources/print_preview/search/destination_list.js
+++ b/chrome/browser/resources/print_preview/search/destination_list.js
@@ -336,7 +336,8 @@ cr.define('print_preview', function() {
       if (focusedEl)
         itemEl.classList.add('moving');
       // Move it to the end of the list.
-      listEl.appendChild(itemEl);
+      if (listEl.lastChild != itemEl)

[dpapad, 2017/05/23 01:02:06]

Without a regression test, and without any comment explaining why this check is
necessary, this check is not obvious and it is likely that someone might try to
remove it in the future, and silently regress what this CL is fixing.

Can we at least add a comment?

[tkent, 2017/05/23 03:54:53]

I found why :focus didn't work, and Patch Set 2 removed this hacky |if|.

+        listEl.appendChild(itemEl);
       // Restore focus.
       if (focusedEl) {
         if (focusedEl == itemEl || focusedEl == focusedInnerEl)