Skip to content

Commit

Permalink
Merge pull request #14827 from brave/pr14816_maxk-fix-shields-favicon…
Browse files Browse the repository at this point in the history
…_1.44.x

[cr105 follow up] Fixes favicon in Shields Panel. (uplift to 1.44.x)
  • Loading branch information
kjozwiak authored Aug 26, 2022
2 parents 68b6fe0 + 191186e commit d5fdaa0
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions browser/ui/brave_shields_data_controller.cc
Original file line number Diff line number Diff line change
Expand Up @@ -163,9 +163,9 @@ GURL BraveShieldsDataController::GetCurrentSiteURL() {
GURL BraveShieldsDataController::GetFaviconURL(bool refresh) {
auto url = GURL("chrome://favicon2/");
url = AppendQueryParameter(url, "size", "16");
url = AppendQueryParameter(url, "scale_factor", "2x");
url = AppendQueryParameter(url, "show_fallback_monogram", "");
url = AppendQueryParameter(url, "page_url",
url = AppendQueryParameter(url, "scaleFactor", "2x");
url = AppendQueryParameter(url, "showFallbackMonogram", "");
url = AppendQueryParameter(url, "pageUrl",
GetCurrentSiteURL().GetWithoutFilename().spec());

if (refresh) {
Expand Down

0 comments on commit d5fdaa0

Please sign in to comment.