From 4da1ba558a85ec48220dd7588acba99b008fb01e Mon Sep 17 00:00:00 2001 From: lannka Date: Mon, 1 Feb 2021 00:39:25 +0000 Subject: [PATCH 1/7] (2019-10-01, 0ab715ef1) `amp-action-macro`: 1 Previous history on prod-config.json: - 0ab715ef1 - 2019-10-01T10:32:36-04:00 - Launch --- build-system/global-configs/canary-config.json | 1 - build-system/global-configs/prod-config.json | 1 - extensions/amp-action-macro/0.1/amp-action-macro.js | 4 ++-- .../amp-action-macro/0.1/test/test-amp-action-macro.js | 8 ++------ tools/experiments/experiments-config.js | 6 ------ 5 files changed, 4 insertions(+), 16 deletions(-) diff --git a/build-system/global-configs/canary-config.json b/build-system/global-configs/canary-config.json index baf51247b648..9fd8e360c479 100644 --- a/build-system/global-configs/canary-config.json +++ b/build-system/global-configs/canary-config.json @@ -5,7 +5,6 @@ "canary": 1, "a4aProfilingRate": 0.01, "adsense-ad-size-optimization": 0.1, - "amp-action-macro": 1, "amp-story-responsive-units": 1, "amp-story-v1": 1, "chunked-amp": 1, diff --git a/build-system/global-configs/prod-config.json b/build-system/global-configs/prod-config.json index 031d91172019..c3a0d48cbf30 100644 --- a/build-system/global-configs/prod-config.json +++ b/build-system/global-configs/prod-config.json @@ -5,7 +5,6 @@ "a4aProfilingRate": 0.01, "adsense-ad-size-optimization": 0.1, "amp-accordion-display-locking": 1, - "amp-action-macro": 1, "amp-story-responsive-units": 1, "amp-story-v1": 1, "chunked-amp": 1, diff --git a/extensions/amp-action-macro/0.1/amp-action-macro.js b/extensions/amp-action-macro/0.1/amp-action-macro.js index 199c9fbbdeda..f722181715f3 100644 --- a/extensions/amp-action-macro/0.1/amp-action-macro.js +++ b/extensions/amp-action-macro/0.1/amp-action-macro.js @@ -15,7 +15,6 @@ */ import {LayoutPriority} from '../../../src/layout'; import {Services} from '../../../src/services'; -import {isExperimentOn} from '../../../src/experiments'; import {userAssert} from '../../../src/log'; /** @const {string} */ @@ -39,7 +38,8 @@ export class AmpActionMacro extends AMP.BaseElement { /** @override */ buildCallback() { userAssert( - isExperimentOn(this.win, 'amp-action-macro'), + /* isExperimentOn(this.win, 'amp-action-macro') // launched: true */ + true, 'Experiment is off' ); const {element} = this; diff --git a/extensions/amp-action-macro/0.1/test/test-amp-action-macro.js b/extensions/amp-action-macro/0.1/test/test-amp-action-macro.js index 4759511a07a3..52f334bc776c 100644 --- a/extensions/amp-action-macro/0.1/test/test-amp-action-macro.js +++ b/extensions/amp-action-macro/0.1/test/test-amp-action-macro.js @@ -18,7 +18,6 @@ import {ActionInvocation} from '../../../../src/service/action-impl'; import {ActionTrust} from '../../../../src/action-constants'; import {AmpActionMacro} from '../amp-action-macro'; import {Services} from '../../../../src/services'; -import {toggleExperiment} from '../../../../src/experiments'; describes.realWin( 'amp-action-macro', @@ -35,8 +34,6 @@ describes.realWin( beforeEach(() => { win = env.win; doc = win.document; - - toggleExperiment(win, 'amp-action-macro', true); }); function newActionMacro() { @@ -60,7 +57,8 @@ describes.realWin( it('should not build if experiment is off', () => { return allowConsoleError(() => { - toggleExperiment(env.win, 'amp-action-macro', false); + /* toggleExperiment(env.win, 'amp-action-macro', false) // launched: true */ + false; return newActionMacro().catch((err) => { expect(err.message).to.include('Experiment is off'); }); @@ -75,8 +73,6 @@ describes.realWin( let unreferrableMacro; let unreferrableMacroElement; beforeEach(() => { - toggleExperiment(win, 'amp-action-macro', true); - // This macro is referrable and can be invoked by the macro element(s) // defined after it. referrableMacroElement = doc.createElement('amp-action-macro'); diff --git a/tools/experiments/experiments-config.js b/tools/experiments/experiments-config.js index 44407c742bfb..9b84debe06f2 100644 --- a/tools/experiments/experiments-config.js +++ b/tools/experiments/experiments-config.js @@ -75,12 +75,6 @@ export const EXPERIMENTS = [ name: 'Allows the new lightbox experience to be used in A4A (prototype).', spec: 'https://github.com/ampproject/amphtml/issues/7743', }, - { - id: 'amp-action-macro', - name: 'AMP extension for defining action macros', - spec: 'https://github.com/ampproject/amphtml/issues/19494', - cleanupIssue: 'https://github.com/ampproject/amphtml/pull/19495', - }, { id: 'ios-fixed-no-transfer', name: 'Remove fixed transfer from iOS 12.2 and up', From 49359e04de785e5308602629db1a0593185ce99c Mon Sep 17 00:00:00 2001 From: lannka Date: Mon, 1 Feb 2021 00:39:46 +0000 Subject: [PATCH 2/7] (2018-11-05, d26327a3c) `amp-story-responsive-units`: 1 Previous history on prod-config.json: - d26327a3c - 2018-11-05T12:02:25-05:00 - Re-launch amp-story-responsive-units experiment - d5ec88b8a - 2018-10-30T16:01:59-07:00 - Unlaunch amp-story-responsive-units for urgent release - 65379df93 - 2018-10-26T15:27:09-04:00 - Launch amp-story-responsive-units experiment --- build-system/global-configs/canary-config.json | 1 - build-system/global-configs/prod-config.json | 1 - extensions/amp-story/1.0/test/test-amp-story.js | 7 +++---- tools/experiments/experiments-config.js | 6 ------ 4 files changed, 3 insertions(+), 12 deletions(-) diff --git a/build-system/global-configs/canary-config.json b/build-system/global-configs/canary-config.json index 9fd8e360c479..1ca176f3dc53 100644 --- a/build-system/global-configs/canary-config.json +++ b/build-system/global-configs/canary-config.json @@ -5,7 +5,6 @@ "canary": 1, "a4aProfilingRate": 0.01, "adsense-ad-size-optimization": 0.1, - "amp-story-responsive-units": 1, "amp-story-v1": 1, "chunked-amp": 1, "doubleclickSraExp": 0.01, diff --git a/build-system/global-configs/prod-config.json b/build-system/global-configs/prod-config.json index c3a0d48cbf30..a12f775016f0 100644 --- a/build-system/global-configs/prod-config.json +++ b/build-system/global-configs/prod-config.json @@ -5,7 +5,6 @@ "a4aProfilingRate": 0.01, "adsense-ad-size-optimization": 0.1, "amp-accordion-display-locking": 1, - "amp-story-responsive-units": 1, "amp-story-v1": 1, "chunked-amp": 1, "doubleclickSraExp": 0.01, diff --git a/extensions/amp-story/1.0/test/test-amp-story.js b/extensions/amp-story/1.0/test/test-amp-story.js index 1ac8cb2319e8..5d8162cbf8f6 100644 --- a/extensions/amp-story/1.0/test/test-amp-story.js +++ b/extensions/amp-story/1.0/test/test-amp-story.js @@ -1656,12 +1656,11 @@ describes.realWin( }); describe('amp-story rewriteStyles', () => { - beforeEach(() => { - toggleExperiment(win, 'amp-story-responsive-units', true); - }); + beforeEach(() => {}); afterEach(() => { - toggleExperiment(win, 'amp-story-responsive-units', false); + /* toggleExperiment(win, 'amp-story-responsive-units', false) // launched: true */ + false; }); it('should rewrite vw styles', async () => { diff --git a/tools/experiments/experiments-config.js b/tools/experiments/experiments-config.js index 9b84debe06f2..8083d9623892 100644 --- a/tools/experiments/experiments-config.js +++ b/tools/experiments/experiments-config.js @@ -124,12 +124,6 @@ export const EXPERIMENTS = [ spec: 'https://github.com/ampproject/amphtml/issues/11329', cleanupIssue: 'https://github.com/ampproject/amphtml/issues/14357', }, - { - id: 'amp-story-responsive-units', - name: 'Scale pages in amp-story by rewriting responsive units', - spec: 'https://github.com/ampproject/amphtml/issues/15955', - cleanupIssue: 'https://github.com/ampproject/amphtml/issues/15960', - }, { id: 'amp-next-page', name: 'Document level next page recommendations and infinite scroll', From 861818eafeb5487991483336c2f0842bf8e4e8e5 Mon Sep 17 00:00:00 2001 From: lannka Date: Mon, 1 Feb 2021 00:39:55 +0000 Subject: [PATCH 3/7] (2018-07-11, 366b9d53e) `amp-story-v1`: 1 Previous history on prod-config.json: - 366b9d53e - 2018-07-11T12:03:09-04:00 - Revert "Remove launched experiments. (#16528)" - 3a3dbd963 - 2018-07-06T14:47:49-04:00 - Remove launched experiments. - 1194c1652 - 2018-06-12T11:38:38-04:00 - Start canonical adsense fast fetch experiments at 1% --- build-system/global-configs/canary-config.json | 1 - build-system/global-configs/prod-config.json | 1 - 2 files changed, 2 deletions(-) diff --git a/build-system/global-configs/canary-config.json b/build-system/global-configs/canary-config.json index 1ca176f3dc53..47eb479f540c 100644 --- a/build-system/global-configs/canary-config.json +++ b/build-system/global-configs/canary-config.json @@ -5,7 +5,6 @@ "canary": 1, "a4aProfilingRate": 0.01, "adsense-ad-size-optimization": 0.1, - "amp-story-v1": 1, "chunked-amp": 1, "doubleclickSraExp": 0.01, "doubleclickSraReportExcludedBlock": 0.1, diff --git a/build-system/global-configs/prod-config.json b/build-system/global-configs/prod-config.json index a12f775016f0..1a62a485b881 100644 --- a/build-system/global-configs/prod-config.json +++ b/build-system/global-configs/prod-config.json @@ -5,7 +5,6 @@ "a4aProfilingRate": 0.01, "adsense-ad-size-optimization": 0.1, "amp-accordion-display-locking": 1, - "amp-story-v1": 1, "chunked-amp": 1, "doubleclickSraExp": 0.01, "doubleclickSraReportExcludedBlock": 0.1, From 30783d1f70abeb243c6a15fb2a305f241c73295a Mon Sep 17 00:00:00 2001 From: lannka Date: Mon, 1 Feb 2021 00:40:05 +0000 Subject: [PATCH 4/7] (2018-07-11, 366b9d53e) `chunked-amp`: 1 Previous history on prod-config.json: - 366b9d53e - 2018-07-11T12:03:09-04:00 - Revert "Remove launched experiments. (#16528)" - 3a3dbd963 - 2018-07-06T14:47:49-04:00 - Remove launched experiments. - 34ffcdb16 - 2017-02-17T13:10:59-08:00 - turn on chunked-amp --- build-system/global-configs/canary-config.json | 1 - build-system/global-configs/prod-config.json | 1 - tools/experiments/experiments-config.js | 5 ----- 3 files changed, 7 deletions(-) diff --git a/build-system/global-configs/canary-config.json b/build-system/global-configs/canary-config.json index 47eb479f540c..aace97f1bf74 100644 --- a/build-system/global-configs/canary-config.json +++ b/build-system/global-configs/canary-config.json @@ -5,7 +5,6 @@ "canary": 1, "a4aProfilingRate": 0.01, "adsense-ad-size-optimization": 0.1, - "chunked-amp": 1, "doubleclickSraExp": 0.01, "doubleclickSraReportExcludedBlock": 0.1, "dfp-render-on-idle-cwv-exp": 1, diff --git a/build-system/global-configs/prod-config.json b/build-system/global-configs/prod-config.json index 1a62a485b881..fa887a8979b1 100644 --- a/build-system/global-configs/prod-config.json +++ b/build-system/global-configs/prod-config.json @@ -5,7 +5,6 @@ "a4aProfilingRate": 0.01, "adsense-ad-size-optimization": 0.1, "amp-accordion-display-locking": 1, - "chunked-amp": 1, "doubleclickSraExp": 0.01, "doubleclickSraReportExcludedBlock": 0.1, "expand-json-targeting": 1, diff --git a/tools/experiments/experiments-config.js b/tools/experiments/experiments-config.js index 8083d9623892..4924d8327b1a 100644 --- a/tools/experiments/experiments-config.js +++ b/tools/experiments/experiments-config.js @@ -81,11 +81,6 @@ export const EXPERIMENTS = [ spec: 'https://github.com/ampproject/amphtml/issues/22220', cleanupIssue: 'https://github.com/ampproject/amphtml/issues/22220', }, - { - id: 'chunked-amp', - name: "Split AMP's loading phase into chunks", - cleanupIssue: 'https://github.com/ampproject/amphtml/issues/5535', - }, { id: 'pump-early-frame', name: From d7aca0d696e3e2b0c06a87137f2e37a1e11a2e80 Mon Sep 17 00:00:00 2001 From: lannka Date: Mon, 1 Feb 2021 00:40:24 +0000 Subject: [PATCH 5/7] (2019-10-28, 4ca140956) `fix-inconsistent-responsive-height-selection`: 0 Previous history on prod-config.json: - 4ca140956 - 2019-10-28T15:54:36+00:00 - Ramp down inconsistent height selection fix experiment to 0%. This doesn't work very well on desktop / tablet so we need to fix that before proceeding. --- build-system/global-configs/canary-config.json | 1 - build-system/global-configs/prod-config.json | 1 - tools/experiments/experiments-config.js | 6 ------ 3 files changed, 8 deletions(-) diff --git a/build-system/global-configs/canary-config.json b/build-system/global-configs/canary-config.json index aace97f1bf74..3a1b4f140b2a 100644 --- a/build-system/global-configs/canary-config.json +++ b/build-system/global-configs/canary-config.json @@ -9,7 +9,6 @@ "doubleclickSraReportExcludedBlock": 0.1, "dfp-render-on-idle-cwv-exp": 1, "expand-json-targeting": 1, - "fix-inconsistent-responsive-height-selection": 0, "flexAdSlots": 0.05, "intersect-resources": 0, "ios-fixed-no-transfer": 1, diff --git a/build-system/global-configs/prod-config.json b/build-system/global-configs/prod-config.json index fa887a8979b1..e40d7a890994 100644 --- a/build-system/global-configs/prod-config.json +++ b/build-system/global-configs/prod-config.json @@ -8,7 +8,6 @@ "doubleclickSraExp": 0.01, "doubleclickSraReportExcludedBlock": 0.1, "expand-json-targeting": 1, - "fix-inconsistent-responsive-height-selection": 0, "flexAdSlots": 0.05, "intersect-resources": 0, "ios-fixed-no-transfer": 0, diff --git a/tools/experiments/experiments-config.js b/tools/experiments/experiments-config.js index 4924d8327b1a..073adf479fc6 100644 --- a/tools/experiments/experiments-config.js +++ b/tools/experiments/experiments-config.js @@ -168,12 +168,6 @@ export const EXPERIMENTS = [ spec: 'https://github.com/ampproject/amphtml/issues/23568', cleanupIssue: 'https://github.com/ampproject/amphtml/issues/24165', }, - { - id: 'fix-inconsistent-responsive-height-selection', - name: 'Fix inconsistent responsive height selection.', - spec: 'https://github.com/ampproject/amphtml/issues/24166', - cleanupIssue: 'https://github.com/ampproject/amphtml/issues/24167', - }, { id: 'intersect-resources', name: 'Use IntersectionObserver for resource scheduling.', From aab7afa41819f43f1070577876561eb929dc64af Mon Sep 17 00:00:00 2001 From: lannka Date: Mon, 1 Feb 2021 00:40:44 +0000 Subject: [PATCH 6/7] (2017-05-16, ca2005971) `pump-early-frame`: 1 Previous history on prod-config.json: - ca2005971 - 2017-05-16T09:31:30-04:00 - Start Doubleclick Fast Fetch potential SRA delay measure experiment --- build-system/global-configs/canary-config.json | 3 +-- build-system/global-configs/prod-config.json | 3 +-- src/runtime.js | 2 +- test/unit/test-runtime.js | 4 ---- tools/experiments/experiments-config.js | 7 ------- 5 files changed, 3 insertions(+), 16 deletions(-) diff --git a/build-system/global-configs/canary-config.json b/build-system/global-configs/canary-config.json index 3a1b4f140b2a..662550cb0789 100644 --- a/build-system/global-configs/canary-config.json +++ b/build-system/global-configs/canary-config.json @@ -1,6 +1,6 @@ { "allow-doc-opt-in": ["amp-next-page"], - "allow-url-opt-in": ["pump-early-frame"], + "allow-url-opt-in": [], "amp-accordion-display-locking": 1, "canary": 1, "a4aProfilingRate": 0.01, @@ -12,7 +12,6 @@ "flexAdSlots": 0.05, "intersect-resources": 0, "ios-fixed-no-transfer": 1, - "pump-early-frame": 1, "remove-task-timeout": 0, "build-in-chunks": 1, "visibility-trigger-improvements": 1, diff --git a/build-system/global-configs/prod-config.json b/build-system/global-configs/prod-config.json index e40d7a890994..51f8291abdbe 100644 --- a/build-system/global-configs/prod-config.json +++ b/build-system/global-configs/prod-config.json @@ -1,6 +1,6 @@ { "allow-doc-opt-in": ["amp-next-page"], - "allow-url-opt-in": ["pump-early-frame"], + "allow-url-opt-in": [], "canary": 0, "a4aProfilingRate": 0.01, "adsense-ad-size-optimization": 0.1, @@ -11,7 +11,6 @@ "flexAdSlots": 0.05, "intersect-resources": 0, "ios-fixed-no-transfer": 0, - "pump-early-frame": 1, "fie-resources": 1, "visibility-trigger-improvements": 1, "sticky-ad-transition": 0.1 diff --git a/src/runtime.js b/src/runtime.js index 0f881b562cb6..46c1a9f5fd1d 100644 --- a/src/runtime.js +++ b/src/runtime.js @@ -496,7 +496,7 @@ function maybeLoadCorrectVersion(win, fnOrStruct) { * pumped. */ function maybePumpEarlyFrame(win, cb) { - if (!isExperimentOn(win, 'pump-early-frame')) { + if (!(/* isExperimentOn(win, 'pump-early-frame') // launched: true */ true)) { cb(); return; } diff --git a/test/unit/test-runtime.js b/test/unit/test-runtime.js index a599a9e4f6c8..34b69645604c 100644 --- a/test/unit/test-runtime.js +++ b/test/unit/test-runtime.js @@ -33,7 +33,6 @@ import {installPlatformService} from '../../src/service/platform-impl'; import {installTimerService} from '../../src/service/timer-impl'; import {setShadowDomSupportedVersionForTesting} from '../../src/web-components'; import {toArray} from '../../src/types'; -import {toggleExperiment} from '../../src/experiments'; import {vsyncForTesting} from '../../src/service/vsync-impl'; describes.fakeWin( @@ -202,7 +201,6 @@ describes.fakeWin( ); it('should not maybePumpEarlyFrame when body not yet present', () => { - toggleExperiment(win, 'pump-early-frame', true); // Make document.body be null on first invocation to simulate // JS executing before the rest of the doc has been parsed. const {body} = win.document; @@ -223,14 +221,12 @@ describes.fakeWin( 'should not maybePumpEarlyFrame ' + 'when a renderDelayingExtension is present', () => { - toggleExperiment(win, 'pump-early-frame', true); win.document.body.appendChild(document.createElement('amp-experiment')); extensionRegistrationTest(); } ); it('should maybePumpEarlyFrame and delay extension execution', () => { - toggleExperiment(win, 'pump-early-frame', true); let progress = ''; const queueExtensions = win.AMP; const highPriority = regularExtension((amp) => { diff --git a/tools/experiments/experiments-config.js b/tools/experiments/experiments-config.js index 073adf479fc6..f9ac6331ba5e 100644 --- a/tools/experiments/experiments-config.js +++ b/tools/experiments/experiments-config.js @@ -81,13 +81,6 @@ export const EXPERIMENTS = [ spec: 'https://github.com/ampproject/amphtml/issues/22220', cleanupIssue: 'https://github.com/ampproject/amphtml/issues/22220', }, - { - id: 'pump-early-frame', - name: - 'If applicable, let the browser paint the current frame before ' + - 'executing the callback.', - cleanupIssue: 'https://github.com/ampproject/amphtml/issues/8237', - }, { id: 'web-worker', name: 'Web worker for background processing', From f0b2803c422eb23cfce3ddc649ea02bc4dc720a5 Mon Sep 17 00:00:00 2001 From: lannka Date: Mon, 1 Feb 2021 00:40:44 +0000 Subject: [PATCH 7/7] =?UTF-8?q?=F0=9F=9A=AE=20Sweep=20experiments=20older?= =?UTF-8?q?=20than=202020-02-01?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sweep experiments last flipped globally up to 2020-02-01: - (2019-10-01, 0ab715ef1) `amp-action-macro`: 1 - (2018-11-05, d26327a3c) `amp-story-responsive-units`: 1 - (2018-07-11, 366b9d53e) `amp-story-v1`: 1 - (2018-07-11, 366b9d53e) `chunked-amp`: 1 - (2019-10-28, 4ca140956) `fix-inconsistent-responsive-height-selection`: 0 - (2017-05-16, ca2005971) `pump-early-frame`: 1 --- ### ⚠️ Javascript source files require intervention The following may contain errors and/or require intervention to remove superfluous conditionals: - `extensions/amp-action-macro/0.1/amp-action-macro.js` - `extensions/amp-action-macro/0.1/test/test-amp-action-macro.js` - `extensions/amp-story/1.0/test/test-amp-story.js` - `src/runtime.js` - `test/unit/test-runtime.js` Refer to the removal guide for [suggestions on handling these modified Javascript files.](https://github.com/ampproject/amphtml/blob/master/build-system/tasks/sweep-experiments/README.md#followup) --- ### ⚠️ HTML files may still contain references The following HTML files contain references to experiment names which may be stale and should be manually removed: - `examples/amp-action-macro.html` Refer to the removal guide for [suggestions on handling these HTML files.](https://github.com/ampproject/amphtml/blob/master/build-system/tasks/sweep-experiments/README.md#followup:html)