Bug 1823697 - Process adServerAttributes before check is called - r=Standard8

Differential Revision: https://phabricator.services.mozilla.com/D175783
This commit is contained in:
James Teow 2023-04-19 14:17:47 +00:00
Родитель b413f97c05
Коммит f4eecb4413
1 изменённых файлов: 2 добавлений и 2 удалений

Просмотреть файл

@ -72,6 +72,7 @@ class SearchProviders {
return component; return component;
}); });
} }
p.adServerAttributes = p.adServerAttributes ?? [];
return { return {
...p, ...p,
searchPageRegexp: new RegExp(p.searchPageRegexp), searchPageRegexp: new RegExp(p.searchPageRegexp),
@ -341,10 +342,9 @@ class SearchAdImpression {
if (!anchor.href) { if (!anchor.href) {
return false; return false;
} }
let adServerAttributes = this.#providerInfo.adServerAttributes ?? [];
let regexps = this.#providerInfo.extraAdServersRegexps; let regexps = this.#providerInfo.extraAdServersRegexps;
// Anchors can contain ad links in a data-attribute. // Anchors can contain ad links in a data-attribute.
for (let name of adServerAttributes) { for (let name of this.#providerInfo.adServerAttributes) {
if ( if (
anchor.dataset[name] && anchor.dataset[name] &&
regexps.some(regexp => regexp.test(anchor.dataset[name])) regexps.some(regexp => regexp.test(anchor.dataset[name]))