From 8ea7a167522a24be192e958af46a41d49e78504c Mon Sep 17 00:00:00 2001 From: "ajwong@chromium.org" Date: Mon, 4 Oct 2010 19:48:42 +0000 Subject: This is a monster CL. It started as an attempt to put the decoder onto another thread. However, this became complicated due to multiple object ownership transfers and coupling between the decode layer and the network layer; the decoder's states were highly coupled with how the network packets were processed. This could probably be broken up slightly, but at this point, it's easier to just commit as a whole The refactor includes: 1) Making the decoder interface unaware of "network packet" types. 2) Making the network layer process packets in order. 3) Threading through asynchronous APIs all over the place. 4) Simplifying the rectangle update protocol. 5) Cleaning up object lifetime and ownership semantics between the decode layer and the renderer. As of right now, the Verbatim format is still broken on the encode side because it uses the old protocol. BUG=52883, 57351 TEST=still connects to chromoting_simple_host Review URL: http://codereview.chromium.org/3305001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61402 0039d316-1c4b-4281-b951-d872f2087c98 --- remoting/host/chromoting_host_context.h | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'remoting/host/chromoting_host_context.h') diff --git a/remoting/host/chromoting_host_context.h b/remoting/host/chromoting_host_context.h index 1f6b011..713afb8 100644 --- a/remoting/host/chromoting_host_context.h +++ b/remoting/host/chromoting_host_context.h @@ -20,6 +20,10 @@ class ChromotingHostContext { ChromotingHostContext(); virtual ~ChromotingHostContext(); + // TODO(ajwong): Move the Start/Stop methods out of this class. Then + // create a static factory for construction, and destruction. We + // should be able to remove the need for virtual functions below with that + // design, while preserving the relative simplicity of this API. virtual void Start(); virtual void Stop(); -- cgit v1.1