commit 9868f6c13635c8311585f6adc3e82d9bf4d0f71e parent 3d27100ba4127a113fa2ec8c1d3091dfaec60845 Author: Emilio Cobos Álvarez <emilio@crisal.io> Date: Mon, 1 Dec 2025 15:33:51 +0000 Bug 2003279 - Remove IsAboutPage check from Document::WarnOnceAbout. r=smaug,dom-core It's intended to exempt internal about: pages from telemetry, but ShouldIncludeInTelemetry() already accounts for that. Differential Revision: https://phabricator.services.mozilla.com/D274594 Diffstat:
| M | dom/base/Document.cpp | | | 10 | +--------- |
1 file changed, 1 insertion(+), 9 deletions(-)
diff --git a/dom/base/Document.cpp b/dom/base/Document.cpp @@ -14420,15 +14420,7 @@ void Document::WarnOnceAbout( return; } mDeprecationWarnedAbout[static_cast<size_t>(aOperation)] = true; - // Don't count deprecated operations for about pages since those pages - // are almost in our control, and we always need to remove uses there - // before we remove the operation itself anyway. - // FIXME: This seems wrong for about:blank. we don't report telemetry for - // internal about: pages anyways (see ShouldIncludeInTelemetry()). - if (!IsAboutPage()) { - const_cast<Document*>(this)->SetUseCounter( - OperationToUseCounter(aOperation)); - } + const_cast<Document*>(this)->SetUseCounter(OperationToUseCounter(aOperation)); uint32_t flags = asError ? nsIScriptError::errorFlag : nsIScriptError::warningFlag; nsContentUtils::ReportToConsole(