Files
brave-core/components/sync
Claudio DeSouza 8ef8c74a80 [CodeHealth] Use base::Base64Encode by return
Upstream is set to remove the variant for `base::Base64Encode` that
returns its result by argument. This is being done to remove dead code
around assuming that enconding could fail.

This change corrects our use of `base::Base64Encode`, and also avoids
extra copies in certain cases by doing away with the temporary storage
used in certain cases as a local variable.

Chromium change:
https://chromium.googlesource.com/chromium/src/+/2c860fd4bbb72d3b4574f723f847c1c1d6c3b9ec

commit 2c860fd4bbb72d3b4574f723f847c1c1d6c3b9ec
Author: Tom Sepez <tsepez@chromium.org>
Date:   Thu Feb 1 21:31:26 2024 +0000

    Remove dead code for base::Base64Encode().

    Previous Cls have removed most of usages of this one form, fix
    a few missed by them and then remove the form itself.

    Fixed: 1486214
2024-03-08 09:17:48 -03:00
..