• Vasilii Sukhanov's avatar
    Create a separate structure that tracks the current generation status for a · 774332ae
    Vasilii Sukhanov authored
    field in PasswordGenerationAgent.
    
    Currently |generation_element_| has a double meaning. It's an element that
    should trigger the automatic generation. However, if a manual generation is
    triggered then it's overwritten by a password field. That's confusing.
    The CL decouples the field that should trigger the automatic generation from
    whatever is happening right now on a random password field.
    In the future that will allow us to have multiple generation elements
    simultaneously. For now it fixes 870220 where the manual generation on the
    ambiguous field was suppressing the automatic generation on it.
    
    Bug: 870220,852309
    Change-Id: I6a3edd351e615289f6271fda5a45f6ff47e653af
    Reviewed-on: https://chromium-review.googlesource.com/c/1310713Reviewed-by: default avatarVadym Doroshenko <dvadym@chromium.org>
    Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#605668}
    774332ae
password_generation_agent.cc 32.9 KB