Make BootstrapSandboxPolicy a struct, containing the existing rule map and a new default rule.

Make BootstrapSandboxPolicy a struct, containing the existing rule map and a new default rule.

The default rule will be used to specify the action to take when the requested
name does not have an entry in the rule map.

BUG=367863
R=mark@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=274676

[Mark Mentovai, 2014-06-03 16:57:26]

LGTM.

https://codereview.chromium.org/310833003/diff/1/sandbox/mac/policy.cc
File sandbox/mac/policy.cc (right):

https://codereview.chromium.org/310833003/diff/1/sandbox/mac/policy.cc#newcode49
sandbox/mac/policy.cc:49: for
(BootstrapSandboxPolicy::NamedRules::const_iterator it =
You can just write "for (const auto& kv : policy.rules)" now (or if you don’t
like auto, you can use “const BootstrapSandboxPolicy::NamedRules::value_type&”
or “const std::pair<std::string, Rule>&”.

[Robert Sesek, 2014-06-03 17:30:52]

https://codereview.chromium.org/310833003/diff/1/sandbox/mac/policy.cc
File sandbox/mac/policy.cc (right):

https://codereview.chromium.org/310833003/diff/1/sandbox/mac/policy.cc#newcode49
sandbox/mac/policy.cc:49: for
(BootstrapSandboxPolicy::NamedRules::const_iterator it =
On 2014/06/03 16:57:27, Mark Mentovai wrote:
> You can just write "for (const auto& kv : policy.rules)" now (or if you don’t
> like auto, you can use “const BootstrapSandboxPolicy::NamedRules::value_type&”
> or “const std::pair<std::string, Rule>&”.

I like const auto&.

[Robert Sesek, 2014-06-03 17:30:54]

The CQ bit was checked by rsesek@chromium.org

[commit-bot: I haz the power, 2014-06-03 17:31:50]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rsesek@chromium.org/310833003/20001

[commit-bot: I haz the power, 2014-06-04 00:10:28]

Change committed as 274676