Commit db0eaaec authored by Sergey Ulanov's avatar Sergey Ulanov Committed by Commit Bot

[Fuchsia] Remove default app permissions

Previously CastComponent was granting PROTECTED_MEDIA_IDENTIFIER
permissions when permissions field is missing from the config. This is
no longer necessary because ApplicationConfigManager now sets
permissions for all apps.

Bug: 922833
Change-Id: I29205db3ca6f3a46daa42924f71e4fff11d60abc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2093395
Auto-Submit: Sergey Ulanov <sergeyu@chromium.org>
Reviewed-by: default avatarWez <wez@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748189}
parent 3ab0c760
...@@ -97,15 +97,6 @@ void CastComponent::StartComponent() { ...@@ -97,15 +97,6 @@ void CastComponent::StartComponent() {
frame()->SetPermissionState(std::move(permission_clone), origin, frame()->SetPermissionState(std::move(permission_clone), origin,
fuchsia::web::PermissionState::GRANTED); fuchsia::web::PermissionState::GRANTED);
} }
} else {
// Grant PROTECTED_MEDIA_IDENTIFIER permission if permissions are not
// specified in the config. This is necessary for compatibility with older
// ApplicationConfigManager implementations that do not set permissions.
fuchsia::web::PermissionDescriptor eme_id_permission;
eme_id_permission.set_type(
fuchsia::web::PermissionType::PROTECTED_MEDIA_IDENTIFIER);
frame()->SetPermissionState(std::move(eme_id_permission), origin,
fuchsia::web::PermissionState::GRANTED);
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment