mozilla
/
gecko-dev
зеркало из https://github.com/mozilla/gecko-dev.git
1
0
Форкнуть 0
Код Задачи Пакеты Проекты Релизы Вики Активность

Backed out changeset 9e7dbd18dcbb (bug 1640734) for newtab failures. CLOSED TREE

This commit is contained in:
Narcis Beleuzu 2020-06-03 04:59:57 +03:00
Родитель d99c47cbe4
Коммит 2bd2f6f375
8 изменённых файлов: 37 добавлений и 334 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

2
browser/app/profile/firefox.js
Просмотреть файл

@ -1293,7 +1293,7 @@ pref("browser.newtabpage.activity-stream.improvesearch.topSiteSearchShortcuts",
// ASRouter provider configuration // ASRouter provider configuration
pref("browser.newtabpage.activity-stream.asrouter.providers.cfr", "{\"id\":\"cfr\",\"enabled\":true,\"type\":\"remote-settings\",\"bucket\":\"cfr\",\"frequency\":{\"custom\":[{\"period\":\"daily\",\"cap\":1}]},\"categories\":[\"cfrAddons\",\"cfrFeatures\"],\"updateCycleInMs\":3600000}"); pref("browser.newtabpage.activity-stream.asrouter.providers.cfr", "{\"id\":\"cfr\",\"enabled\":true,\"type\":\"remote-settings\",\"bucket\":\"cfr\",\"frequency\":{\"custom\":[{\"period\":\"daily\",\"cap\":1}]},\"categories\":[\"cfrAddons\",\"cfrFeatures\"],\"updateCycleInMs\":3600000}");
pref("browser.newtabpage.activity-stream.asrouter.providers.whats-new-panel", "{\"id\":\"whats-new-panel\",\"enabled\":true,\"type\":\"remote-settings\",\"bucket\":\"whats-new-panel\",\"updateCycleInMs\":3600000}"); pref("browser.newtabpage.activity-stream.asrouter.providers.whats-new-panel", "{\"id\":\"whats-new-panel\",\"enabled\":true,\"type\":\"remote-settings\",\"bucket\":\"whats-new-panel\",\"updateCycleInMs\":3600000}");
pref("browser.newtabpage.activity-stream.asrouter.providers.message-groups", "{\"id\":\"message-groups\",\"enabled\":true,\"type\":\"remote-settings\",\"bucket\":\"message-groups\",\"updateCycleInMs\":3600000}"); pref("browser.newtabpage.activity-stream.asrouter.providers.message-groups", "{\"id\":\"message-groups\",\"enabled\":false,\"type\":\"remote-settings\",\"bucket\":\"message-groups\",\"updateCycleInMs\":3600000}");
// This url, if changed, MUST continue to point to an https url. Pulling arbitrary content to inject into // This url, if changed, MUST continue to point to an https url. Pulling arbitrary content to inject into
// this page over http opens us up to a man-in-the-middle attack that we'd rather not face. If you are a downstream // this page over http opens us up to a man-in-the-middle attack that we'd rather not face. If you are a downstream
// repackager of this code using an alternate snippet url, please keep your users safe // repackager of this code using an alternate snippet url, please keep your users safe

30
browser/components/newtab/content-src/components/ASRouterAdmin/ASRouterAdmin.jsx
Просмотреть файл

@ -1620,26 +1620,22 @@ export class ASRouterAdminInner extends React.PureComponent {
<td>Enabled</td> <td>Enabled</td>
<td>Impressions count</td> <td>Impressions count</td>
<td>Custom frequency</td> <td>Custom frequency</td>
<td>User preferences</td>
</tr> </tr>
</thead> </thead>
{this.state.groups && {this.state.groups &&
this.state.groups.map( this.state.groups.map(({ id, enabled, frequency }, index) => (
({ id, enabled, frequency, userPreferences = [] }, index) => ( <Row key={id}>
<Row key={id}> <td>
<td> <TogglePrefCheckbox
<TogglePrefCheckbox checked={enabled}
checked={enabled} pref={id}
pref={id} onChange={this.toggleGroups}
onChange={this.toggleGroups} />
/> </td>
</td> <td>{this._getGroupImpressionsCount(id, frequency)}</td>
<td>{this._getGroupImpressionsCount(id, frequency)}</td> <td>{JSON.stringify(frequency, null, 2)}</td>
<td>{JSON.stringify(frequency, null, 2)}</td> </Row>
<td>{userPreferences.join(", ")}</td> ))}
</Row>
)
)}
</table> </table>
</React.Fragment> </React.Fragment>
); );

7
browser/components/newtab/data/content/activity-stream.bundle.js
Просмотреть файл

@ -1987,18 +1987,17 @@ class ASRouterAdminInner extends react__WEBPACK_IMPORTED_MODULE_4___default.a.Pu
case "groups": case "groups":
return react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement(react__WEBPACK_IMPORTED_MODULE_4___default.a.Fragment, null, react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("h2", null, "Message Groups"), react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("table", null, react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("thead", null, react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("tr", { return react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement(react__WEBPACK_IMPORTED_MODULE_4___default.a.Fragment, null, react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("h2", null, "Message Groups"), react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("table", null, react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("thead", null, react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("tr", {
className: "message-item" className: "message-item"
}, react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("td", null, "Enabled"), react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("td", null, "Impressions count"), react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("td", null, "Custom frequency"), react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("td", null, "User preferences"))), this.state.groups && this.state.groups.map(({ }, react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("td", null, "Enabled"), react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("td", null, "Impressions count"), react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("td", null, "Custom frequency"))), this.state.groups && this.state.groups.map(({
id, id,
enabled, enabled,
frequency, frequency
userPreferences = []
}, index) => react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement(Row, { }, index) => react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement(Row, {
key: id key: id
}, react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("td", null, react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement(TogglePrefCheckbox, { }, react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("td", null, react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement(TogglePrefCheckbox, {
checked: enabled, checked: enabled,
pref: id, pref: id,
onChange: this.toggleGroups onChange: this.toggleGroups
})), react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("td", null, this._getGroupImpressionsCount(id, frequency)), react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("td", null, JSON.stringify(frequency, null, 2)), react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("td", null, userPreferences.join(", ")))))); })), react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("td", null, this._getGroupImpressionsCount(id, frequency)), react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("td", null, JSON.stringify(frequency, null, 2))))));
case "ds": case "ds":
return react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement(react__WEBPACK_IMPORTED_MODULE_4___default.a.Fragment, null, react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("h2", null, "Discovery Stream"), react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement(DiscoveryStreamAdmin, { return react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement(react__WEBPACK_IMPORTED_MODULE_4___default.a.Fragment, null, react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement("h2", null, "Discovery Stream"), react__WEBPACK_IMPORTED_MODULE_4___default.a.createElement(DiscoveryStreamAdmin, {

21
browser/components/newtab/lib/ASRouter.jsm
Просмотреть файл

@ -972,6 +972,17 @@ class _ASRouter {
(await this._storage.get("groupBlockList")) || [] (await this._storage.get("groupBlockList")) || []
).concat(providerBlockList); ).concat(providerBlockList);
// Merge any existing provider impressions into the corresponding group
// Don't keep providerImpressions in state anymore
const providerImpressions =
(await this._storage.get("providerImpressions")) || {};
for (const provider of Object.keys(providerImpressions)) {
groupImpressions[provider] = [
...(groupImpressions[provider] || []),
...providerImpressions[provider],
];
}
const previousSessionEnd = const previousSessionEnd =
(await this._storage.get("previousSessionEnd")) || 0; (await this._storage.get("previousSessionEnd")) || 0;
await this.setState({ await this.setState({
@ -1518,6 +1529,11 @@ class _ASRouter {
state.groups, state.groups,
"groupImpressions" "groupImpressions"
); );
this._cleanupImpressionsForItems(
state,
state.providers,
"providerImpressions"
);
return { messageImpressions, groupImpressions }; return { messageImpressions, groupImpressions };
}); });
} }
@ -1710,8 +1726,11 @@ class _ASRouter {
await this.setState(state => { await this.setState(state => {
const providerBlockList = [...state.providerBlockList, ...idsToBlock]; const providerBlockList = [...state.providerBlockList, ...idsToBlock];
// When a provider is blocked, its impressions should be cleared as well
const providerImpressions = { ...state.providerImpressions };
idsToBlock.forEach(id => delete providerImpressions[id]);
this._storage.set("providerBlockList", providerBlockList); this._storage.set("providerBlockList", providerBlockList);
return { providerBlockList }; return { providerBlockList, providerImpressions };
}); });
} }

2
browser/components/newtab/test/browser/browser.ini
Просмотреть файл

@ -43,6 +43,4 @@ tags = remote-settings
[browser_asrouter_momentspagehub.js] [browser_asrouter_momentspagehub.js]
tags = remote-settings tags = remote-settings
[browser_asrouter_experimentsAPILoader.js] [browser_asrouter_experimentsAPILoader.js]
[browser_asrouter_group_frequency.js]
[browser_asrouter_group_userprefs.js]
[browser_asrouter_trigger_docs.js] [browser_asrouter_trigger_docs.js]

163
browser/components/newtab/test/browser/browser_asrouter_group_frequency.js
Просмотреть файл

@ -1,163 +0,0 @@
const { ASRouter } = ChromeUtils.import(
"resource://activity-stream/lib/ASRouter.jsm"
);
const { RemoteSettings } = ChromeUtils.import(
"resource://services-settings/remote-settings.js"
);
const { CFRMessageProvider } = ChromeUtils.import(
"resource://activity-stream/lib/CFRMessageProvider.jsm"
);
/**
* Load and modify a message for the test.
*/
add_task(async function setup() {
// Force the WNPanel provider cache to 0 by modifying updateCycleInMs
await SpecialPowers.pushPrefEnv({
set: [
[
"browser.newtabpage.activity-stream.asrouter.providers.cfr",
`{"id":"cfr","enabled":true,"type":"remote-settings","bucket":"cfr","categories":["cfrAddons","cfrFeatures"],"updateCycleInMs":0}`,
],
],
});
const initialMsgCount = ASRouter.state.messages.length;
const heartbeatMsg = CFRMessageProvider.getMessages().find(
m => m.id === "HEARTBEAT_TACTIC_2"
);
const testMessage = {
...heartbeatMsg,
groups: ["messaging-experiments"],
targeting: "true",
// Ensure no overlap due to frequency capping with other tests
id: `HEARTBEAT_MESSAGE_${Date.now()}`,
};
const client = RemoteSettings("cfr");
await client.db.clear();
await client.db.create(testMessage);
await client.db.saveLastModified(42); // Prevent from loading JSON dump.
// Reload the providers
await BrowserTestUtils.waitForCondition(async () => {
await ASRouter._updateMessageProviders();
await ASRouter.loadMessagesFromAllProviders();
return ASRouter.state.messages.length > initialMsgCount;
}, "Should load the extra heartbeat message");
BrowserTestUtils.waitForCondition(
() => ASRouter.state.messages.find(m => m.id === testMessage.id),
"Wait to load the message"
);
const msg = ASRouter.state.messages.find(m => m.id === testMessage.id);
Assert.equal(msg.targeting, "true");
Assert.equal(msg.groups[0], "messaging-experiments");
registerCleanupFunction(async () => {
await client.db.clear();
// Reload the providers
await BrowserTestUtils.waitForCondition(async () => {
await ASRouter._updateMessageProviders();
await ASRouter.loadMessagesFromAllProviders();
return ASRouter.state.messages.length === initialMsgCount;
}, "Should reset messages");
await SpecialPowers.popPrefEnv();
});
});
/**
* Test group frequency capping.
* Message has a lifetime frequency of 3 but it's group has a lifetime frequency
* of 2. It should only show up twice.
* We update the provider to remove any daily limitations so it should show up
* on every new tab load.
*/
add_task(async function test_heartbeat_tactic_2() {
const TEST_URL = "http://example.com";
// Force the WNPanel provider cache to 0 by modifying updateCycleInMs
await SpecialPowers.pushPrefEnv({
set: [
[
"browser.newtabpage.activity-stream.asrouter.providers.message-groups",
`{"id":"message-groups","enabled":true,"type":"remote-settings","bucket":"message-groups","updateCycleInMs":0}`,
],
],
});
const groupConfiguration = {
id: "messaging-experiments",
enabled: true,
userPreferences: [],
frequency: { lifetime: 2 },
};
const client = RemoteSettings("message-groups");
await client.db.clear();
await client.db.create(groupConfiguration);
await client.db.saveLastModified(42); // Prevent from loading JSON dump.
// Reload the providers
await ASRouter._updateMessageProviders();
await ASRouter.loadMessagesFromAllProviders();
await BrowserTestUtils.waitForCondition(
() => ASRouter.state.groups.find(g => g.id === groupConfiguration.id),
"Wait for group config to load"
);
let groupState = ASRouter.state.groups.find(
g => g.id === groupConfiguration.id
);
Assert.ok(groupState, "Group config found");
Assert.ok(groupState.enabled, "Group is enabled");
let tab1 = await BrowserTestUtils.openNewForegroundTab(gBrowser, TEST_URL);
await BrowserTestUtils.loadURI(tab1.linkedBrowser, TEST_URL);
let chiclet = document.getElementById("contextual-feature-recommendation");
Assert.ok(chiclet, "CFR chiclet element found (tab1)");
await BrowserTestUtils.waitForCondition(
() => !chiclet.hidden,
"Chiclet should be visible (tab1)"
);
let tab2 = await BrowserTestUtils.openNewForegroundTab(gBrowser, TEST_URL);
await BrowserTestUtils.loadURI(tab2.linkedBrowser, TEST_URL);
Assert.ok(chiclet, "CFR chiclet element found (tab2)");
await BrowserTestUtils.waitForCondition(
() => !chiclet.hidden,
"Chiclet should be visible (tab2)"
);
// Wait for the message to become blocked (frequency limit reached)
const msg = ASRouter.state.messages.find(m =>
m.groups.includes("messaging-experiments")
);
Assert.ok(msg, "Message found");
await BrowserTestUtils.waitForCondition(
() => !ASRouter.isBelowFrequencyCaps(msg),
"Message frequency limit should be reached"
);
let tab3 = await BrowserTestUtils.openNewForegroundTab(gBrowser, TEST_URL);
await BrowserTestUtils.loadURI(tab3.linkedBrowser, TEST_URL);
await BrowserTestUtils.waitForCondition(
() => chiclet.hidden,
"Heartbeat button should be hidden"
);
info("Cleanup");
BrowserTestUtils.removeTab(tab1);
BrowserTestUtils.removeTab(tab2);
BrowserTestUtils.removeTab(tab3);
await client.db.clear();
// Reset group impressions
await ASRouter.setGroupState({ id: "messaging-experiments", value: true });
await ASRouter.setGroupState({ id: "cfr", value: true });
// Reload the providers
await ASRouter._updateMessageProviders();
await ASRouter.loadMessagesFromAllProviders();
await SpecialPowers.popPrefEnv();
});

143
browser/components/newtab/test/browser/browser_asrouter_group_userprefs.js
Просмотреть файл

@ -1,143 +0,0 @@
const { ASRouter } = ChromeUtils.import(
"resource://activity-stream/lib/ASRouter.jsm"
);
const { RemoteSettings } = ChromeUtils.import(
"resource://services-settings/remote-settings.js"
);
const { CFRMessageProvider } = ChromeUtils.import(
"resource://activity-stream/lib/CFRMessageProvider.jsm"
);
/**
* Load and modify a message for the test.
*/
add_task(async function setup() {
// Force the WNPanel provider cache to 0 by modifying updateCycleInMs
await SpecialPowers.pushPrefEnv({
set: [
[
"browser.newtabpage.activity-stream.asrouter.providers.cfr",
`{"id":"cfr","enabled":true,"type":"remote-settings","bucket":"cfr","updateCycleInMs":0}`,
],
],
});
const initialMsgCount = ASRouter.state.messages.length;
const heartbeatMsg = CFRMessageProvider.getMessages().find(
m => m.id === "HEARTBEAT_TACTIC_2"
);
const testMessage = {
...heartbeatMsg,
groups: ["messaging-experiments"],
targeting: "true",
// Ensure no overlap due to frequency capping with other tests
id: `HEARTBEAT_MESSAGE_${Date.now()}`,
};
const client = RemoteSettings("cfr");
await client.db.clear();
await client.db.create(testMessage);
await client.db.saveLastModified(42); // Prevent from loading JSON dump.
// Reload the providers
await BrowserTestUtils.waitForCondition(async () => {
await ASRouter._updateMessageProviders();
await ASRouter.loadMessagesFromAllProviders();
return ASRouter.state.messages.length > initialMsgCount;
}, "Should load the extra heartbeat message");
BrowserTestUtils.waitForCondition(
() => ASRouter.state.messages.find(m => m.id === testMessage.id),
"Wait to load the message"
);
const msg = ASRouter.state.messages.find(m => m.id === testMessage.id);
Assert.equal(msg.targeting, "true");
Assert.equal(msg.groups[0], "messaging-experiments");
Assert.ok(ASRouter.isUnblockedMessage(msg), "Message is unblocked");
registerCleanupFunction(async () => {
await client.db.clear();
// Reload the providers
await BrowserTestUtils.waitForCondition(async () => {
await ASRouter._updateMessageProviders();
await ASRouter.loadMessagesFromAllProviders();
return ASRouter.state.messages.length === initialMsgCount;
}, "Should reset messages");
await SpecialPowers.popPrefEnv();
});
});
/**
* Test group user preferences.
* Group is enabled if both user preferences are enabled.
*/
add_task(async function test_heartbeat_tactic_2() {
const TEST_URL = "http://example.com";
await SpecialPowers.pushPrefEnv({
set: [
[
"browser.newtabpage.activity-stream.asrouter.providers.message-groups",
`{"id":"message-groups","enabled":true,"type":"remote-settings","bucket":"message-groups","updateCycleInMs":0}`,
],
],
});
const groupConfiguration = {
id: "messaging-experiments",
enabled: true,
userPreferences: ["browser.userPreference.messaging-experiments"],
};
const client = RemoteSettings("message-groups");
await client.db.clear();
await client.db.create(groupConfiguration);
await client.db.saveLastModified(42); // Prevent from loading JSON dump.
// Reload the providers
await ASRouter._updateMessageProviders();
await ASRouter.loadMessagesFromAllProviders();
await BrowserTestUtils.waitForCondition(
() => ASRouter.state.groups.find(g => g.id === groupConfiguration.id),
"Wait for group config to load"
);
let groupState = ASRouter.state.groups.find(
g => g.id === groupConfiguration.id
);
Assert.ok(groupState, "Group config found");
Assert.ok(groupState.enabled, "Group is enabled");
let tab1 = await BrowserTestUtils.openNewForegroundTab(gBrowser, TEST_URL);
await BrowserTestUtils.loadURI(tab1.linkedBrowser, TEST_URL);
let chiclet = document.getElementById("contextual-feature-recommendation");
Assert.ok(chiclet, "CFR chiclet element found");
await BrowserTestUtils.waitForCondition(
() => !chiclet.hidden,
"Chiclet should be visible (userprefs enabled)"
);
await SpecialPowers.pushPrefEnv({
set: [["browser.userPreference.messaging-experiments", false]],
});
let tab2 = await BrowserTestUtils.openNewForegroundTab(gBrowser, TEST_URL);
await BrowserTestUtils.loadURI(tab2.linkedBrowser, TEST_URL);
await BrowserTestUtils.waitForCondition(
() => chiclet.hidden,
"Heartbeat button should not be visible (userprefs disabled)"
);
info("Cleanup");
BrowserTestUtils.removeTab(tab1);
BrowserTestUtils.removeTab(tab2);
await client.db.clear();
// Reset group impressions
await ASRouter.setGroupState({ id: "messaging-experiments", value: true });
await ASRouter.setGroupState({ id: "cfr", value: true });
// Reload the providers
await ASRouter._updateMessageProviders();
await ASRouter.loadMessagesFromAllProviders();
await SpecialPowers.popPrefEnv();
});

3
testing/profiles/common/user.js
Просмотреть файл

@ -9,9 +9,6 @@ user_pref("devtools.console.stdout.chrome", true);
user_pref("browser.newtabpage.activity-stream.asrouter.providers.cfr", "[]"); user_pref("browser.newtabpage.activity-stream.asrouter.providers.cfr", "[]");
user_pref("browser.newtabpage.activity-stream.asrouter.providers.cfr-fxa", "[]"); user_pref("browser.newtabpage.activity-stream.asrouter.providers.cfr-fxa", "[]");
user_pref("browser.newtabpage.activity-stream.asrouter.providers.snippets", "[]"); user_pref("browser.newtabpage.activity-stream.asrouter.providers.snippets", "[]");
user_pref("browser.newtabpage.activity-stream.asrouter.providers.message-groups", "[]");
user_pref("browser.newtabpage.activity-stream.asrouter.providers.whats-new-panel", "[]");
user_pref("browser.newtabpage.activity-stream.asrouter.providers.messaging-experiments", "[]");
user_pref("browser.newtabpage.activity-stream.feeds.section.topstories", false); user_pref("browser.newtabpage.activity-stream.feeds.section.topstories", false);
user_pref("browser.newtabpage.activity-stream.feeds.snippets", false); user_pref("browser.newtabpage.activity-stream.feeds.snippets", false);
user_pref("browser.newtabpage.activity-stream.tippyTop.service.endpoint", ""); user_pref("browser.newtabpage.activity-stream.tippyTop.service.endpoint", "");