• Mikel Astiz's avatar
    Mark synced notifications type as deprecated · 4026ebc5
    Mikel Astiz authored
    These ancient sync types haven't been used for years and deprecated in
    2014 with https://codereview.chromium.org/780363003.
    
    We now rename the enum to prevent accidental references, and remove
    some code leftovers that were effectively dead (e.g. type expansion
    in SyncPrefs).
    
    While doing this, we unify some strings in model_type.cc to avoid
    prefix "Deprecated" in type names, which some types did, and instead
    the presubmit checks are updated to whitelist deprecated datatypes and
    allow their strings to be inconsistent.
    
    Bug: 923826
    Change-Id: Ib1d7b90042ef4598cc80d036cc50c3494eb170a6
    Reviewed-on: https://chromium-review.googlesource.com/c/1454520
    Commit-Queue: Mikel Astiz <mastiz@chromium.org>
    Reviewed-by: default avatarMarc Treib <treib@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#629158}
    4026ebc5
nigori_util.cc 13.8 KB