New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AdSense/Doubleclick Fast Fetch - setup delay request by viewport experiments #16136
AdSense/Doubleclick Fast Fetch - setup delay request by viewport experiments #16136
Conversation
addExperimentIdToElement(adsenseFormatExpId, this.element); | ||
|
||
this.size_ = adsenseFormatExpId == '21062004' | ||
this.size_ = getExperimentBranch(this.win, FORMAT_EXP) == '21062004' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I realize you didn't name this attribute in this PR, but this seems like a bad name if it is a bool....
@@ -234,6 +257,27 @@ export class AmpAdNetworkAdsenseImpl extends AmpA4A { | |||
return null; | |||
} | |||
|
|||
/** @visibleForTesting */ | |||
divertExperiments() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could probably use a little more jsdoc or inline comments explaining what the experiments are
Add experiments for AdSense & Doubleclick Fast Fetch that delays sending ad request depending on slots location relative to viewport (see infrastructure added in #15989). For AdSense, experiment includes multiple viewport ranges (3, 4, 6, 12 viewports) and for Doubleclick add experiment that delays until within 12 viewports if using render on idle.