Refactor BlimpConnection to TCPConnection

blimp/engine/browser_tests/blimp_browser_test.cc

Index: blimp/engine/browser_tests/blimp_browser_test.cc
diff --git a/blimp/engine/browser_tests/blimp_browser_test.cc b/blimp/engine/browser_tests/blimp_browser_test.cc
index e8ed5b9eaa55cb6c4b6f4c94f8933a0dfa56b68d..5440666b78bc7c6c17ce1aafb9586ddbde53d46c 100644
--- a/blimp/engine/browser_tests/blimp_browser_test.cc
+++ b/blimp/engine/browser_tests/blimp_browser_test.cc
@@ -93,6 +93,8 @@ void BlimpBrowserTest::SetUpCommandLine(base::CommandLine* command_line) {
     command_line->AppendSwitchASCII(blimp::engine::kEnginePort, "0");
   }
 
+  // TODO(perumaal): Switch to gRPC when it's ready. See crbug.com/659279.

[Wez, 2016/11/09 22:47:17]

nit: Similarly, this placeholder doesn't really add anything that the bug
doesn't already contain.

[perumaal, 2016/11/10 02:05:05]

These are more like breadcrumbs for me to track and also help the reviewers
understand where the gRPC pieces fit in... but I agree it's duplicating what's
in the bug.

It's hard to see the big picture otherwise IMO as a reviewer without these
bread-crumbs, so one way would be for me to remove these before checkin.

+
   base::FilePath src_root;
   PathService::Get(base::DIR_SOURCE_ROOT, &src_root);
   command_line->AppendSwitchASCII(