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

Fix missing add observer for password store observer #209

Merged
merged 1 commit into from
Jun 8, 2017

Conversation

darkdh
Copy link
Member

@darkdh darkdh commented Jun 8, 2017

@darkdh darkdh self-assigned this Jun 8, 2017
@darkdh darkdh requested review from bridiver and bbondy June 8, 2017 00:30
darkdh added a commit to brave/browser-laptop that referenced this pull request Jun 8, 2017
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
@bridiver bridiver merged commit e571682 into master Jun 8, 2017
@bridiver bridiver deleted the password_store_observer branch June 8, 2017 00:39
darkdh added a commit to brave/browser-laptop that referenced this pull request Jun 8, 2017
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 to brave/browser-laptop that referenced this pull request Jun 10, 2017
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 to brave/browser-laptop that referenced this pull request Jun 10, 2017
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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants