Skip to content

Commit

Permalink
Drop use of federated_learning::kFlocIdSortingLshBasedComputation
Browse files Browse the repository at this point in the history
This definition has been dropped upstream as it's enabled by default
now, but even in that case that doesn't make a difference for Brave
as the FlocIdProviderImpl object is never created as per [1], so we
can simply drop these 2 lines.

[1] #8468

Chromium change:

https://chromium.googlesource.com/chromium/src.git/+/77462226d2aa2980c80e009ab7de098407af653d

commit 77462226d2aa2980c80e009ab7de098407af653d
Author: Yao Xiao <yaoxia@chromium.org>
Date:   Wed Mar 31 23:11:46 2021 +0000

    [floc] Use sorting-lsh by default. Remove base::Feature and clean up tests.

    Bug: 1190992
  • Loading branch information
mariospr committed Apr 9, 2021
1 parent a328717 commit 765039c
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion app/brave_main_delegate.cc
Expand Up @@ -222,7 +222,6 @@ bool BraveMainDelegate::BasicStartupComplete(int* exit_code) {
features::kWebOTP.name,
federated_learning::kFederatedLearningOfCohorts.name,
federated_learning::kFlocIdComputedEventLogging.name,
federated_learning::kFlocIdSortingLshBasedComputation.name,
kSharingQRCodeGenerator.name,
net::features::kFirstPartySets.name,
network_time::kNetworkTimeServiceQuerying.name,
Expand Down
1 change: 0 additions & 1 deletion app/brave_main_delegate_browsertest.cc
Expand Up @@ -80,7 +80,6 @@ IN_PROC_BROWSER_TEST_F(BraveMainDelegateBrowserTest, DisabledFeatures) {
&features::kWebOTP,
&federated_learning::kFederatedLearningOfCohorts,
&federated_learning::kFlocIdComputedEventLogging,
&federated_learning::kFlocIdSortingLshBasedComputation,
&kSharingQRCodeGenerator,
&net::features::kFirstPartySets,
&network_time::kNetworkTimeServiceQuerying,
Expand Down

0 comments on commit 765039c

Please sign in to comment.