Merge pull request #14816 from brave/maxk-fix-shields-favicon

[cr105 follow up] Fixes favicon in Shields Panel.
brave · Aug 25, 2022 · 9afd6a5 · 9afd6a5
2 parents 1016114 + 15ec8a1
commit 9afd6a5
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/browser/ui/brave_shields_data_controller.cc b/browser/ui/brave_shields_data_controller.cc
@@ -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) {