• Kush Sinha's avatar
    Store account emails in Chrome OS Account Manager · 722e5748
    Kush Sinha authored
    Chrome OS Account Manager's clients relied on |AccountTrackerService| to
    map |AccountManager::AccountKey| to other representations (namely raw
    email and canonical email). Due to the relation between one such client,
    |ChromeOSOAuth2TokenServiceDelegate|, and |AccountTrackerService|, it
    was required that |AccountTrackerService| should be able to store
    account mappings for which a refresh token was not yet known. This
    usage model is no longer supported. See the attached bug for context.
    
    Fix this by adding a |raw_email| field in Chrome OS Account Manager's
    Account proto definition. Future patches will change clients to read
    this value.
    
    Bug: 925827
    Test: chromeos_unittests --gtest_filter="*AccountManager*Test*"
    Test: unit_tests --gtest_filter="*CrOSOAuthDelegateTest*"
    Test: browser_tests --gtest_filter="*Arc*AuthService*Test*"
    Change-Id: I76ef741ec0a9ea38ae8757ff234c8be6dd7a3155
    Reviewed-on: https://chromium-review.googlesource.com/c/1452003Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
    Reviewed-by: default avatarColin Blundell <blundell@chromium.org>
    Reviewed-by: default avatarLutz Justen <ljusten@chromium.org>
    Commit-Queue: Kush Sinha <sinhak@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#635493}
    722e5748
account_manager.h 14.2 KB