• Parastoo Geranmayeh's avatar
    [AF] Multiple Nameless/Unowned forms · c5d65c1e
    Parastoo Geranmayeh authored
    with repetitive names. The correct form should get filled.
    
    Problem:
    1. When an element would go invalid, we would use the section of
    elements as one of the hints to find the new version of the old
    element in the refill, but only the section of the autofilled fields
    were passed. This would break the refills when the visibility of a
    field changes. => Set and pass section of all fields.
    2. In finding the new version of the old element, the case with more
    than two elements with the matching names were not considered.
    3. There was a bug in the logic for the nameless forms.
    
    Fixes: https://jsfiddle.net/parastoog/tnh8j9ws/, ~/ds5h7kg4/,
    ~/w81yc6La/
    
    Bug: 900185
    Change-Id: I26fda7a50b3beb7f7c484a697633ab3b7b5a76d8
    Reviewed-on: https://chromium-review.googlesource.com/c/1336492
    Commit-Queue: Parastoo Geranmayeh <parastoog@google.com>
    Reviewed-by: default avatarSebastien Seguin-Gagnon <sebsg@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#609921}
    c5d65c1e
autofill_interactive_uitest.cc 129 KB