Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3)

Unified Diff: tools/android/forwarder2/forwarder.h

Issue 19478003: Remove Thread wrapper class in forwarder2. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address Marcus' comments Created 7 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/android/forwarder2/device_listener.cc ('k') | tools/android/forwarder2/forwarder.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/android/forwarder2/forwarder.h
diff --git a/tools/android/forwarder2/forwarder.h b/tools/android/forwarder2/forwarder.h
index 1ba0923271e39f6dd466a6131dca231e88db0a51..651b5e80f7175d203832167b2f215e4d89854a39 100644
--- a/tools/android/forwarder2/forwarder.h
+++ b/tools/android/forwarder2/forwarder.h
@@ -5,35 +5,15 @@
#ifndef TOOLS_ANDROID_FORWARDER2_FORWARDER_H_
#define TOOLS_ANDROID_FORWARDER2_FORWARDER_H_
-#include "base/compiler_specific.h"
-#include "base/logging.h"
#include "base/memory/scoped_ptr.h"
-#include "tools/android/forwarder2/socket.h"
-#include "tools/android/forwarder2/thread.h"
+#include "base/threading/thread.h"
namespace forwarder2 {
-class Forwarder : public Thread {
- public:
- Forwarder(scoped_ptr<Socket> socket1, scoped_ptr<Socket> socket2);
- virtual ~Forwarder();
+class Socket;
- // This object self deletes after running, so one cannot join.
- // Thread:
- virtual void Join() OVERRIDE;
+void StartForwarder(scoped_ptr<Socket> socket1, scoped_ptr<Socket> socket2);
- protected:
- // Thread:
- // This object self deletes after running.
- virtual void Run() OVERRIDE;
-
- private:
- scoped_ptr<Socket> socket1_;
- scoped_ptr<Socket> socket2_;
-
- DISALLOW_COPY_AND_ASSIGN(Forwarder);
-};
-
-} // namespace forwarder
+} // namespace forwarder2
#endif // TOOLS_ANDROID_FORWARDER2_FORWARDER_H_
« no previous file with comments | « tools/android/forwarder2/device_listener.cc ('k') | tools/android/forwarder2/forwarder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698