Commit f3d1cd0e authored by Karan Bhatia's avatar Karan Bhatia Committed by Commit Bot

Extensions: Remove redundant code in Dispatcher::OnUpdatePermissions.

Code to set the policy host restrictions is duplicated in
Dispatcher::OnUpdatePermissions. Fix it.

BUG=None

Change-Id: I3cfc5cbe6b7565c2093204690bd88333a23f683a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1695758
Auto-Submit: Karan Bhatia <karandeepb@chromium.org>
Reviewed-by: default avatarDevlin <rdevlin.cronin@chromium.org>
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#676614}
parent ee4d2bd5
...@@ -1167,13 +1167,6 @@ void Dispatcher::OnUpdatePermissions( ...@@ -1167,13 +1167,6 @@ void Dispatcher::OnUpdatePermissions(
std::move(withheld)); std::move(withheld));
UpdateOriginPermissions(*extension); UpdateOriginPermissions(*extension);
if (params.uses_default_policy_host_restrictions) {
extension->permissions_data()->SetUsesDefaultHostRestrictions();
} else {
extension->permissions_data()->SetPolicyHostRestrictions(
params.policy_blocked_hosts, params.policy_allowed_hosts);
}
bindings_system_->OnExtensionPermissionsUpdated(params.extension_id); bindings_system_->OnExtensionPermissionsUpdated(params.extension_id);
UpdateBindings(extension->id()); UpdateBindings(extension->id());
} }
......
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