DescriptionRevert of New run shell implementation for DeviceUtils (patchset #8 id:140001 of https://codereview.chromium.org/659533002/)
Reason for revert:
Broke some telemetry telemetry unittests and perf unittests
Original issue's description:
> New run shell implementation for DeviceUtils
>
> The main differences are:
> - it uses AdbWrapper.Shell to actually execute the command.
> - when the cmd is supplied as a list of a command and its arguments,
> the arguments are quoted to prevent them from being (mis)interpreted
> by the shell.
> - a new single_line option to check that the output produces contains a single line, and return the value of that line.
>
> BUG=267773
>
> Committed: https://crrev.com/ab18bcca0cc1475cf381211e595e75c1146c9115
> Cr-Commit-Position: refs/heads/master@{#300237}
TBR=jbudorick@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=267773
Committed: https://crrev.com/3938cc489efbde98f6cdeae6cb880ef3ef97b66b
Cr-Commit-Position: refs/heads/master@{#300258}
Patch Set 1 #
Created: 6 years, 2 months ago
(Patch set is too large to download)
Messages
Total messages: 9 (2 generated)
|