[ChromeDriver] Added CommandListener interface

chrome/test/chromedriver/command_listener.cc

Index: chrome/test/chromedriver/command_listener.cc
diff --git a/sync/engine/commit_contribution.cc b/chrome/test/chromedriver/command_listener.cc
similarity index 52%
copy from sync/engine/commit_contribution.cc
copy to chrome/test/chromedriver/command_listener.cc
index cef02b57b9021598370cc9e7554ab15042a9489a..80437b97bf0cff5f3db39f99a23383764acde8b9 100644
--- a/sync/engine/commit_contribution.cc
+++ b/chrome/test/chromedriver/command_listener.cc
@@ -2,12 +2,11 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "sync/engine/commit_contribution.h"
+#include "chrome/test/chromedriver/command_listener.h"
+#include "chrome/test/chromedriver/chrome/status.h"
 
-namespace syncer {
+CommandListener::~CommandListener() {}
 
-CommitContribution::CommitContribution() {}
-
-CommitContribution::~CommitContribution() {}
-
-}  // namespace syncer
+Status CommandListener::OnCommand(const std::string& command_name) {

[stgao, 2014/06/15 23:54:40]

I think this default implementation file could be removed, if we don't create
any instance of CommandListener.

[klm, 2014/06/16 16:11:29]

+1 -- pure virtual, just like other interfaces, e.g. there is no web_view.cc.
The virtual destructor body should be inline in the header.

+  return Status(kOk);
+}