In catapult repo at
//devil/devil/android/sdk/adb_wrapper.py
there are following issues:

1. PersistentShell::RunCommand::run_cmd receives \r\r\n as a new-line separator on Windows. It leads to empty string at the end of list produced by splitlines(). [kraynov@ gonna fix it]

2. If run_cmd runs a command not ending with a new-line it leads to get exit code concatenated with the last line.

3. Long commands passed to run_cmd lead in getting command echo-ed back by interactive shell which implements interactive scrolling. It pollutes the output.

... and, for sure, some other bugs we're not aware of yet

It worked fine because PersistentShell::RunCommand used only once in catapult codebase (in //systrace/systrace/tracing_agents/atrace_agent.py) and it doesn't seems to be tested on Windows.

So basically I'd expect:
1. Fixing these issues
2. Tests

I gonna land a CL quick-fixing systrace on Windows, but not going to fix other problems. Anyone willing to fix it is really welcome to take this bug ;) Thanks!