Skip to content
This repository has been archived by the owner. It is now read-only.

/ browser-laptop Archived

Regression: isPotentialPhishingUrl is not defined #7903

Closed
luixxiul opened this issue Mar 26, 2017 · 2 comments
Closed

Regression: isPotentialPhishingUrl is not defined #7903

luixxiul opened this issue Mar 26, 2017 · 2 comments
Assignees
@luixxiul
Labels
QA/test-plan-specified bug release-notes/exclude

Comments

@luixxiul
Copy link
Contributor

@luixxiul luixxiul commented Mar 26, 2017 •
edited

Test plan

#7904 (comment)

Describe the issue you encountered:
Regression: isPotentialPhishingUrl is not defined.

This is the cause: 17b7cbe#diff-a680fa16146451835432a63cc9d196b8L9
@luixxiul luixxiul self-assigned this Mar 26, 2017
@NejcZdovc
Copy link
Member

@NejcZdovc NejcZdovc commented Mar 26, 2017

@luixxiul This is a rebase issue, will you fix it?
@luixxiul luixxiul mentioned this issue Mar 26, 2017
Merged
3 of 4 tasks complete
@luixxiul
Copy link
Contributor Author

@luixxiul luixxiul commented Mar 26, 2017

@NejcZdovc done, will you review the PR?
bsclifton added a commit that referenced this issue Mar 26, 2017
Suguru Hirahara @bsclifton
Fix regression: isPotentialPhishingUrl is not defined
Unverified
This user has not uploaded their public key yet.
GPG key ID: 37FF5F19022427F2 Learn about signing commits
Loading status checks…
62032d4 
Closes #7903

Auditors:

Test Plan:
1. Open a page in a new tab
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@luixxiul luixxiul

Labels
QA/test-plan-specified bug release-notes/exclude
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@luixxiul @NejcZdovc
You can’t perform that action at this time.