• esprehn's avatar
    Don't use consume() in CSSTokenizer when ignoring the return value. · c0d8da05
    esprehn authored
    consume() (the no argument version) does a bunch of work handling out of bounds
    access, NUL replacement characters, and returning the char. Many of the callers
    just want to increment the offset. We could do that by replacing the call sites
    with consume(1) which is the "fast" version that doesn't have a return value,
    but it'd make the code more clear to just inline advance() in the right spots.
    
    BUG=605792
    
    Review-Url: https://codereview.chromium.org/2099413002
    Cr-Commit-Position: refs/heads/master@{#402328}
    c0d8da05
CSSTokenizer.h 3.5 KB