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 remove delay request experiments #16623
AdSense/Doubleclick remove delay request experiments #16623
Conversation
* @visibleForTesting | ||
*/ | ||
export const DELAY_REQUEST_EXP_BRANCHES = { | ||
'21062224': true, // control, delay by renderOutsideViewport |
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.
Please remove all server side references to these experiments.
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.
will do
@@ -301,12 +288,6 @@ export class AmpAdNetworkDoubleclickImpl extends AmpA4A { | |||
return this.isAmpAdElement(); | |||
} | |||
|
|||
/** @override */ | |||
delayAdRequestEnabled() { | |||
return getExperimentBranch(this.win, RENDER_IDLE_DELAY_REQUEST_EXP) == |
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.
Just want to doublecheck that the intent here is to fully delete, as opposed to in the adsense impl where it just returns true
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.
confirmed
Codecov Report
@@ Coverage Diff @@
## master #16623 +/- ##
==========================================
+ Coverage 77.11% 78.1% +0.98%
==========================================
Files 550 551 +1
Lines 40274 40362 +88
==========================================
+ Hits 31059 31523 +464
+ Misses 9215 8839 -376
Continue to review full report at Codecov.
|
Added in #16136 and being removed as unlaunchable