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

Yandex favicon is 404 on about:preferences#search #6082

Closed
luixxiul opened this issue Dec 8, 2016 · 1 comment
Closed

Yandex favicon is 404 on about:preferences#search #6082

luixxiul opened this issue Dec 8, 2016 · 1 comment
Assignees
@darkdh
Labels
misc/favicon QA/checked-Win32 QA/checked-Win64 QA/test-plan-specified release-notes/include settings
Milestone
0.13.0

Comments

@luixxiul
Copy link
Contributor

luixxiul commented Dec 8, 2016

Describe the issue you encountered:
screenshot 2016-12-08 13 57 47

Expected behavior:

The favicon should be fetched successfully

  • Platform (Win7, 8, 10? macOS? Linux distro?): macOS

  • Brave Version: master branch

  • Any related issues:
@darkdh darkdh self-assigned this Dec 8, 2016
@darkdh darkdh closed this as completed in a6c6384 Dec 8, 2016
bbondy pushed a commit that referenced this issue Dec 13, 2016
@darkdh @bbondy
Fix Yandex favicon link
9ece875 
fix #6082

Auditors: @bbondy

Test Plan: Covered by automated test
@luixxiul luixxiul added this to the 0.13.0 milestone Dec 15, 2016
This was referenced Dec 20, 2016
Closed
Closed
Closed
Closed
@alexwykoff alexwykoff mentioned this issue Dec 20, 2016
Closed
@luixxiul
Copy link
Contributor Author

Test plan:

1 Open about:preferences#search
2 Make sure that the favicon of Yandex is loaded

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@darkdh darkdh

Labels
misc/favicon QA/checked-Win32 QA/checked-Win64 QA/test-plan-specified release-notes/include settings
Projects
None yet
Milestone
0.13.0
Development

No branches or pull requests
3 participants
@luixxiul @darkdh @srirambv