lighthouse 9.6.8 → 10.0.0-dev.20230210

package/lighthouse-core/audits/csp-xss.js

@@ -1,186 +0,0 @@
-/**
- * @license Copyright 2021 The Lighthouse Authors. All Rights Reserved.
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
- * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
- */
-'use strict';
-
-const Audit = require('./audit.js');
-const MainResource = require('../computed/main-resource.js');
-const i18n = require('../lib/i18n/i18n.js');
-const {
-  evaluateRawCspsForXss,
-  getTranslatedDescription,
-} = require('../lib/csp-evaluator.js');
-
-/** @typedef {import('../lib/csp-evaluator.js').Finding} Finding */
-
-const UIStrings = {
-  /** Title of a Lighthouse audit that evaluates the security of a page's CSP. "CSP" stands for "Content Security Policy". "XSS" stands for "Cross Site Scripting". "CSP" and "XSS" do not need to be translated. */
-  title: 'Ensure CSP is effective against XSS attacks',
-  /** Description of a Lighthouse audit that evaluates the security of a page's CSP. This is displayed after a user expands the section to see more. No character length limits. 'Learn More' becomes link text to additional documentation. "CSP" stands for "Content Security Policy". "XSS" stands for "Cross Site Scripting". "CSP" and "XSS" do not need to be translated. */
-  description: 'A strong Content Security Policy (CSP) significantly ' +
-    'reduces the risk of cross-site scripting (XSS) attacks. ' +
-    '[Learn more](https://web.dev/csp-xss/)',
-  /** Summary text for the results of a Lighthouse audit that evaluates the security of a page's CSP. This is displayed if no CSP is being enforced. "CSP" stands for "Content Security Policy". "CSP" does not need to be translated. */
-  noCsp: 'No CSP found in enforcement mode',
-  /** Message shown when one or more CSPs are defined in a <meta> tag. Shown in a table with a list of other CSP bypasses and warnings. "CSP" stands for "Content Security Policy". "CSP" and "HTTP" do not need to be translated. */
-  metaTagMessage: 'The page contains a CSP defined in a <meta> tag. ' +
-    'Consider defining the CSP in an HTTP header if you can.',
-  /** Label for a column in a data table; entries will be a directive of a CSP. "CSP" stands for "Content Security Policy". */
-  columnDirective: 'Directive',
-  /** Label for a column in a data table; entries will be the severity of an issue with the CSP. "CSP" stands for "Content Security Policy". */
-  columnSeverity: 'Severity',
-  /** Table item value calling out the presence of a syntax error. */
-  itemSeveritySyntax: 'Syntax',
-};
-
-const str_ = i18n.createMessageInstanceIdFn(__filename, UIStrings);
-
-class CspXss extends Audit {
-  /**
-   * @return {LH.Audit.Meta}
-   */
-  static get meta() {
-    return {
-      id: 'csp-xss',
-      scoreDisplayMode: Audit.SCORING_MODES.INFORMATIVE,
-      title: str_(UIStrings.title),
-      description: str_(UIStrings.description),
-      requiredArtifacts: ['devtoolsLogs', 'MetaElements', 'URL'],
-    };
-  }
-
-  /**
-   * @param {LH.Artifacts} artifacts
-   * @param {LH.Audit.Context} context
-   * @return {Promise<{cspHeaders: string[], cspMetaTags: string[]}>}
-   */
-  static async getRawCsps(artifacts, context) {
-    const devtoolsLog = artifacts.devtoolsLogs[Audit.DEFAULT_PASS];
-    const mainResource = await MainResource.request({devtoolsLog, URL: artifacts.URL}, context);
-
-    const cspMetaTags = artifacts.MetaElements
-      .filter(m => {
-        return m.httpEquiv && m.httpEquiv.toLowerCase() === 'content-security-policy';
-      })
-      .flatMap(m => (m.content || '').split(','))
-      .filter(rawCsp => rawCsp.replace(/\s/g, ''));
-    const cspHeaders = mainResource.responseHeaders
-      .filter(h => {
-        return h.name.toLowerCase() === 'content-security-policy';
-      })
-      .flatMap(h => h.value.split(','))
-      .filter(rawCsp => rawCsp.replace(/\s/g, ''));
-
-    return {cspHeaders, cspMetaTags};
-  }
-
-  /**
-   * @param {Finding} finding
-   * @param {LH.IcuMessage=} severity
-   * @return {LH.Audit.Details.TableItem}
-   */
-  static findingToTableItem(finding, severity) {
-    return {
-      directive: finding.directive,
-      description: getTranslatedDescription(finding),
-      severity,
-    };
-  }
-
-  /**
-   * @param {Finding[][]} syntaxFindings
-   * @param {string[]} rawCsps
-   * @return {LH.Audit.Details.TableItem[]}
-   */
-  static constructSyntaxResults(syntaxFindings, rawCsps) {
-    /** @type {LH.Audit.Details.TableItem[]} */
-    const results = [];
-
-    for (let i = 0; i < syntaxFindings.length; ++i) {
-      const items = syntaxFindings[i].map(f => this.findingToTableItem(f));
-      if (!items.length) continue;
-      results.push({
-        severity: str_(UIStrings.itemSeveritySyntax),
-        description: {
-          type: 'code',
-          value: rawCsps[i],
-        },
-        subItems: {
-          type: 'subitems',
-          items,
-        },
-      });
-    }
-
-    return results;
-  }
-
-  /**
-   * @param {string[]} cspHeaders
-   * @param {string[]} cspMetaTags
-   * @return {{score: number, results: LH.Audit.Details.TableItem[]}}
-   */
-  static constructResults(cspHeaders, cspMetaTags) {
-    const rawCsps = [...cspHeaders, ...cspMetaTags];
-    if (!rawCsps.length) {
-      return {
-        score: 0,
-        results: [{
-          severity: str_(i18n.UIStrings.itemSeverityHigh),
-          description: str_(UIStrings.noCsp),
-          directive: undefined,
-        }],
-      };
-    }
-
-    const {bypasses, warnings, syntax} = evaluateRawCspsForXss(rawCsps);
-
-    const results = [
-      ...this.constructSyntaxResults(syntax, rawCsps),
-      ...bypasses.map(f => this.findingToTableItem(f, str_(i18n.UIStrings.itemSeverityHigh))),
-      ...warnings.map(f => this.findingToTableItem(f, str_(i18n.UIStrings.itemSeverityMedium))),
-    ];
-
-    // Add extra warning for a CSP defined in a meta tag.
-    if (cspMetaTags.length) {
-      results.push({
-        severity: str_(i18n.UIStrings.itemSeverityMedium),
-        description: str_(UIStrings.metaTagMessage),
-        directive: undefined,
-      });
-    }
-
-    return {score: bypasses.length ? 0 : 1, results};
-  }
-
-  /**
-   * @param {LH.Artifacts} artifacts
-   * @param {LH.Audit.Context} context
-   * @return {Promise<LH.Audit.Product>}
-   */
-  static async audit(artifacts, context) {
-    const {cspHeaders, cspMetaTags} = await this.getRawCsps(artifacts, context);
-    const {score, results} = this.constructResults(cspHeaders, cspMetaTags);
-
-    /** @type {LH.Audit.Details.Table['headings']} */
-    const headings = [
-      /* eslint-disable max-len */
-      {key: 'description', itemType: 'text', subItemsHeading: {key: 'description'}, text: str_(i18n.UIStrings.columnDescription)},
-      {key: 'directive', itemType: 'code', subItemsHeading: {key: 'directive'}, text: str_(UIStrings.columnDirective)},
-      {key: 'severity', itemType: 'text', subItemsHeading: {key: 'severity'}, text: str_(UIStrings.columnSeverity)},
-      /* eslint-enable max-len */
-    ];
-    const details = Audit.makeTableDetails(headings, results);
-
-    return {
-      score,
-      notApplicable: !results.length,
-      details,
-    };
-  }
-}
-
-module.exports = CspXss;
-module.exports.UIStrings = UIStrings;

package/lighthouse-core/audits/deprecations.js

@@ -1,115 +0,0 @@
-/**
- * @license Copyright 2017 The Lighthouse Authors. All Rights Reserved.
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
- * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
- */
-'use strict';
-
-/**
- * @fileoverview Audits a page to determine if it is calling deprecated APIs.
- */
-
-const Audit = require('./audit.js');
-const JsBundles = require('../computed/js-bundles.js');
-const i18n = require('../lib/i18n/i18n.js');
-const {getIssueDetailDescription} = require('../lib/deprecations-strings.js');
-
-/* eslint-disable max-len */
-const UIStrings = {
-  /** Title of a Lighthouse audit that provides detail on the use of deprecated APIs. This descriptive title is shown to users when the page does not use deprecated APIs. */
-  title: 'Avoids deprecated APIs',
-  /** Title of a Lighthouse audit that provides detail on the use of deprecated APIs. This descriptive title is shown to users when the page uses deprecated APIs. */
-  failureTitle: 'Uses deprecated APIs',
-  /** Description of a Lighthouse audit that tells the user why they should not use deprecated APIs on their page. This is displayed after a user expands the section to see more. No character length limits. 'Learn More' becomes link text to additional documentation. */
-  description: 'Deprecated APIs will eventually be removed from the browser. ' +
-      '[Learn more](https://web.dev/deprecations/).',
-  /** [ICU Syntax] Label for the audit identifying the number of warnings generated by using deprecated APIs. */
-  displayValue: `{itemCount, plural,
-    =1 {1 warning found}
-    other {# warnings found}
-    }`,
-  /** Header of the table column which displays the warning message describing use of a deprecated API by code running in the web page. */
-  columnDeprecate: 'Deprecation / Warning',
-  /** Table column header for line of code (eg. 432) that is using a deprecated API. */
-  columnLine: 'Line',
-};
-/* eslint-enable max-len */
-
-const str_ = i18n.createMessageInstanceIdFn(__filename, UIStrings);
-
-class Deprecations extends Audit {
-  /**
-   * @return {LH.Audit.Meta}
-   */
-  static get meta() {
-    return {
-      id: 'deprecations',
-      title: str_(UIStrings.title),
-      failureTitle: str_(UIStrings.failureTitle),
-      description: str_(UIStrings.description),
-      requiredArtifacts: ['InspectorIssues', 'SourceMaps', 'ScriptElements'],
-    };
-  }
-
-  /**
-   * @param {LH.Artifacts} artifacts
-   * @param {LH.Audit.Context} context
-   * @return {Promise<LH.Audit.Product>}
-   */
-  static async audit(artifacts, context) {
-    const bundles = await JsBundles.request(artifacts, context);
-
-    const deprecations = artifacts.InspectorIssues.deprecationIssue
-      .map(deprecation => {
-        const {url, lineNumber, columnNumber} = deprecation.sourceCodeLocation;
-        const bundle = bundles.find(bundle => bundle.script.src === url);
-        const deprecationMeta = getIssueDetailDescription(deprecation);
-
-        /** @type {LH.Audit.Details.TableSubItems=} */
-        let subItems = undefined;
-        if (deprecationMeta.links.length) {
-          subItems = {
-            type: 'subitems',
-            items: deprecationMeta.links.map(link => ({
-              type: 'link',
-              url: link.link,
-              text: link.linkTitle,
-            })),
-          };
-        }
-
-        // @ts-expect-error: The english string used to be included on the protocol, but no longer is.
-        const legacyMessage = deprecation.message;
-
-        /** @type {LH.Audit.Details.TableItem} */
-        const item = {
-          value: deprecationMeta.message || legacyMessage || deprecation.type,
-          // Protocol.Audits.SourceCodeLocation.columnNumber is 1-indexed, but we use 0-indexed.
-          source: Audit.makeSourceLocation(url, lineNumber, columnNumber - 1, bundle),
-          subItems,
-        };
-        return item;
-      });
-
-    /** @type {LH.Audit.Details.Table['headings']} */
-    const headings = [
-      {key: 'value', itemType: 'text', text: str_(UIStrings.columnDeprecate)},
-      {key: 'source', itemType: 'source-location', text: str_(i18n.UIStrings.columnSource)},
-    ];
-    const details = Audit.makeTableDetails(headings, deprecations);
-
-    let displayValue;
-    if (deprecations.length > 0) {
-      displayValue = str_(UIStrings.displayValue, {itemCount: deprecations.length});
-    }
-
-    return {
-      score: Number(deprecations.length === 0),
-      displayValue,
-      details,
-    };
-  }
-}
-
-module.exports = Deprecations;
-module.exports.UIStrings = UIStrings;

package/lighthouse-core/audits/diagnostics.js

@@ -1,79 +0,0 @@
-/**
- * @license Copyright 2019 The Lighthouse Authors. All Rights Reserved.
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
- * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
- */
-'use strict';
-
-const Audit = require('./audit.js');
-const MainThreadTasksComputed = require('../computed/main-thread-tasks.js');
-const NetworkRecordsComputed = require('../computed/network-records.js');
-const NetworkAnalysisComputed = require('../computed/network-analysis.js');
-const NetworkAnalyzer = require('../lib/dependency-graph/simulator/network-analyzer.js');
-
-class Diagnostics extends Audit {
-  /**
-   * @return {LH.Audit.Meta}
-   */
-  static get meta() {
-    return {
-      id: 'diagnostics',
-      scoreDisplayMode: Audit.SCORING_MODES.INFORMATIVE,
-      title: 'Diagnostics',
-      description: 'Collection of useful page vitals.',
-      supportedModes: ['navigation'],
-      requiredArtifacts: ['traces', 'devtoolsLogs'],
-    };
-  }
-
-  /**
-   * @param {LH.Artifacts} artifacts
-   * @param {LH.Audit.Context} context
-   * @return {Promise<LH.Audit.Product>}
-   */
-  static async audit(artifacts, context) {
-    const trace = artifacts.traces[Audit.DEFAULT_PASS];
-    const devtoolsLog = artifacts.devtoolsLogs[Audit.DEFAULT_PASS];
-    const tasks = await MainThreadTasksComputed.request(trace, context);
-    const records = await NetworkRecordsComputed.request(devtoolsLog, context);
-    const analysis = await NetworkAnalysisComputed.request(devtoolsLog, context);
-
-    const toplevelTasks = tasks.filter(t => !t.parent);
-    const mainDocumentTransferSize = NetworkAnalyzer.findMainDocument(records).transferSize;
-    const totalByteWeight = records.reduce((sum, r) => sum + (r.transferSize || 0), 0);
-    const totalTaskTime = toplevelTasks.reduce((sum, t) => sum + (t.duration || 0), 0);
-    const maxRtt = Math.max(...analysis.additionalRttByOrigin.values()) + analysis.rtt;
-    const maxServerLatency = Math.max(...analysis.serverResponseTimeByOrigin.values());
-
-    const diagnostics = {
-      numRequests: records.length,
-      numScripts: records.filter(r => r.resourceType === 'Script').length,
-      numStylesheets: records.filter(r => r.resourceType === 'Stylesheet').length,
-      numFonts: records.filter(r => r.resourceType === 'Font').length,
-      numTasks: toplevelTasks.length,
-      numTasksOver10ms: toplevelTasks.filter(t => t.duration > 10).length,
-      numTasksOver25ms: toplevelTasks.filter(t => t.duration > 25).length,
-      numTasksOver50ms: toplevelTasks.filter(t => t.duration > 50).length,
-      numTasksOver100ms: toplevelTasks.filter(t => t.duration > 100).length,
-      numTasksOver500ms: toplevelTasks.filter(t => t.duration > 500).length,
-      rtt: analysis.rtt,
-      throughput: analysis.throughput,
-      maxRtt,
-      maxServerLatency,
-      totalByteWeight,
-      totalTaskTime,
-      mainDocumentTransferSize,
-    };
-
-    return {
-      score: 1,
-      details: {
-        type: 'debugdata',
-        // TODO: Consider not nesting diagnostics under `items`.
-        items: [diagnostics],
-      },
-    };
-  }
-}
-
-module.exports = Diagnostics;

package/lighthouse-core/audits/dobetterweb/charset.js

@@ -1,96 +0,0 @@
-/**
- * @license Copyright 2020 The Lighthouse Authors. All Rights Reserved.
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
- * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
- */
-
-/**
- * @fileoverview Audits a page to ensure charset it configured properly.
- * It must be defined within the first 1024 bytes of the HTML document, defined in the HTTP header, or the document source must start with a BOM.
- *
- * @see https://github.com/GoogleChrome/lighthouse/issues/10023
- */
-'use strict';
-
-const Audit = require('../audit.js');
-const i18n = require('../../lib/i18n/i18n.js');
-const MainResource = require('../../computed/main-resource.js');
-
-const UIStrings = {
-  /** Title of a Lighthouse audit that provides detail on if the charset is set properly for a page. This title is shown when the charset is defined correctly. Charset defines the character encoding (eg UTF-8) of the page content. */
-  title: 'Properly defines charset',
-  /** Title of a Lighthouse audit that provides detail on if the charset is set properly for a page. This title is shown when the charset meta tag is missing or defined too late in the page. */
-  failureTitle: 'Charset declaration is missing or occurs too late in the HTML',
-  /** Description of a Lighthouse audit that tells the user why the charset needs to be defined early on. */
-  description: 'A character encoding declaration is required. It can be done with a `<meta>` tag ' +
-    'in the first 1024 bytes of the HTML or in the Content-Type HTTP response header. ' +
-    '[Learn more](https://web.dev/charset/).',
-};
-
-const str_ = i18n.createMessageInstanceIdFn(__filename, UIStrings);
-
-const CONTENT_TYPE_HEADER = 'content-type';
-// /^[a-zA-Z0-9-_:.()]{2,}$/ matches all known IANA charset names (https://www.iana.org/assignments/character-sets/character-sets.xhtml)
-const IANA_REGEX = /^[a-zA-Z0-9-_:.()]{2,}$/;
-const CHARSET_HTML_REGEX = /<meta[^>]+charset[^<]+>/i;
-const CHARSET_HTTP_REGEX = /charset\s*=\s*[a-zA-Z0-9-_:.()]{2,}/i;
-
-class CharsetDefined extends Audit {
-  /**
-   * @return {LH.Audit.Meta}
-   */
-  static get meta() {
-    return {
-      id: 'charset',
-      title: str_(UIStrings.title),
-      failureTitle: str_(UIStrings.failureTitle),
-      description: str_(UIStrings.description),
-      requiredArtifacts: ['MainDocumentContent', 'URL', 'devtoolsLogs', 'MetaElements'],
-    };
-  }
-
-  /**
-   * @param {LH.Artifacts} artifacts
-   * @param {LH.Audit.Context} context
-   * @return {Promise<LH.Audit.Product>}
-   */
-  static async audit(artifacts, context) {
-    const devtoolsLog = artifacts.devtoolsLogs[Audit.DEFAULT_PASS];
-    const mainResource = await MainResource.request({devtoolsLog, URL: artifacts.URL}, context);
-    let isCharsetSet = false;
-    // Check the http header 'content-type' to see if charset is defined there
-    if (mainResource.responseHeaders) {
-      const contentTypeHeader = mainResource.responseHeaders
-        .find(header => header.name.toLowerCase() === CONTENT_TYPE_HEADER);
-
-      if (contentTypeHeader) {
-        isCharsetSet = CHARSET_HTTP_REGEX.test(contentTypeHeader.value);
-      }
-    }
-
-    // Check if there is a BOM byte marker
-    const BOM_FIRSTCHAR = 0xFEFF;
-    isCharsetSet = isCharsetSet || artifacts.MainDocumentContent.charCodeAt(0) === BOM_FIRSTCHAR;
-
-    // Check if charset-ish meta tag is defined within the first 1024 characters(~1024 bytes) of the HTML document
-    if (CHARSET_HTML_REGEX.test(artifacts.MainDocumentContent.slice(0, 1024))) {
-      // If so, double-check the DOM attributes, considering both legacy http-equiv and html5 charset styles.
-      isCharsetSet = isCharsetSet || artifacts.MetaElements.some(meta => {
-        return (meta.charset && IANA_REGEX.test(meta.charset)) ||
-          (meta.httpEquiv === 'content-type' &&
-          meta.content &&
-          CHARSET_HTTP_REGEX.test(meta.content));
-      });
-    }
-
-    return {
-      score: Number(isCharsetSet),
-    };
-  }
-}
-
-module.exports = CharsetDefined;
-module.exports.UIStrings = UIStrings;
-module.exports.CHARSET_HTML_REGEX = CHARSET_HTML_REGEX;
-module.exports.CHARSET_HTTP_REGEX = CHARSET_HTTP_REGEX;
-module.exports.IANA_REGEX = IANA_REGEX;

package/lighthouse-core/audits/dobetterweb/doctype.js

@@ -1,94 +0,0 @@
-/**
- * @license Copyright 2018 The Lighthouse Authors. All Rights Reserved.
- * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at http://www.apache.org/licenses/LICENSE-2.0
- * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License.
- */
-'use strict';
-
-const Audit = require('../audit.js');
-const i18n = require('../../lib/i18n/i18n.js');
-
-const UIStrings = {
-  /** Title of a Lighthouse audit that provides detail on the doctype of a page. This descriptive title is shown to users when the pages's doctype is set to HTML. */
-  title: 'Page has the HTML doctype',
-  /** Title of a Lighthouse audit that provides detail on the doctype of a page. This descriptive title is shown to users when the page's doctype is not set to HTML. */
-  failureTitle: 'Page lacks the HTML doctype, thus triggering quirks-mode',
-  /** Description of a Lighthouse audit that tells the user why they should define an HTML doctype. This is displayed after a user expands the section to see more. No character length limits. 'Learn More' becomes link text to additional documentation. */
-  description: 'Specifying a doctype prevents the browser ' +
-    'from switching to quirks-mode. ' +
-    '[Learn more](https://web.dev/doctype/).',
-  /** Explanatory message stating that the document has no doctype. */
-  explanationNoDoctype: 'Document must contain a doctype',
-  /** Explanatory message stating that the publicId field is not empty. */
-  explanationPublicId: 'Expected publicId to be an empty string',
-  /** Explanatory message stating that the systemId field is not empty. */
-  explanationSystemId: 'Expected systemId to be an empty string',
-  /** Explanatory message stating that the doctype is set, but is not "html" and is therefore invalid. */
-  explanationBadDoctype: 'Doctype name must be the lowercase string `html`',
-};
-
-const str_ = i18n.createMessageInstanceIdFn(__filename, UIStrings);
-
-class Doctype extends Audit {
-  /**
-   * @return {LH.Audit.Meta}
-   */
-  static get meta() {
-    return {
-      id: 'doctype',
-      title: str_(UIStrings.title),
-      failureTitle: str_(UIStrings.failureTitle),
-      description: str_(UIStrings.description),
-      requiredArtifacts: ['Doctype'],
-    };
-  }
-
-  /**
-   * @param {LH.Artifacts} artifacts
-   * @return {LH.Audit.Product}
-   */
-  static audit(artifacts) {
-    if (!artifacts.Doctype) {
-      return {
-        score: 0,
-        explanation: str_(UIStrings.explanationNoDoctype),
-      };
-    }
-
-    // only set constants once we know there is a doctype
-    const doctypeName = artifacts.Doctype.name.trim();
-    const doctypePublicId = artifacts.Doctype.publicId;
-    const doctypeSystemId = artifacts.Doctype.systemId;
-
-    if (doctypePublicId !== '') {
-      return {
-        score: 0,
-        explanation: str_(UIStrings.explanationPublicId),
-      };
-    }
-
-    if (doctypeSystemId !== '') {
-      return {
-        score: 0,
-        explanation: str_(UIStrings.explanationSystemId),
-      };
-    }
-
-    /* Note that the value for name is case sensitive,
-       and must be the string `html`. For details see:
-       https://html.spec.whatwg.org/multipage/parsing.html#the-initial-insertion-mode */
-    if (doctypeName === 'html') {
-      return {
-        score: 1,
-      };
-    } else {
-      return {
-        score: 0,
-        explanation: str_(UIStrings.explanationBadDoctype),
-      };
-    }
-  }
-}
-
-module.exports = Doctype;
-module.exports.UIStrings = UIStrings;