Discharge7443/brave-core
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Adapt to change in content_settings.mojom.AllowStorageAccess signature

Browse Source 
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 <arichiv@chromium.org>
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
This commit is contained in:
Mario Sanchez Prada
2021-09-13 14:53:30 -04:00
committed by mkarolin
parent 08d9a5f051
commit b4802ea750
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
components/content_settings/renderer/brave_content_settings_agent_impl_autoplay_browsertest.cc
+1 -1
View File
@@ -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<void(bool)> callback) override {}