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

Import favicon fix: #4930

Merged
merged 1 commit into from
Oct 19, 2016
Merged

Import favicon fix: #4930

merged 1 commit into from
Oct 19, 2016

Commits on Oct 19, 2016

  1. Import favicon fix:

    1. Iterate all urls using the favicon
    2. Adjust made-up-favicon condition and override old made-up-favicon url
    
    fix brave#4882
    
    Auditors: @bridiver, @bbondy
    
    Test Plan:
    1. Import bookmarks from firefox on brave 0.12.3
    2. Import bookmarks from firefox on brave contains this commit
    3. The bookmarks under "Mozilla Firefox" should have favicons
    
    1. Import bookmarks from any browsers
    2. The imported bookmarks should have favicons
    
    1. Import bookmarks from html bookmarks file
    2. The imported bookmarks should have favicons
    darkdh committed Oct 19, 2016
    Configuration menu
    Copy the full SHA
    d804e39 View commit details
    Browse the repository at this point in the history