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

Test for password manager #9346

Closed
darkdh opened this issue Jun 8, 2017 · 1 comment · Fixed by #9375
Closed

Test for password manager #9346

darkdh opened this issue Jun 8, 2017 · 1 comment · Fixed by #9375
Assignees
@darkdh
Labels
automated-tests feature/password-manager QA/checked release-notes/exclude
Milestone
0.19.x (Beta Chan...

Comments

@darkdh
Copy link
Member

darkdh commented Jun 8, 2017 •
edited by luixxiul
Loading

Test plan

See #9375 (comment)

Leverage https of jsfiddle to write test page on it
@darkdh darkdh self-assigned this Jun 8, 2017
@darkdh darkdh mentioned this issue Jun 8, 2017
Merged
7 tasks
@bsclifton
Copy link
Member

Per our convo in the CI meeting, maybe we can try GitHub pages if the jsfiddle output being in an iframe is a blocker

darkdh added a commit that referenced this issue Jun 9, 2017
@darkdh
Re-enable https test
e1c8c37 
test page: https://github.com/brave/brave-tests

fix #9346

Auditors: @bridiver, @bbondy
@darkdh darkdh mentioned this issue Jun 9, 2017
Merged
8 tasks
@luixxiul luixxiul added this to the 0.19.x (Nightly Channel) milestone Jun 23, 2017
This was referenced Sep 20, 2017
Closed
Closed
Closed
Closed
This was referenced Sep 20, 2017
Closed
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@darkdh darkdh

Labels
automated-tests feature/password-manager QA/checked release-notes/exclude
Projects
None yet
Milestone
0.19.x (Beta Channel)
Development

Successfully merging a pull request may close this issue.

Re-enable https test brave/browser-laptop
3 participants
@luixxiul @bsclifton @darkdh