Skip to content

Commit

Permalink
core(constants): increase default maxWaitForFcp to 30s (#9509)
Browse files Browse the repository at this point in the history
  • Loading branch information
patrickhulce authored and brendankenny committed Aug 8, 2019
1 parent c04f1ab commit d8410d5
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 6 deletions.
4 changes: 2 additions & 2 deletions lighthouse-cli/test/cli/__snapshots__/index-test.js.snap
Original file line number Diff line number Diff line change
Expand Up @@ -1248,7 +1248,7 @@ Object {
"extraHeaders": null,
"gatherMode": false,
"locale": "en-US",
"maxWaitForFcp": 15000,
"maxWaitForFcp": 30000,
"maxWaitForLoad": 45000,
"onlyAudits": null,
"onlyCategories": null,
Expand Down Expand Up @@ -1382,7 +1382,7 @@ Object {
"extraHeaders": null,
"gatherMode": false,
"locale": "en-US",
"maxWaitForFcp": 15000,
"maxWaitForFcp": 30000,
"maxWaitForLoad": 45000,
"onlyAudits": Array [
"metrics",
Expand Down
2 changes: 1 addition & 1 deletion lighthouse-core/config/constants.js
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ const throttling = {
/** @type {LH.Config.Settings} */
const defaultSettings = {
output: 'json',
maxWaitForFcp: 15 * 1000,
maxWaitForFcp: 30 * 1000,
maxWaitForLoad: 45 * 1000,
throttlingMethod: 'simulate',
throttling: throttling.mobileSlow4G,
Expand Down
1 change: 1 addition & 0 deletions lighthouse-core/config/lr-desktop-config.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@
const config = {
extends: 'lighthouse:default',
settings: {
maxWaitForFcp: 15 * 1000,
maxWaitForLoad: 35 * 1000,
emulatedFormFactor: 'desktop',
throttling: {
Expand Down
1 change: 1 addition & 0 deletions lighthouse-core/config/lr-mobile-config.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@
const config = {
extends: 'lighthouse:default',
settings: {
maxWaitForFcp: 15 * 1000,
maxWaitForLoad: 35 * 1000,
// Skip the h2 audit so it doesn't lie to us. See https://github.com/GoogleChrome/lighthouse/issues/6539
skipAudits: ['uses-http2'],
Expand Down
4 changes: 2 additions & 2 deletions lighthouse-core/test/results/artifacts/artifacts.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
"output": [
"html"
],
"maxWaitForFcp": 15000,
"maxWaitForFcp": 30000,
"maxWaitForLoad": 45000,
"throttlingMethod": "devtools",
"throttling": {
Expand Down Expand Up @@ -2017,4 +2017,4 @@
"systemId": "",
"publicId": ""
}
}
}
2 changes: 1 addition & 1 deletion lighthouse-core/test/results/sample_v2.json
Original file line number Diff line number Diff line change
Expand Up @@ -3246,7 +3246,7 @@
"output": [
"json"
],
"maxWaitForFcp": 15000,
"maxWaitForFcp": 30000,
"maxWaitForLoad": 45000,
"throttlingMethod": "devtools",
"throttling": {
Expand Down

0 comments on commit d8410d5

Please sign in to comment.