Skip to content

Commit c954168

Browse files
committed
Bug 1826293 - Remove pref privacy.partition.network_state.ws_connection_queue. r=hpeuckmann,necko-reviewers,valentin
Differential Revision: https://phabricator.services.mozilla.com/D174611
1 parent 28ebdc6 commit c954168

File tree

3 files changed

+3
-17
lines changed

3 files changed

+3
-17
lines changed

modules/libpref/init/StaticPrefList.yaml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13249,12 +13249,6 @@
1324913249
value: false
1325013250
mirror: always
1325113251

13252-
# Partition the websocket pending connection queue by OriginAttributes.
13253-
- name: privacy.partition.network_state.ws_connection_queue
13254-
type: RelaxedAtomicBool
13255-
value: true
13256-
mirror: always
13257-
1325813252
# Partition the service workers unconditionally when dFPI is enabled.
1325913253
- name: privacy.partition.serviceWorkers
1326013254
type: RelaxedAtomicBool

netwerk/protocol/websocket/WebSocketChannel.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -518,10 +518,8 @@ class nsWSAdmissionManager {
518518

519519
int32_t IndexOf(nsCString& aAddress, nsCString& aOriginSuffix) {
520520
for (uint32_t i = 0; i < mQueue.Length(); i++) {
521-
bool isPartitioned =
522-
(StaticPrefs::privacy_partition_network_state() ||
523-
StaticPrefs::privacy_firstparty_isolate()) &&
524-
StaticPrefs::privacy_partition_network_state_ws_connection_queue();
521+
bool isPartitioned = StaticPrefs::privacy_partition_network_state() ||
522+
StaticPrefs::privacy_firstparty_isolate();
525523
if (aAddress == (mQueue[i])->mAddress &&
526524
(!isPartitioned || aOriginSuffix == (mQueue[i])->mOriginSuffix)) {
527525
return i;

toolkit/components/antitracking/test/browser/browser_staticPartition_websocket.js

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -81,14 +81,8 @@ function connect(browsingContext, host, protocol) {
8181
// By observing the order of the handshakes we can ensure that the queue
8282
// partitioning is working correctly.
8383
async function runTest(partitioned) {
84-
let prefs = [
85-
["privacy.partition.network_state.ws_connection_queue", partitioned],
86-
];
87-
if (partitioned) {
88-
prefs.push(["privacy.partition.network_state", partitioned]);
89-
}
9084
await SpecialPowers.pushPrefEnv({
91-
set: prefs,
85+
set: [["privacy.partition.network_state", partitioned]],
9286
});
9387

9488
let tabA = BrowserTestUtils.addTab(gBrowser, FIRST_PARTY_A);

0 commit comments

Comments
 (0)