Updated latency fixture and refactored connection helper

remoting/test/rgb_value.h

Index: remoting/test/rgb_value.h
diff --git a/remoting/test/rgb_value.h b/remoting/test/rgb_value.h
new file mode 100644
index 0000000000000000000000000000000000000000..2d2b872b1d638ad587ef7c8938643bab0dd7747d
--- /dev/null
+++ b/remoting/test/rgb_value.h
@@ -0,0 +1,26 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef REMOTING_TEST_RGB_VALUE_H_
+#define REMOTING_TEST_RGB_VALUE_H_
+
+namespace remoting {
+namespace test {
+
+// Used to store a RGB color.
+struct RGBValue {
+  // Black is the default color for tests.

[joedow, 2015/07/24 13:50:58]

Not sure what the value in this comment is for, I think it should be moved up a
line and state that the default c'tor will initialize the value to black. 
Whether it is used for tests or not is a usage detail independent of the
definition of the struct.

[liaoyuke, 2015/07/25 01:43:49]

Done.

+  RGBValue();
+  RGBValue(int r, int g, int b);
+  ~RGBValue();
+
+  int red;
+  int green;
+  int blue;
+};
+
+}  // namespace test
+}  // namespace remoting
+
+#endif  // REMOTING_TEST_RGB_VALUE_H_