Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Deleting saved password still shown until page refresh #8906

Closed
srirambv opened this issue May 17, 2017 · 1 comment · Fixed by #9085
Closed

Deleting saved password still shown until page refresh #8906

srirambv opened this issue May 17, 2017 · 1 comment · Fixed by #9085
Assignees
@darkdh
Labels
bug duplicate Issue has already been reported feature/about-pages feature/password-manager QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified regression
Milestone
0.17.13 (Release ...

Comments

@srirambv
Copy link
Collaborator

srirambv commented May 17, 2017 •
edited by luixxiul
Loading

Test plan

#9085 (comment)

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    Deleting saved password still shown until page refresh

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Windows 10 x64

  • Brave Version (revision SHA):
    Brave 0.15.304
    rev b872523
    Muon 3.0.103

  • Steps to reproduce:

    1. Keep Brave builtin password manager
    2. Visit any site and save the login details
    3. Open about:passwords, click on the delete button next to the saved entry, doesn't clear, reload the page and the entry is gone

  • Actual result:
    Deleting saved password still shown until page refresh

  • Expected result:
    Should clear the entry from table after clicking the delete button

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes

  • Is this an issue in the currently released version?
    No

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:
    passwords

  • Any related issues:
@srirambv srirambv added this to the 0.15.300 milestone May 17, 2017
@diracdeltas
Copy link
Member

dupe of #8874 item 1

@diracdeltas diracdeltas added the duplicate Issue has already been reported label May 17, 2017
@luixxiul luixxiul removed this from the 0.15.300 milestone May 18, 2017
darkdh added a commit that referenced this issue May 26, 2017
@darkdh
Update passwords and blacklist when deleting
3932e5d 
fix #8906
resolve item 1 of #8874

Auditors: @diracdeltas, @bridiver

Test Plan:
1. Makue sure you have passwords and never saved sites in
about:passwords
2. delete any of them or clear all passwords should relfect changes
instantly
darkdh added a commit that referenced this issue May 26, 2017
@darkdh
Update passwords and blacklist when deleting
a13292c 
fix #8906
resolve item 1 of #8874

Auditors: @diracdeltas, @bridiver

Test Plan:
1. Makue sure you have passwords and never saved sites in
about:passwords
2. delete any of them or clear all passwords should relfect changes
instantly
@darkdh darkdh mentioned this issue May 26, 2017
Merged
7 tasks
@darkdh darkdh added this to the 0.16.100 (Frozen, only critical adds from here) milestone May 26, 2017
@darkdh darkdh mentioned this issue Jun 1, 2017
Closed
darkdh added a commit that referenced this issue Jun 7, 2017
@darkdh
Update passwords and blacklist when deleting
cd84fbb 
fix #8906
resolve item 1 of #8874

Auditors: @diracdeltas, @bridiver

Test Plan:
1. Makue sure you have passwords and never saved sites in
about:passwords
2. delete any of them or clear all passwords should relfect changes
instantly
darkdh added a commit that referenced this issue Jun 8, 2017
@darkdh
Update passwords and blacklist when deleting
047a3e5 
fix #8906
resolve item 1 of #8874

requires brave/muon#209

Auditors: @diracdeltas, @bridiver

Test Plan:
1. Makue sure you have passwords and never saved sites in
about:passwords
2. delete any of them or clear all passwords should relfect changes
instantly
darkdh added a commit that referenced this issue Jun 8, 2017
@darkdh
Update passwords and blacklist when deleting
3fece85 
fix #8906
resolve item 1 of #8874

requires brave/muon#209

Auditors: @diracdeltas, @bridiver

Test Plan:
1. Makue sure you have passwords and never saved sites in
about:passwords
2. delete any of them or clear all passwords should relfect changes
instantly
darkdh added a commit that referenced this issue Jun 10, 2017
@darkdh
Update passwords and blacklist when deleting
52a5dbb 
fix #8906
resolve item 1 of #8874

requires brave/muon#209

Auditors: @diracdeltas, @bridiver

Test Plan:
1. Makue sure you have passwords and never saved sites in
about:passwords
2. delete any of them or clear all passwords should relfect changes
instantly
darkdh added a commit that referenced this issue Jun 10, 2017
@darkdh
Update passwords and blacklist when deleting
756d066 
fix #8906
resolve item 1 of #8874

requires brave/muon#209

Auditors: @diracdeltas, @bridiver

Test Plan:
1. Makue sure you have passwords and never saved sites in
about:passwords
2. delete any of them or clear all passwords should relfect changes
instantly
This was referenced Jun 19, 2017
Closed
Closed
Closed
Closed
@Jacalz Jacalz mentioned this issue Jun 21, 2017
Closed
94 tasks
This was referenced Jun 22, 2017
Closed
Closed
@LaurenWags LaurenWags mentioned this issue Jun 23, 2017
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@darkdh darkdh

Labels
bug duplicate Issue has already been reported feature/about-pages feature/password-manager QA/checked-Linux QA/checked-macOS QA/checked-Win64 QA/test-plan-specified regression
Projects
None yet
Milestone
0.17.13 (Release Channel)
Development

Successfully merging a pull request may close this issue.

Update passwords and blacklist when deleting brave/browser-laptop
5 participants
@diracdeltas @luixxiul @darkdh @srirambv @LaurenWags