Fix bug when transitioning from an empty to a non-empty password list

Fix bug when transitioning from an empty to a non-empty password list

This change fixes a bug that occurs when a list transitions from an empty to a
non-empty state. Before this change, the entries of the list were updated before
its visibility, which causes |List.redraw()| to skip adding items to the
viewport when the list was hidden. Now the visibility will be updated first,
leading to correct behavior.

BUG=672869
R=dbeam@chromium.org
CC=kolos@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2571443002
Cr-Commit-Position: refs/heads/master@{#440048}
(cherry picked from commit 0edd353c07b8cebe483ec64f5e1abe638001d2d4)

Committed: https://chromium.googlesource.com/chromium/src/+/d446c28066e5c0e524f0420125207d75da682299

[vabr (Chromium), 2016-12-23 10:12:56]

Description was changed from

==========
Fix bug when transitioning from an empty to a non-empty password list

This change fixes a bug that occurs when a list transitions from an empty to a
non-empty state. Before this change, the entries of the list were updated before
its visibility, which causes |List.redraw()| to skip adding items to the
viewport when the list was hidden. Now the visibility will be updated first,
leading to correct behavior.

BUG=672869
R=dbeam@chromium.org
CC=kolos@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2571443002
Cr-Commit-Position: refs/heads/master@{#440048}
(cherry picked from commit 0edd353c07b8cebe483ec64f5e1abe638001d2d4)
==========

to

==========
Fix bug when transitioning from an empty to a non-empty password list

This change fixes a bug that occurs when a list transitions from an empty to a
non-empty state. Before this change, the entries of the list were updated before
its visibility, which causes |List.redraw()| to skip adding items to the
viewport when the list was hidden. Now the visibility will be updated first,
leading to correct behavior.

BUG=672869
R=dbeam@chromium.org
CC=kolos@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2571443002
Cr-Commit-Position: refs/heads/master@{#440048}
(cherry picked from commit 0edd353c07b8cebe483ec64f5e1abe638001d2d4)

Committed:
https://chromium.googlesource.com/chromium/src/+/d446c28066e5c0e524f042012520...
==========

[vabr (Chromium), 2016-12-23 10:12:58]

Committed patchset #1 (id:1) manually as
d446c28066e5c0e524f0420125207d75da682299.