brave-core

Files

T

History

Claudio DeSouza 502133225b [cr147] DecodeUrlEscapeSequences simplified

There is now a variant of this function that return an utf-8 string,
simplifying how this mehtod is used.

Chromium changes:
https://chromium.googlesource.com/chromium/src/+/7020f45881b59d21fdfa505fae7896d6e7ff8f5c
https://chromium.googlesource.com/chromium/src/+/f3aa498c5cf8995d340c96861c7e3ecd67a08aa4
https://chromium.googlesource.com/chromium/src/+/cc9e49b56e502928873a876eca6538231c23f49c

commit 7020f45881b59d21fdfa505fae7896d6e7ff8f5c
Author: Kent Tamura <tkent@chromium.org>
Date:   Mon Feb 23 20:18:38 2026 -0800

    url: Rename DecodeURLMode and DecodeURLEscapeSequences

    This CL renames DecodeURLMode to DecodeUrlMode, and
    DecodeURLEscapeSequences() to DecodeUrlEscapeSequences() to follow the
    Google C++ style guide.

    The style guide recommends treating acronyms as words in names (e.g.,
    `StartRpc` instead of `StartRPC`). This change applies this convention
    to the "URL" acronym in these names for consistency across the codebase.

    Change-Id: I29ac4456e9745f66a2cd9e1d733cacecfe683a88
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7593238
    Reviewed-by: Hayato Ito <hayato@chromium.org>
    Auto-Submit: Kent Tamura <tkent@chromium.org>
    Commit-Queue: Hayato Ito <hayato@chromium.org>
    Owners-Override: Hayato Ito <hayato@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#1589145}

commit f3aa498c5cf8995d340c96861c7e3ecd67a08aa4
Author: Kent Tamura <tkent@chromium.org>
Date:   Tue Feb 24 17:20:59 2026 -0800

    url: Add an overload of DecodeUrlEscapeSequences()

    This CL introduces an overload of DecodeUrlEscapeSequences(), which
    returns a decoded UTF-8 string.

    Change-Id: I3349cf8376801831675391228c9c4bd3e5d1a5ef
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7603356
    Commit-Queue: Hayato Ito <hayato@chromium.org>
    Reviewed-by: Hayato Ito <hayato@chromium.org>
    Auto-Submit: Kent Tamura <tkent@chromium.org>
    Owners-Override: Hayato Ito <hayato@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#1589812}

commit cc9e49b56e502928873a876eca6538231c23f49c
Author: Kent Tamura <tkent@chromium.org>
Date:   Wed Feb 25 22:50:18 2026 -0800

    url: Introduce UrlEscapeDecoder to simplify URL decoding

    This change introduces url::UrlEscapeDecoder, a helper class that
    simplifies the process of decoding URL escape sequences.

    Previously, decoding required manually setting up a RawCanonOutputW
    buffer and then calling DecodeUrlEscapeSequences(). This pattern was
    verbose.

    The new UrlEscapeDecoder encapsulates this logic. It performs the
    decoding in its constructor. This makes the calling code more concise
    and readable:

      url::UrlEscapeDecoder decoder(input, mode);
      ... use decoder.view() ...

    This CL refactors several existing call sites to use the new, simpler
    pattern.

    Change-Id: I7b3c1538136b537ac33d6f101ca0e011308cc974
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7610180
    Commit-Queue: Kent Tamura <tkent@chromium.org>
    Owners-Override: Hayato Ito <hayato@chromium.org>
    Auto-Submit: Kent Tamura <tkent@chromium.org>
    Reviewed-by: Hayato Ito <hayato@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#1590661}