• Mikel Astiz's avatar
    Avoid crashes if all favicons already processed · fe5ab212
    Mikel Astiz authored
    We have evidence of DownloadCurrentCandidateOrAskFaviconService()
    crashing due to current_candidate() being null, at least until recent
    fixes.
    
    Temporarily and as a precaution, we return early in case the problematic
    scenario is reproduced. We suspect this condition is not reached in
    trunk, but it's a simple workaround to cherry-pick.
    
    Bug: 828196
    Change-Id: I41e2236f341a9ed9e18a950540f096e9de42df0b
    Reviewed-on: https://chromium-review.googlesource.com/1002093Reviewed-by: default avatarPeter Kotwicz <pkotwicz@chromium.org>
    Commit-Queue: Mikel Astiz <mastiz@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#549264}
    fe5ab212
favicon_handler.cc 27.3 KB