Bug 1709472 - Change error page favicon from warning.svg to info.svg. r=jaws

This also updates the icon used for the "focus tab by prompting" permission in
the permission panel to use info.svg.

Differential Revision: https://phabricator.services.mozilla.com/D114653
This commit is contained in:
Mike Conley 2021-05-17 18:56:01 +00:00
Родитель d492fcc9e4
Коммит 2d22fd4032
10 изменённых файлов: 10 добавлений и 10 удалений

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

@ -14,7 +14,7 @@
FAVICON_ERRORPAGE_URL symbol in
toolkit/components/places/src/nsFaviconService.h should be updated. -->
<link rel="icon" type="image/png" id="favicon"
href="chrome://global/skin/icons/warning.svg"/>
href="chrome://global/skin/icons/info.svg"/>
<link rel="localization" href="branding/brand.ftl"/>
<link rel="localization" href="browser/aboutRestartRequired.ftl"/>
</head>

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

@ -24,7 +24,7 @@
<link rel="stylesheet" href="chrome://browser/skin/aboutNetError.css" type="text/css" media="all" />
<!-- If the location of the favicon is changed here, the FAVICON_ERRORPAGE_URL symbol in
toolkit/components/places/src/nsFaviconService.h should be updated. -->
<link rel="icon" id="favicon" href="chrome://global/skin/icons/warning.svg"/>
<link rel="icon" id="favicon" href="chrome://global/skin/icons/info.svg"/>
<link rel="localization" href="browser/aboutCertError.ftl" />
<link rel="localization" href="browser/nsserrors.ftl" />
<link rel="localization" href="branding/brand.ftl"/>

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

@ -15,7 +15,7 @@
<title data-l10n-id="restore-page-tab-title"></title>
<link rel="stylesheet" href="chrome://global/skin/in-content/info-pages.css" type="text/css" media="all"/>
<link rel="stylesheet" href="chrome://browser/skin/aboutSessionRestore.css" type="text/css" media="all"/>
<link rel="icon" href="chrome://global/skin/icons/warning.svg"/>
<link rel="icon" href="chrome://global/skin/icons/info.svg"/>
<link rel="localization" href="browser/aboutSessionRestore.ftl"/>
<link rel="localization" href="branding/brand.ftl"/>
<script src="chrome://browser/content/aboutSessionRestore.js"/>

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

@ -55,7 +55,7 @@
/* INDIVIDUAL NOTIFICATIONS */
.focus-tab-by-prompt-icon {
list-style-image: url(chrome://global/skin/icons/warning.svg);
list-style-image: url(chrome://global/skin/icons/info.svg);
}
.popup-notification-icon[popupid="storage-access"],

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

@ -27,7 +27,7 @@
<link rel="stylesheet" href="chrome://global/skin/netError.css" type="text/css" media="all" />
<!-- If the location of the favicon is changed here, the FAVICON_ERRORPAGE_URL symbol in
toolkit/components/places/src/nsFaviconService.h should be updated. -->
<link rel="icon" id="favicon" href="chrome://global/skin/icons/warning.svg"/>
<link rel="icon" id="favicon" href="chrome://global/skin/icons/info.svg"/>
</head>
<body dir="&locale.dir;">

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

@ -13,7 +13,7 @@
<link rel="localization" href="toolkit/about/aboutHttpsOnlyError.ftl">
<!-- If the location of the favicon is changed here, the FAVICON_ERRORPAGE_URL symbol in
toolkit/components/places/src/nsFaviconService.h should be updated. -->
<link rel="icon" id="favicon" href="chrome://global/skin/icons/warning.svg"/>
<link rel="icon" id="favicon" href="chrome://global/skin/icons/info.svg"/>
<title data-l10n-id="about-httpsonly-title-connection-not-available"></title>
</head>
<body>

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

@ -331,6 +331,6 @@ interface nsIFaviconDataCallback : nsISupports
#define FAVICON_DEFAULT_URL "chrome://global/skin/icons/defaultFavicon.svg"
#define FAVICON_DEFAULT_MIMETYPE "image/svg+xml"
#define FAVICON_ERRORPAGE_URL "chrome://global/skin/icons/warning.svg"
#define FAVICON_ERRORPAGE_URL "chrome://global/skin/icons/info.svg"
%}

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

@ -20,7 +20,7 @@ const systemPrincipal = Services.scriptSecurityManager.getSystemPrincipal();
// This error icon must stay in sync with FAVICON_ERRORPAGE_URL in
// nsIFaviconService.idl, aboutCertError.xhtml and netError.xhtml.
const FAVICON_ERRORPAGE_URI = Services.io.newURI(
"chrome://global/skin/icons/warning.svg"
"chrome://global/skin/icons/info.svg"
);
/**

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

@ -111,7 +111,7 @@ add_task(async function() {
// This error icon must stay in sync with FAVICON_ERRORPAGE_URL in
// nsIFaviconService.idl, aboutCertError.xhtml and netError.xhtml.
let faviconErrorPageURI = Services.io.newURI(
"chrome://global/skin/icons/warning.svg"
"chrome://global/skin/icons/info.svg"
);
pageURI = Services.io.newURI("http://example.com/errorIcon");
await PlacesTestUtils.addVisits({

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

@ -185,7 +185,7 @@ add_task(async function test_import_chromefavicon() {
const PAGE_URI = NetUtil.newURI("http://example.com/chromefavicon_page");
const CHROME_FAVICON_URI = NetUtil.newURI(
"chrome://global/skin/icons/info.svg"
"chrome://global/skin/icons/warning.svg"
);
const CHROME_FAVICON_URI_2 = NetUtil.newURI(
"chrome://global/skin/icons/error.svg"