diff --git a/addon/Feeds/MetadataFeed.js b/addon/Feeds/MetadataFeed.js index 9bf1f58175..6505aa5115 100644 --- a/addon/Feeds/MetadataFeed.js +++ b/addon/Feeds/MetadataFeed.js @@ -32,7 +32,7 @@ module.exports = class MetadataFeed extends Feed { this.linksToFetch.clear(); // if we are in the experiment, make a network request through PageScraper - if (simplePrefs.prefs["experiments.locallyFetchMetadata"]) { + if (simplePrefs.prefs["experiments.locallyFetchMetadata20"]) { return this.options.fetchNewMetadataLocally(links, "METADATA_FEED_REQUEST").then(() => (am.actions.Response("METADATA_FEED_UPDATED"))); } return this.options.fetchNewMetadata(links, "METADATA_FEED_REQUEST").then(() => (am.actions.Response("METADATA_FEED_UPDATED"))); diff --git a/content-test/addon/Feeds/MetadataFeed.test.js b/content-test/addon/Feeds/MetadataFeed.test.js index 092df4a7f1..b694aedecc 100644 --- a/content-test/addon/Feeds/MetadataFeed.test.js +++ b/content-test/addon/Feeds/MetadataFeed.test.js @@ -33,11 +33,11 @@ describe("MetadataFeed", () => { assert.calledOnce(instance.options.fetchNewMetadata))) ); it("should run sites through fetchNewMetadataLocally if experiment pref is on", () => { - simplePrefs.prefs["experiments.locallyFetchMetadata"] = true; + simplePrefs.prefs["experiments.locallyFetchMetadata20"] = true; return instance.getData().then(() => { assert.notCalled(instance.options.fetchNewMetadata); assert.calledOnce(instance.options.fetchNewMetadataLocally); - simplePrefs.prefs["experiments.locallyFetchMetadata"] = false; + simplePrefs.prefs["experiments.locallyFetchMetadata20"] = false; }); }); it("should resolve with an action, but no data", () => ( diff --git a/experiments.json b/experiments.json index 5c6fcb0fe2..4b5d156e82 100644 --- a/experiments.json +++ b/experiments.json @@ -91,7 +91,7 @@ }, "locallyFetchMetadata": { "name": "Fetch Page Content Locally", - "active": true, + "active": false, "description": "Make a network request for content of a URL", "control": { "value": false, @@ -133,5 +133,20 @@ "threshold": 0.2, "description": "Show screenshots" } + }, + "locallyFetchMetadata20": { + "name": "Fetch Page Content Locally", + "active": true, + "description": "Make a network request for content of a URL", + "control": { + "value": false, + "description": "Use remote service for metadata" + }, + "variant": { + "id": "exp-010-locally-fetch-metadata", + "value": true, + "threshold": 0.2, + "description": "Fetch page content locally" + } } }