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 paulirish committed Nov 6, 2019
1 parent 4fd8873 commit f0ffaca
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 5 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
2 changes: 1 addition & 1 deletion 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
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 f0ffaca

Please sign in to comment.