Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

core(lightwallet): Deprecate 'performance-budget' audit / add 'resource-budget' audit #9907

Closed
wants to merge 2 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions lighthouse-cli/test/cli/__snapshots__/index-test.js.snap
Original file line number Diff line number Diff line change
Expand Up @@ -126,6 +126,9 @@ Object {
Object {
"path": "offline-start-url",
},
Object {
"path": "resource-budget",
},
Object {
"path": "performance-budget",
},
Expand Down Expand Up @@ -884,6 +887,11 @@ Object {
"id": "performance-budget",
"weight": 0,
},
Object {
"group": "budgets",
"id": "resource-budget",
"weight": 0,
},
Object {
"group": "diagnostics",
"id": "resource-summary",
Expand Down
2 changes: 1 addition & 1 deletion lighthouse-cli/test/smokehouse/perf/expectations.js
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ module.exports = [
],
},
},
'performance-budget': {
'resource-budget': {
score: null,
details: {
// Undefined items are asserting that the property isn't included in the table item.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ const i18n = require('../lib/i18n/i18n.js');

const UIStrings = {
/** Title of a Lighthouse audit that compares the size and quantity of page resources against targets set by the user. These targets are thought of as "performance budgets" because these metrics impact page performance (i.e. how quickly a page loads). */
title: 'Performance budget',
title: 'Resource budget',
/** Description of a Lighthouse audit where a user sets budgets for the quantity and size of page resources. No character length limits. 'Learn More' becomes link text to additional documentation. */
description: 'Keep the quantity and size of network requests under the targets ' +
'set by the provided performance budget. [Learn more](https://developers.google.com/web/tools/lighthouse/audits/budgets).',
Expand All @@ -37,7 +37,7 @@ class ResourceBudget extends Audit {
*/
static get meta() {
return {
id: 'performance-budget',
id: 'resource-budget',
title: str_(UIStrings.title),
description: str_(UIStrings.description),
scoreDisplayMode: Audit.SCORING_MODES.INFORMATIVE,
Expand Down
2 changes: 2 additions & 0 deletions lighthouse-core/config/default-config.js
Original file line number Diff line number Diff line change
Expand Up @@ -210,6 +210,7 @@ const defaultConfig = {
'main-thread-tasks',
'metrics',
'offline-start-url',
'resource-budget',
'performance-budget',
'resource-summary',
'third-party-summary',
Expand Down Expand Up @@ -410,6 +411,7 @@ const defaultConfig = {
{id: 'mainthread-work-breakdown', weight: 0, group: 'diagnostics'},
{id: 'font-display', weight: 0, group: 'diagnostics'},
{id: 'performance-budget', weight: 0, group: 'budgets'},
{id: 'resource-budget', weight: 0, group: 'budgets'},
{id: 'resource-summary', weight: 0, group: 'diagnostics'},
{id: 'third-party-summary', weight: 0, group: 'diagnostics'},
// Audits past this point don't belong to a group and will not be shown automatically
Expand Down
12 changes: 0 additions & 12 deletions lighthouse-core/lib/i18n/locales/ar-XB.json
Original file line number Diff line number Diff line change
Expand Up @@ -815,18 +815,6 @@
"lighthouse-core/audits/offline-start-url.js | warningCantStart": {
"message": "‏‮Lighthouse‬‏ ‏‮couldn‬‏'‏‮t‬‏ ‏‮read‬‏ ‏‮the‬‏ `start_url` ‏‮from‬‏ ‏‮the‬‏ ‏‮manifest‬‏. ‏‮As‬‏ ‏‮a‬‏ ‏‮result‬‏, ‏‮the‬‏ `start_url` ‏‮was‬‏ ‏‮assumed‬‏ ‏‮to‬‏ ‏‮be‬‏ ‏‮the‬‏ ‏‮document‬‏'‏‮s‬‏ ‏‮URL‬‏. ‏‮Error‬‏ ‏‮message‬‏: '{manifestWarning}'."
},
"lighthouse-core/audits/performance-budget.js | columnOverBudget": {
"message": "‏‮Over‬‏ ‏‮Budget‬‏"
},
"lighthouse-core/audits/performance-budget.js | description": {
"message": "‏‮Keep‬‏ ‏‮the‬‏ ‏‮quantity‬‏ ‏‮and‬‏ ‏‮size‬‏ ‏‮of‬‏ ‏‮network‬‏ ‏‮requests‬‏ ‏‮under‬‏ ‏‮the‬‏ ‏‮targets‬‏ ‏‮set‬‏ ‏‮by‬‏ ‏‮the‬‏ ‏‮provided‬‏ ‏‮performance‬‏ ‏‮budget‬‏. [‏‮Learn‬‏ ‏‮more‬‏](https://developers.google.com/web/tools/lighthouse/audits/budgets)."
},
"lighthouse-core/audits/performance-budget.js | requestCountOverBudget": {
"message": "{count,plural, =1{1 ‏‮request‬‏}zero{# ‏‮requests‬‏}two{# ‏‮requests‬‏}few{# ‏‮requests‬‏}many{# ‏‮requests‬‏}other{# ‏‮requests‬‏}}"
},
"lighthouse-core/audits/performance-budget.js | title": {
"message": "‏‮Performance‬‏ ‏‮budget‬‏"
},
"lighthouse-core/audits/redirects-http.js | description": {
"message": "‏‮If‬‏ ‏‮you‬‏'‏‮ve‬‏ ‏‮already‬‏ ‏‮set‬‏ ‏‮up‬‏ ‏‮HTTPS‬‏, ‏‮make‬‏ ‏‮sure‬‏ ‏‮that‬‏ ‏‮you‬‏ ‏‮redirect‬‏ ‏‮all‬‏ ‏‮HTTP‬‏ ‏‮traffic‬‏ ‏‮to‬‏ ‏‮HTTPS‬‏ ‏‮in‬‏ ‏‮order‬‏ ‏‮to‬‏ ‏‮enable‬‏ ‏‮secure‬‏ ‏‮web‬‏ ‏‮features‬‏ ‏‮for‬‏ ‏‮all‬‏ ‏‮your‬‏ ‏‮users‬‏. [‏‮Learn‬‏ ‏‮more‬‏](https://web.dev/redirects-http)."
},
Expand Down
12 changes: 0 additions & 12 deletions lighthouse-core/lib/i18n/locales/ar.json
Original file line number Diff line number Diff line change
Expand Up @@ -815,18 +815,6 @@
"lighthouse-core/audits/offline-start-url.js | warningCantStart": {
"message": "تعذّر على Lighthouse قراءة `start_url` من البيان. ونتيجة لذلك، كان من المفترض أن يكون `start_url` هو عنوان URL للمستند. رسالة خطأ: '{manifestWarning}'."
},
"lighthouse-core/audits/performance-budget.js | columnOverBudget": {
"message": "تجاوز الميزانية"
},
"lighthouse-core/audits/performance-budget.js | description": {
"message": "يمكنك الحفاظ على كمية طلبات الشبكة وحجمها ضمن الاستهدافات المحدّدة في ميزانية الأداء المقدّمة. [مزيد من المعلومات](https://developers.google.com/web/tools/lighthouse/audits/budgets)"
},
"lighthouse-core/audits/performance-budget.js | requestCountOverBudget": {
"message": "{count,plural, =1{طلب واحد}zero{# طلب}two{طلبان (#)}few{# طلبات}many{# طلبًا}other{# طلب}}"
},
"lighthouse-core/audits/performance-budget.js | title": {
"message": "ميزانية الأداء"
},
"lighthouse-core/audits/redirects-http.js | description": {
"message": "في حال أعددت HTTPS مسبقًا، تأكّد من إعادة توجيه جميع زيارات HTTP إلى HTTPS من أجل تفعيل ميزات الويب الآمنة لجميع المستخدمين. [مزيد من المعلومات](https://web.dev/redirects-http)"
},
Expand Down
12 changes: 0 additions & 12 deletions lighthouse-core/lib/i18n/locales/bg.json
Original file line number Diff line number Diff line change
Expand Up @@ -815,18 +815,6 @@
"lighthouse-core/audits/offline-start-url.js | warningCantStart": {
"message": "Lighthouse не можа да прочете `start_url` от манифеста. В резултат на това бе предположено, че URL адресът на документа изпълнява функцията на `start_url`. Съобщение за грешка: „{manifestWarning}“."
},
"lighthouse-core/audits/performance-budget.js | columnOverBudget": {
"message": "Надхвърля бюджета"
},
"lighthouse-core/audits/performance-budget.js | description": {
"message": "Поддържайте количеството и обема на мрежовите заявки под целевите стойности в посочения бюджет за ефективността. [Научете повече](https://developers.google.com/web/tools/lighthouse/audits/budgets)."
},
"lighthouse-core/audits/performance-budget.js | requestCountOverBudget": {
"message": "{count,plural, =1{1 заявка}other{# заявки}}"
},
"lighthouse-core/audits/performance-budget.js | title": {
"message": "Бюджет за ефективността"
},
"lighthouse-core/audits/redirects-http.js | description": {
"message": "Ако вече сте настроили HTTPS, целият HTTP трафик трябва да се пренасочва към HTTPS, така че функциите за сигурност в мрежата да са активирани за всичките ви потребители. [Научете повече](https://web.dev/redirects-http)."
},
Expand Down
12 changes: 0 additions & 12 deletions lighthouse-core/lib/i18n/locales/ca.json
Original file line number Diff line number Diff line change
Expand Up @@ -815,18 +815,6 @@
"lighthouse-core/audits/offline-start-url.js | warningCantStart": {
"message": "Lighthouse no ha pogut llegir l'atribut `start_url` del fitxer de manifest, de manera que s'ha donat per fet que `start_url` era l'URL del document. Missatge d'error: \"{manifestWarning}\"."
},
"lighthouse-core/audits/performance-budget.js | columnOverBudget": {
"message": "Per sobre del pressupost"
},
"lighthouse-core/audits/performance-budget.js | description": {
"message": "Manté la quantitat i la mida de les sol·licituds de xarxa ajustades als objectius establerts al pressupost de rendiment que s'ha proporcionat. [Obtén més informació](https://developers.google.com/web/tools/lighthouse/audits/budgets)."
},
"lighthouse-core/audits/performance-budget.js | requestCountOverBudget": {
"message": "{count,plural, =1{1 sol·licitud}other{# sol·licituds}}"
},
"lighthouse-core/audits/performance-budget.js | title": {
"message": "Pressupost de rendiment"
},
"lighthouse-core/audits/redirects-http.js | description": {
"message": "Si ja has configurat el format HTTPS, assegura't de redirigir tot el trànsit HTTP cap a HTTPS per activar funcions web segures per a tots els usuaris. [Obtén més informació](https://web.dev/redirects-http)."
},
Expand Down
12 changes: 0 additions & 12 deletions lighthouse-core/lib/i18n/locales/cs.json
Original file line number Diff line number Diff line change
Expand Up @@ -815,18 +815,6 @@
"lighthouse-core/audits/offline-start-url.js | warningCantStart": {
"message": "Nástroji Lighthouse se v manifestu nepodařilo přečíst atribut `start_url`. Proto se předpokládá, že adresou URL dokumentu je `start_url`. Chybová zpráva: {manifestWarning}"
},
"lighthouse-core/audits/performance-budget.js | columnOverBudget": {
"message": "Nad rozpočet"
},
"lighthouse-core/audits/performance-budget.js | description": {
"message": "Udržujte množství a velikost síťových požadavků pod cílovými hodnotami, které udává poskytnutý rozpočet výkonu. [Další informace](https://developers.google.com/web/tools/lighthouse/audits/budgets)"
},
"lighthouse-core/audits/performance-budget.js | requestCountOverBudget": {
"message": "{count,plural, =1{1 požadavek}few{# požadavky}many{# požadavku}other{# požadavků}}"
},
"lighthouse-core/audits/performance-budget.js | title": {
"message": "Rozpočete výkonu"
},
"lighthouse-core/audits/redirects-http.js | description": {
"message": "Pokud jste protokol HTTPS už nastavili, nezapomeňte veškerý provoz přes protokol HTTP přesměrovat na HTTPS, aby všichni vaši uživatelé měli k dispozici funkce zabezpečeného webu. [Další informace](https://web.dev/redirects-http)"
},
Expand Down
12 changes: 0 additions & 12 deletions lighthouse-core/lib/i18n/locales/da.json
Original file line number Diff line number Diff line change
Expand Up @@ -815,18 +815,6 @@
"lighthouse-core/audits/offline-start-url.js | warningCantStart": {
"message": "Lighthouse kunne ikke læse `start_url` fra manifestet. `start_url` blev derfor betragtet som dokumentets webadresse. Fejlmeddelelse: \"{manifestWarning}\"."
},
"lighthouse-core/audits/performance-budget.js | columnOverBudget": {
"message": "Over budget"
},
"lighthouse-core/audits/performance-budget.js | description": {
"message": "Sørg for, at antallet af og størrelsen på netværksanmodningerne ikke overskrider målene i det angivne budget for ydeevne. [Få flere oplysninger](https://developers.google.com/web/tools/lighthouse/audits/budgets)."
},
"lighthouse-core/audits/performance-budget.js | requestCountOverBudget": {
"message": "{count,plural, =1{1 anmodning}one{# anmodning}other{# anmodninger}}"
},
"lighthouse-core/audits/performance-budget.js | title": {
"message": "Budget for ydeevne"
},
"lighthouse-core/audits/redirects-http.js | description": {
"message": "Hvis du allerede har konfigureret HTTPS, skal du sørge for at omdirigere al HTTP-trafik, så alle dine brugere får sikre webfunktioner. [Få flere oplysninger](https://web.dev/redirects-http)."
},
Expand Down
12 changes: 0 additions & 12 deletions lighthouse-core/lib/i18n/locales/de.json
Original file line number Diff line number Diff line change
Expand Up @@ -815,18 +815,6 @@
"lighthouse-core/audits/offline-start-url.js | warningCantStart": {
"message": "Lighthouse konnte die `start_url` nicht im Manifest abrufen. Daher wurde angenommen, dass es sich bei der `start_url` um die URL des Dokuments handelt. Fehlermeldung: \"{manifestWarning}\"."
},
"lighthouse-core/audits/performance-budget.js | columnOverBudget": {
"message": "Über dem Budget"
},
"lighthouse-core/audits/performance-budget.js | description": {
"message": "Die Anzahl und Größe der Netzwerkanfragen sollten unter den Zielvorgaben des Leistungsbudgets liegen. [Weitere Informationen.](https://developers.google.com/web/tools/lighthouse/audits/budgets)"
},
"lighthouse-core/audits/performance-budget.js | requestCountOverBudget": {
"message": "{count,plural, =1{1 Anfrage}other{# Anfragen}}"
},
"lighthouse-core/audits/performance-budget.js | title": {
"message": "Leistungsbudget"
},
"lighthouse-core/audits/redirects-http.js | description": {
"message": "Wenn Sie HTTPS bereits eingerichtet haben, sollten Sie den gesamten HTTP-Traffic auf HTTPS weiterleiten, damit für alle Ihre Nutzer sichere Webfunktionen aktiviert sind. [Weitere Informationen.](https://web.dev/redirects-http)"
},
Expand Down
12 changes: 0 additions & 12 deletions lighthouse-core/lib/i18n/locales/el.json
Original file line number Diff line number Diff line change
Expand Up @@ -815,18 +815,6 @@
"lighthouse-core/audits/offline-start-url.js | warningCantStart": {
"message": "Το Lighthouse δεν μπόρεσε να διαβάσει το `start_url` από το μανιφέστο. Επομένως, το `start_url` θεωρήθηκε ότι είναι το URL του εγγράφου. Μήνυμα σφάλματος: '{manifestWarning}'."
},
"lighthouse-core/audits/performance-budget.js | columnOverBudget": {
"message": "Υπέρβαση προϋπολογισμού"
},
"lighthouse-core/audits/performance-budget.js | description": {
"message": "Διατηρήστε την ποσότητα και το μέγεθος των αιτημάτων δικτύου εντός των στόχων που ορίζονται από τον παρεχόμενο προϋπολογισμό απόδοσης. [Μάθετε περισσότερα](https://developers.google.com/web/tools/lighthouse/audits/budgets)."
},
"lighthouse-core/audits/performance-budget.js | requestCountOverBudget": {
"message": "{count,plural, =1{1 αίτημα}other{# αιτήματα}}"
},
"lighthouse-core/audits/performance-budget.js | title": {
"message": "Προϋπολογισμός απόδοσης"
},
"lighthouse-core/audits/redirects-http.js | description": {
"message": "Αν έχετε ήδη ρυθμίσει το HTTPS, βεβαιωθείτε ότι ανακατευθύνετε όλη την επισκεψιμότητα HTTP σε HTTPS, προκειμένου να ενεργοποιήσετε ασφαλείς λειτουργίες ιστού για όλους τους χρήστες σας. [Μάθετε περισσότερα](https://web.dev/redirects-http)."
},
Expand Down
12 changes: 0 additions & 12 deletions lighthouse-core/lib/i18n/locales/en-GB.json
Original file line number Diff line number Diff line change
Expand Up @@ -815,18 +815,6 @@
"lighthouse-core/audits/offline-start-url.js | warningCantStart": {
"message": "Lighthouse couldn't read the `start_url` from the manifest. As a result, the `start_url` was assumed to be the document's URL. Error message: '{manifestWarning}'."
},
"lighthouse-core/audits/performance-budget.js | columnOverBudget": {
"message": "Over budget"
},
"lighthouse-core/audits/performance-budget.js | description": {
"message": "Keep the quantity and size of network requests under the targets set by the provided performance budget. [Learn more](https://developers.google.com/web/tools/lighthouse/audits/budgets)."
},
"lighthouse-core/audits/performance-budget.js | requestCountOverBudget": {
"message": "{count,plural, =1{1 request}other{# requests}}"
},
"lighthouse-core/audits/performance-budget.js | title": {
"message": "Performance budget"
},
"lighthouse-core/audits/redirects-http.js | description": {
"message": "If you've already set up HTTPS, make sure that you redirect all HTTP traffic to HTTPS in order to enable secure web features for all your users. [Learn more](https://web.dev/redirects-http)."
},
Expand Down
12 changes: 12 additions & 0 deletions lighthouse-core/lib/i18n/locales/en-US.json
Original file line number Diff line number Diff line change
Expand Up @@ -914,6 +914,18 @@
"lighthouse-core/audits/redirects.js | title": {
"message": "Avoid multiple page redirects"
},
"lighthouse-core/audits/resource-budget.js | columnOverBudget": {
"message": "Over Budget"
},
"lighthouse-core/audits/resource-budget.js | description": {
"message": "Keep the quantity and size of network requests under the targets set by the provided performance budget. [Learn more](https://developers.google.com/web/tools/lighthouse/audits/budgets)."
},
"lighthouse-core/audits/resource-budget.js | requestCountOverBudget": {
"message": "{count, plural,\n =1 {1 request}\n other {# requests}\n }"
},
"lighthouse-core/audits/resource-budget.js | title": {
"message": "Resource budget"
},
"lighthouse-core/audits/resource-summary.js | description": {
"message": "To set budgets for the quantity and size of page resources, add a budget.json file. [Learn more](https://developers.google.com/web/tools/lighthouse/audits/budgets)."
},
Expand Down
12 changes: 0 additions & 12 deletions lighthouse-core/lib/i18n/locales/en-XA.json
Original file line number Diff line number Diff line change
Expand Up @@ -815,18 +815,6 @@
"lighthouse-core/audits/offline-start-url.js | warningCantStart": {
"message": "[Ļîĝĥţĥöûšé çöûļðñ'ţ ŕéåð ţĥé ᐅ`start_url`ᐊ ƒŕöm ţĥé måñîƒéšţ. Åš å ŕéšûļţ, ţĥé ᐅ`start_url`ᐊ ŵåš åššûméð ţö бé ţĥé ðöçûméñţ'š ÛŔĻ. Éŕŕöŕ méššåĝé: 'ᐅ{manifestWarning}ᐊ'. one two three four five six seven eight nine ten eleven twelve thirteen fourteen fifteen sixteen seventeen eighteen nineteen twenty]"
},
"lighthouse-core/audits/performance-budget.js | columnOverBudget": {
"message": "[Övéŕ Бûðĝéţ one two]"
},
"lighthouse-core/audits/performance-budget.js | description": {
"message": "[Ķééþ ţĥé qûåñţîţý åñð šîžé öƒ ñéţŵöŕķ ŕéqûéšţš ûñðéŕ ţĥé ţåŕĝéţš šéţ бý ţĥé þŕövîðéð þéŕƒöŕmåñçé бûðĝéţ. ᐅ[ᐊĻéåŕñ möŕéᐅ](https://developers.google.com/web/tools/lighthouse/audits/budgets)ᐊ. one two three four five six seven eight nine ten eleven twelve thirteen fourteen fifteen sixteen seventeen eighteen nineteen]"
},
"lighthouse-core/audits/performance-budget.js | requestCountOverBudget": {
"message": "{count,plural, =1{[1 ŕéqûéšţ one two]}other{[# ŕéqûéšţš one two]}}"
},
"lighthouse-core/audits/performance-budget.js | title": {
"message": "[Þéŕƒöŕmåñçé бûðĝéţ one two three]"
},
"lighthouse-core/audits/redirects-http.js | description": {
"message": "[΃ ýöû'vé åļŕéåðý šéţ ûþ ĤŢŢÞŠ, måķé šûŕé ţĥåţ ýöû ŕéðîŕéçţ åļļ ĤŢŢÞ ţŕ僃îç ţö ĤŢŢÞŠ îñ öŕðéŕ ţö éñåбļé šéçûŕé ŵéб ƒéåţûŕéš ƒöŕ åļļ ýöûŕ ûšéŕš. ᐅ[ᐊĻéåŕñ möŕéᐅ](https://web.dev/redirects-http)ᐊ. one two three four five six seven eight nine ten eleven twelve thirteen fourteen fifteen sixteen seventeen eighteen nineteen twenty twentyone twentytwo twentythree]"
},
Expand Down
Loading