Bug 1709379, don't special case picture in picture mode in tooltips as the button is still a mute button, r=mconley

Differential Revision: https://phabricator.services.mozilla.com/D115555
This commit is contained in:
Neil Deakin 2021-05-31 13:09:44 +00:00
Родитель 7c670aecf4
Коммит 5b9b440b1f
2 изменённых файлов: 1 добавлений и 15 удалений

Просмотреть файл

@ -453,16 +453,6 @@
if (this.multiselected) {
gBrowser.toggleMuteAudioOnMultiSelectedTabs(this);
} else {
if (
event.target.classList.contains("tab-icon-sound") &&
this.pictureinpicture
) {
// When Picture-in-Picture is open, we repurpose '.tab-icon-sound' as
// an inert Picture-in-Picture indicator, and expose the '.tab-icon-overlay'
// as the mechanism for muting the tab, so we don't need to handle clicks on
// '.tab-icon-sound' in this case.
return;
}
this.toggleMuteAudio();
}
return;

Просмотреть файл

@ -5392,11 +5392,7 @@
affectedTabsLength,
gTabBrowserBundle.GetStringFromName("tabs.closeTabs.tooltip")
).replace("#1", affectedTabsLength);
}
// When Picture-in-Picture is open, we repurpose '.tab-icon-sound' as
// an inert Picture-in-Picture indicator, so we should display
// the default tooltip
else if (tab._overPlayingIcon && !tab.pictureinpicture) {
} else if (tab._overPlayingIcon) {
let stringID;
if (tab.selected) {
stringID = tab.linkedBrowser.audioMuted