fbe55ecb90b7ab3ec5e6fdede20a7db362d34ee6
SetSyncFeatureRequested deleted
This function was already gutted functionally, and now this functionality is exclusive to ChromeOS. Chromium changes: https://chromium.googlesource.com/chromium/src/+/4c52eb14ee5ad97ee65048fc7649ded426f7fa63 commit 4c52eb14ee5ad97ee65048fc7649ded426f7fa63 Author: Mikel Astiz <mastiz@chromium.org> Date: Wed May 14 02:50:23 2025 -0700 [sync] Remove SetSyncFeatureRequested API Gradually and definitely starting with the recent https://crrev.com/c/6533710, SyncService::SetSyncFeatureRequested() did nothing outside ChromeOS. On ChromeOS, starting with https://crrev.com/c/6540467, there is a better API to use in SyncUserSettings. In this patch, ChromeOS callers are migrated to the better API and everything else is cleaned up as they are no-op calls. As a result, the SetSyncFeatureRequested API itself, as well as the Java counterpart, can be safely deleted. Bug: 40772592,40911173 Change-Id: I716ac1e59cc99afccf313697c0121858d19898c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6540075 Commit-Queue: Mikel Astiz <mastiz@chromium.org> Reviewed-by: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/main@{#1459963}
…
…
…
…
…
…
…
…
…
Brave Core
Brave Core is a set of changes, APIs, and scripts used for customizing Chromium to make the Brave browser. Please also check https://github.com/brave/brave-browser
Follow @brave on Twitter for important announcements.
Resources
Community
You can ask questions and interact with the community in the following locations:
- Brave Community
communitychannel on Brave Software's Slack
Languages
C++
36.9%
HTML
31.6%
Swift
9.4%
TypeScript
9%
Java
4.8%
Other
8.1%