Skip to content

Commit 2511b06

Browse files
authored
core(apple-touch-icon): remove audit (#14243)
1 parent 15c4236 commit 2511b06

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

65 files changed

+295
-1167
lines changed

build/build-sample-reports.js

-8
Original file line numberDiff line numberDiff line change
@@ -191,14 +191,6 @@ async function generateErrorLHR() {
191191
offscreenImagesAudit.errorMessage = undefined;
192192
offscreenImagesAudit.scoreDisplayMode = 'binary';
193193
offscreenImagesAudit.score = 1;
194-
// pwa-apple-touch-icon - set as passing but with a warning
195-
const appleTouchIconAudit = errorLhr.audits['apple-touch-icon'];
196-
appleTouchIconAudit.warnings = [
197-
'`apple-touch-icon-precomposed` is out of date; `apple-touch-icon` is preferred.',
198-
];
199-
appleTouchIconAudit.errorMessage = undefined;
200-
appleTouchIconAudit.scoreDisplayMode = 'binary';
201-
appleTouchIconAudit.score = 1;
202194

203195
fs.rmSync(TMP, {recursive: true, force: true});
204196
return errorLhr;

lighthouse-cli/test/smokehouse/test-definitions/pwa-airhorner.js

-3
Original file line numberDiff line numberDiff line change
@@ -44,9 +44,6 @@ const expectations = {
4444
'content-width': {
4545
score: 1,
4646
},
47-
'apple-touch-icon': {
48-
score: 1,
49-
},
5047

5148
// "manual" audits. Just verify in the results.
5249
'pwa-cross-browser': {

lighthouse-cli/test/smokehouse/test-definitions/pwa-caltrain.js

-3
Original file line numberDiff line numberDiff line change
@@ -43,9 +43,6 @@ const expectations = {
4343
'content-width': {
4444
score: 1,
4545
},
46-
'apple-touch-icon': {
47-
score: 1,
48-
},
4946

5047
// "manual" audits. Just verify in the results.
5148
'pwa-cross-browser': {

lighthouse-cli/test/smokehouse/test-definitions/pwa-chromestatus.js

-3
Original file line numberDiff line numberDiff line change
@@ -40,9 +40,6 @@ const expectations = {
4040
'content-width': {
4141
score: 1,
4242
},
43-
'apple-touch-icon': {
44-
score: 1,
45-
},
4643

4744
// "manual" audits. Just verify in the results.
4845
'pwa-cross-browser': {

lighthouse-cli/test/smokehouse/test-definitions/pwa-rocks.js

-3
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,6 @@ const expectations = {
4848
'content-width': {
4949
score: 1,
5050
},
51-
'apple-touch-icon': {
52-
score: 1,
53-
},
5451

5552
// "manual" audits. Just verify in the results.
5653
'pwa-cross-browser': {

lighthouse-cli/test/smokehouse/test-definitions/pwa-svgomg.js

-3
Original file line numberDiff line numberDiff line change
@@ -52,9 +52,6 @@ const expectations = {
5252
'content-width': {
5353
score: 1,
5454
},
55-
'apple-touch-icon': {
56-
score: 1,
57-
},
5855

5956
// "manual" audits. Just verify in the results.
6057
'pwa-cross-browser': {

lighthouse-core/audits/apple-touch-icon.js

-72
This file was deleted.

lighthouse-core/config/default-config.js

-2
Original file line numberDiff line numberDiff line change
@@ -298,7 +298,6 @@ const defaultConfig = {
298298
'critical-request-chains',
299299
'redirects',
300300
'installable-manifest',
301-
'apple-touch-icon',
302301
'splash-screen',
303302
'themed-omnibox',
304303
'maskable-icon',
@@ -723,7 +722,6 @@ const defaultConfig = {
723722
{id: 'themed-omnibox', weight: 1, group: 'pwa-optimized'},
724723
{id: 'content-width', weight: 1, group: 'pwa-optimized'},
725724
{id: 'viewport', weight: 2, group: 'pwa-optimized'},
726-
{id: 'apple-touch-icon', weight: 1, group: 'pwa-optimized'},
727725
{id: 'maskable-icon', weight: 1, group: 'pwa-optimized'},
728726
// Manual audits
729727
{id: 'pwa-cross-browser', weight: 0},

lighthouse-core/test/audits/apple-touch-icon-test.js

-95
This file was deleted.

lighthouse-core/test/config/config-test.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -1257,7 +1257,7 @@ describe('Config', () => {
12571257
extends: 'lighthouse:default',
12581258
settings: {
12591259
onlyCategories: ['pwa'],
1260-
onlyAudits: ['apple-touch-icon'],
1260+
onlyAudits: ['viewport'],
12611261
},
12621262
};
12631263
const config = await Config.fromJson(extended);

0 commit comments

Comments
 (0)