Added image pattern comparison logic for test interface and fixture.

remoting/test/app_remoting_latency_test_fixture.h

Index: remoting/test/app_remoting_latency_test_fixture.h
diff --git a/remoting/test/app_remoting_latency_test_fixture.h b/remoting/test/app_remoting_latency_test_fixture.h
index 1cebe84a935ed3cf57e18177f1843bbc74592c66..200707818dec471bbcbf95b41b169719036a08de 100644
--- a/remoting/test/app_remoting_latency_test_fixture.h
+++ b/remoting/test/app_remoting_latency_test_fixture.h
@@ -30,8 +30,6 @@ struct RemoteApplicationDetails;
 class AppRemotingConnectionHelper;
 class TestVideoRenderer;
 
-typedef uint32 RgbaColor;
-
 // Creates a connection to a remote host which is available for tests to use.
 // Provides convenient methods to create test cases to measure the input and
 // rendering latency between client and the remote host.
@@ -46,7 +44,7 @@ class AppRemotingLatencyTestFixture : public testing::Test {
   // Set expected image pattern for comparison and a matched reply will be
   // called when the pattern is matched.
   void SetExpectedImagePattern(const webrtc::DesktopRect& expected_rect,
-                               const RgbaColor& expected_color);
+                               const uint32_t expected_avg_color);

[joedow, 2015/07/10 23:39:15]

why is the uint32 marked const? It is passed by value...

[liaoyuke, 2015/07/13 16:05:38]

Done.

 
   // Waits for an image pattern matched reply up to |max_wait_time|. Returns
   // true if we received a response within the maximum time limit.
@@ -54,7 +52,7 @@ class AppRemotingLatencyTestFixture : public testing::Test {
   bool WaitForImagePatternMatched(const base::TimeDelta& max_wait_time);
 
   // Name of the application being tested.
-  // NOTE: must be initialized in the constructor of derived class.
+  // NOTE: Must be initialized in the constructor of derived class.
   std::string application_name_;
 
  private: