Bug 1552716 - Part 1: Fix the keyboard navigation of the tracking protection icon. r=Jamie

This patch fixes the issue that the keyboard navigation doesn't work for
the tracking protection icon.

Differential Revision: https://phabricator.services.mozilla.com/D39329

--HG--
extra : moz-landing-system : lando

browser/base/content/browser.css

@@ -655,7 +655,8 @@ toolbar:not(#TabsToolbar) > #personal-bookmarks {
   display: none;
 }
 
-#nav-bar:not([keyNav=true]) #identity-box {
+#nav-bar:not([keyNav=true]) #identity-box,
+#nav-bar:not([keyNav=true]) #tracking-protection-icon-container {
   -moz-user-focus: normal;
 }
 

browser/base/content/browser.xhtml

@@ -847,6 +847,7 @@
               <!-- Use onclick instead of normal popup= syntax since the popup
                    code fires onmousedown, and hence eats our favicon drag events. -->
               <box id="tracking-protection-icon-container" align="center"
+                   role="button"
                    onclick="gProtectionsHandler.handleProtectionsButtonEvent(event);"
                    onkeypress="gProtectionsHandler.handleProtectionsButtonEvent(event);">
                 <box id="tracking-protection-icon-box" animationsenabled="true">

browser/themes/osx/browser.css

@@ -267,7 +267,8 @@
 
 %include ../shared/identity-block/identity-block.inc.css
 
-#identity-box:not(:active):-moz-focusring {
+#identity-box:not(:active):-moz-focusring,
+#tracking-protection-icon-container:not(:active):-moz-focusring {
   border-inline-end-style: none;
 }
 

browser/themes/shared/identity-block/identity-block.inc.css

@@ -46,7 +46,8 @@
   fill-opacity: .8;
 }
 
-#identity-box:not(:active):-moz-focusring {
+#identity-box:not(:active):-moz-focusring,
+#tracking-protection-icon-container:not(:active):-moz-focusring {
   outline: var(--toolbarbutton-focus-outline);
   outline-offset: -2px;
   -moz-outline-radius: var(--toolbarbutton-border-radius);