From b4802ea750364891745885562bcd1412e2c519dc Mon Sep 17 00:00:00 2001 From: Mario Sanchez Prada Date: Thu, 12 Aug 2021 17:43:04 +0200 Subject: [PATCH] Adapt to change in content_settings.mojom.AllowStorageAccess signature The |site_for_cookies| parameter is now a net::SiteForCookies, instead of a GURL. Chromium change: https://source.chromium.org/chromium/chromium/src/+/b7a6e74b0f7070dbc595b84bfbbf40b71f2f1db8 commit b7a6e74b0f7070dbc595b84bfbbf40b71f2f1db8 Author: Ari Chivukula Date: Fri Aug 6 20:43:41 2021 +0000 [GURL -> SiteForCookies] components/content_settings/common/content_settings_manager.mojom This is part of a larger effort to replace GURL with SiteForCookies to simplify/consolidate comparison and increase safety. This file was found by looking for mojom files as follows: https://source.chromium.org/search?q=%22mojom.Url%20site_for_cookies%22 Bug: 577565 --- .../brave_content_settings_agent_impl_autoplay_browsertest.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/components/content_settings/renderer/brave_content_settings_agent_impl_autoplay_browsertest.cc b/components/content_settings/renderer/brave_content_settings_agent_impl_autoplay_browsertest.cc index d3532a381d93..bb21e5aebe4b 100644 --- a/components/content_settings/renderer/brave_content_settings_agent_impl_autoplay_browsertest.cc +++ b/components/content_settings/renderer/brave_content_settings_agent_impl_autoplay_browsertest.cc @@ -36,7 +36,7 @@ class MockContentSettingsManagerImpl : public mojom::ContentSettingsManager { void AllowStorageAccess(int32_t render_frame_id, StorageType storage_type, const url::Origin& origin, - const GURL& site_for_cookies, + const ::net::SiteForCookies& site_for_cookies, const url::Origin& top_frame_origin, base::OnceCallback callback) override {}