Allow reporting errors on Html::load()

The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Re-roll patch for Drupal version 10.1.x

Please include interdiffs with your patches.

Resolved Coding standard and added interdiff file.

The last submitted patch, 33: 2950595-33.patch, failed testing. View results →

#33 missed tests so adding tests from #26

#37 added some out of scope changes.

Next reroll should start at #26.

Address #39feedbacks. Re-roll against #26Please review.

Interdiff attached

Thank you @Rassoni!

+++ b/core/lib/Drupal/Component/Utility/Html.php
@@ -291,7 +293,7 @@ public static function load($html) {
+    $display_error ? $dom->loadHTML($document) : @$dom->loadHTML($document, LIBXML_NOBLANKS);

Why use different options depending on error report state?