зеркало из https://github.com/mozilla/gecko-dev.git
Bug 1811450 - Do not cap Firefox version on profiler metadata r=florian
Since Bug 1805967 Firefox version is capped to 109 due to some web compatibility issues. This resulted us to add the wrong Firefox version in the profiler metadata. We don't have the same webcompat related concerns and it's important to see the real version so data is more accurate when performance engineers look at the profiles of others. This http misc field was populated here: https://searchfox.org/mozilla-central/rev/cf3af6bb6657278880f8baf38435eeb8f2d5d86c/netwerk/protocol/http/nsHttpHandler.cpp#407-415 and UA version comes from this function which uses `MOZILLA_UAVERSION` directly: https://searchfox.org/mozilla-central/rev/cf3af6bb6657278880f8baf38435eeb8f2d5d86c/netwerk/protocol/http/nsHttpHandler.cpp#303-320 Differential Revision: https://phabricator.services.mozilla.com/D167657
This commit is contained in:
Родитель
3fe758a7d8
Коммит
51d41599d4
|
@ -2772,7 +2772,10 @@ static PreRecordedMetaInformation PreRecordMetaInformation() {
|
|||
Unused << http->GetOscpu(info.mHttpOscpu);
|
||||
}
|
||||
|
||||
Unused << http->GetMisc(info.mHttpMisc);
|
||||
// Firefox version is capped to 109.0 in the http "misc" field due to some
|
||||
// webcompat issues (Bug 1805967). We need to put the real version instead.
|
||||
info.mHttpMisc.AssignLiteral("rv:");
|
||||
info.mHttpMisc.AppendLiteral(MOZILLA_UAVERSION);
|
||||
}
|
||||
|
||||
if (nsCOMPtr<nsIXULRuntime> runtime =
|
||||
|
|
Загрузка…
Ссылка в новой задаче