Commit acc5fa20 authored by Antonio Sartori's avatar Antonio Sartori Committed by Chromium LUCI CQ

Pass document policies into Policy Container by const reference

Initially, this parameter was just an enum, so we were passing it by
value. Now it became a struct, so we should pass it by const
reference.

Bug: 1130587
Change-Id: I6c02e4453259914c01a7d6c218be66cc5af30ad3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2616560
Auto-Submit: Antonio Sartori <antoniosartori@chromium.org>
Reviewed-by: default avatarArthur Sonzogni <arthursonzogni@chromium.org>
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#841487}
parent 88201ec9
......@@ -8,7 +8,7 @@ namespace content {
PolicyContainerHost::PolicyContainerHost() = default;
PolicyContainerHost::PolicyContainerHost(
PolicyContainerHost::DocumentPolicies document_policies)
const PolicyContainerHost::DocumentPolicies& document_policies)
: document_policies_(document_policies) {}
PolicyContainerHost::~PolicyContainerHost() = default;
......
......@@ -41,7 +41,7 @@ class CONTENT_EXPORT PolicyContainerHost
};
PolicyContainerHost();
explicit PolicyContainerHost(DocumentPolicies document_policies);
explicit PolicyContainerHost(const DocumentPolicies& document_policies);
PolicyContainerHost(const PolicyContainerHost&) = delete;
PolicyContainerHost& operator=(const PolicyContainerHost&) = delete;
~PolicyContainerHost() override;
......
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