-
Olga Sharonova authored
This reverts commit c9552c88. Reason for revert: CrostiniPortForwarderTest.AddPortDuplicateFail failing on Linux Chromium OS ASan LSan Tests (1) First failing build: https://ci.chromium.org/p/chromium/builders/ci/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/36750 Failure example: https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8890488549187658624/+/steps/unit_tests/0/logs/Deterministic_failure:_CrostiniPortForwarderTest.AddPortDuplicateFail__status_CRASH_/0 Original change's description: > Crostini port forwarding activate and add port > > Port forwarding in Cros will allow users to add new ports and activate, > deactivate and remove existing ports. > > When any of the utilities are called, the port forwarding library will > invoke the permission broker client to change the ChromeOS port > forwarding settings. > > In future work we will also save port forwarding preferences such that a > user's preferences will persist over container shutdown. > > Bug: chromium:848127 > Test: crostini_port_forwarder_unittest.cc and everything builds > Change-Id: I9a0478f1486252fb09fab18ab3261e03d00eb480 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1999972 > Reviewed-by: David Munro <davidmunro@google.com> > Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org> > Reviewed-by: Nic Hollingum <hollingum@google.com> > Commit-Queue: Matthew Chen <matterchen@google.com> > Cr-Commit-Position: refs/heads/master@{#734303} TBR=hashimoto@chromium.org,hollingum@google.com,davidmunro@google.com,matterchen@google.com Change-Id: Iabb6d856090b1f34776b547cd1b6ba652ebfb055 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: chromium:848127 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015085Reviewed-by:
Olga Sharonova <olka@chromium.org> Commit-Queue: Olga Sharonova <olka@chromium.org> Cr-Commit-Position: refs/heads/master@{#734416}
0477cee9