tree 2099ec421f93a25c7e701f9b2a3ce5cb91864265
parent 380a5d2d15ec0e280b9c4ab24c8121fb94c613fa
author ajurkowski <ajurkowski@google.com> 1623373238 -0700
committer Copybara-Service <copybara-worker@google.com> 1623373328 -0700

Do not expand flags in invocation policy twice.

`InvocationPolicyEnforcer` [expands all flags before setting](https://github.com/bazelbuild/bazel/blob/60fc2d18fadec5f99849b78bce761da62dd78c32/src/main/java/com/google/devtools/common/options/InvocationPolicyEnforcer.java#L124-L125) them to later call into the `OptionsParser` to set their values, which [repeats the process for the flags with expansions/implicit requirements](https://github.com/bazelbuild/bazel/blob/60fc2d18fadec5f99849b78bce761da62dd78c32/src/main/java/com/google/devtools/common/options/OptionsParserImpl.java#L549-L556). Refactor `OptionsParser` to expose a method for setting flag value without the expansion so that `InvocationPolicyEnforcer` can skip the second, unnecessary expansion.

PiperOrigin-RevId: 378775962