diff options
| author | Stephen Blott | 2014-11-12 09:49:18 +0000 |
|---|---|---|
| committer | Stephen Blott | 2014-11-12 09:49:24 +0000 |
| commit | 95cb81eb90b4c78b253f30a61d819f069b199572 (patch) | |
| tree | 13d40a405a3859386aa14611b44261ad7e818858 /background_scripts | |
| parent | 2687fbe835e447beb875f399c4c150dfe919535e (diff) | |
| download | vimium-95cb81eb90b4c78b253f30a61d819f069b199572.tar.bz2 | |
Favicons in the vomnibar; remove support.
There are security issues with favicons as implemented previously. This
commit removes favicon support entirely.
Diffstat (limited to 'background_scripts')
| -rw-r--r-- | background_scripts/completion.coffee | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/background_scripts/completion.coffee b/background_scripts/completion.coffee index b6a52a15..b411bcba 100644 --- a/background_scripts/completion.coffee +++ b/background_scripts/completion.coffee @@ -26,7 +26,6 @@ class Suggestion generateHtml: -> return @html if @html - favIconUrl = @tabFavIconUrl or "#{@getUrlRoot(@url)}/favicon.ico" relevancyHtml = if @showRelevancy then "<span class='relevancy'>#{@computeRelevancy()}</span>" else "" # NOTE(philc): We're using these vimium-specific class names so we don't collide with the page's CSS. @html = @@ -35,8 +34,7 @@ class Suggestion <span class="vimiumReset vomnibarSource">#{@type}</span> <span class="vimiumReset vomnibarTitle">#{@highlightTerms(Utils.escapeHtml(@title))}</span> </div> - <div class="vimiumReset vomnibarBottomHalf vomnibarIcon" - style="background-image: url(#{favIconUrl});"> + <div class="vimiumReset vomnibarBottomHalf"> <span class="vimiumReset vomnibarUrl">#{@shortenUrl(@highlightTerms(Utils.escapeHtml(@url)))}</span> #{relevancyHtml} </div> @@ -294,7 +292,6 @@ class TabCompleter suggestions = results.map (tab) => suggestion = new Suggestion(queryTerms, "tab", tab.url, tab.title, @computeRelevancy) suggestion.tabId = tab.id - suggestion.tabFavIconUrl = tab.favIconUrl suggestion onComplete(suggestions) |