Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lastpass extension crash when clicking on [...] icon in username field #1096

Closed
mbacchi opened this issue Sep 11, 2018 · 3 comments
Closed

Comments

@mbacchi
Copy link
Contributor

mbacchi commented Sep 11, 2018

Description

Lastpass crashes in Brave Core when you click on the Lastpass icon in the username field. Down in the lower right corner a popup appears, prompting you to reload the extension. This popup looks like:

image

You can also reload the extension from the extension page, which looks like this:

image

I am able to click on the Lastpass icon in some username/password fields, for example on the site mail.yahoo.com, but one that seems to be consistenly crashing the extension is https://invest.ameritrade.com/grid/p/login.

Steps to Reproduce

  1. Browse to https://invest.ameritrade.com/grid/p/login
  2. Click on the Lastpass icon (three dots) in the User ID field.
  3. The extension crashes and you get a popup in the lower right corner that asks to reload the extension.

Actual result:

Extension crashes and requests reloading.

Expected result:

Prompts you to select your userid from the lastpass menu.

Reproduces how often:

The first time you attempt to use the Lastpass extension on page load. After reloading the extension it doesn't necessarily work but it doesn't crash either. You can reproduce again by reloading the page(CTRL-R) and then clicking on the Lastpass icon again.

Brave version (about:brave info)

Version 0.55.2 Chromium: 70.0.3528.4 (Official Build) dev (64-bit)

Reproducible on current release:

No

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields? Yes
  • Is the issue reproducible on the latest version of Chrome? No

Additional Information

@srirambv
Copy link
Contributor

Possibly related to #236

@mbacchi
Copy link
Contributor Author

mbacchi commented Sep 12, 2018

This was fixed with this PR: brave/brave-core#421

Closing.

@mbacchi mbacchi closed this as completed Sep 12, 2018
@bbondy bbondy added the QA/Yes label Sep 27, 2018
@LaurenWags
Copy link
Member

LaurenWags commented Oct 3, 2018

Verified passed with

Brave 0.55.10 Chromium: 70.0.3538.22 (Official Build) beta(64-bit)
Revision ac9418ba9c3bd7f6baaffa0b055dfe147e0f8364-refs/branch-heads/3538@{#468}
OS Mac OS X

Verified passed with

Brave 0.55.10 Chromium: 70.0.3538.22 (Official Build) beta(64-bit)
Revision ac9418ba9c3bd7f6baaffa0b055dfe147e0f8364-refs/branch-heads/3538@{#468}
OS Linux

Verification passed on

Brave 0.55.11 Chromium: 70.0.3538.35 (Official Build) beta (64-bit)
Revision 28dcb499844fa40c28d5f62e337876cb936f79f5-refs/branch-heads/3538@{#678}
OS Windows 7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants