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

Update getActivePasswordManager call to call getSettings #3536

Merged
merged 1 commit into from
Aug 30, 2016
Merged

Update getActivePasswordManager call to call getSettings #3536

merged 1 commit into from
Aug 30, 2016

Conversation

bsclifton
Copy link
Member

@bsclifton bsclifton commented Aug 29, 2016

I couldn't find an existing issue for this

This change only affects the context menu after upgrading from 0.11.4 or earlier to 0.11.5 or newer. Currently, the context menu is the only caller for getActivePasswordManager

Fixes #3549

Auditors: @bbondy @alexwykoff

@bsclifton
Copy link
Member Author

cc: @bbondy

@bbondy
Copy link
Member

bbondy commented Aug 30, 2016

I couldn't find an existing issue for this

Please post one and update PR. Thanks.

@bsclifton
Copy link
Member Author

bsclifton commented Aug 30, 2016

Issue created which has comprehensive STR 😄 (tagged as qa-steps-specified for verification). I have manually verified the fix on Mac. Ready for review!

@bsclifton
Copy link
Member Author

bsclifton commented Aug 30, 2016

To verify the fix

  • Ensure all Brave instances are closed
  • Download session-store-1_bsc0.11.1.zip
  • Unzip / move this session file to the appropriate location (Mac is ~/Libraries/Application\ Support/brave/), making sure to rename it as session-store-1
  • Install a version of Brave which has the fix
  • Launch Brave and navigate to https://trac.torproject.org/projects/tor/login
  • Right click inside the Username field
  • Notice that 1Password shows up, as expected

@bbondy
Copy link
Member

bbondy commented Aug 30, 2016

++

@bbondy bbondy merged commit 93a7ee9 into brave:master Aug 30, 2016
@bsclifton bsclifton deleted the fix-pw-default branch August 30, 2016 19:52
@luixxiul luixxiul added this to the 0.11.6dev milestone Aug 31, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants