Skip to content

Commit

Permalink
Adapt to change in content_settings.mojom.AllowStorageAccess signature
Browse files Browse the repository at this point in the history
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
  • Loading branch information
mariospr committed Sep 2, 2021
1 parent de32d6e commit a3448f6
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -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 {}

Expand Down

0 comments on commit a3448f6

Please sign in to comment.