diff options
| author | sergeyu@chromium.org <sergeyu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-01-17 20:30:11 +0000 |
|---|---|---|
| committer | sergeyu@chromium.org <sergeyu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-01-17 20:30:11 +0000 |
| commit | 8144871ca2fb49c62c3a77aa95a3045e3c7a9c99 (patch) | |
| tree | bbc3ca14e2daf8f0cbc0877b7ee81a7275da7710 /remoting/host/log_to_server.h | |
| parent | f47a5b90a630194445e4c5cafc331acdddcc0778 (diff) | |
| download | chromium_src-8144871ca2fb49c62c3a77aa95a3045e3c7a9c99.zip chromium_src-8144871ca2fb49c62c3a77aa95a3045e3c7a9c99.tar.gz chromium_src-8144871ca2fb49c62c3a77aa95a3045e3c7a9c99.tar.bz2 | |
Add ChromotingHost::RemoveStatusObserver().
Now HostStatusObservers can be added or removed at any time.
ObserverList<> is used to store the list of observers. This allows deletion of status observers before host is destroyed. It was previously causing a crash described in http://crbug.com/110196 .
BUG=110196
Review URL: https://chromiumcodereview.appspot.com/9231002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117947 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'remoting/host/log_to_server.h')
| -rw-r--r-- | remoting/host/log_to_server.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/remoting/host/log_to_server.h b/remoting/host/log_to_server.h index 4d5d77d7..76fba6c 100644 --- a/remoting/host/log_to_server.h +++ b/remoting/host/log_to_server.h @@ -34,7 +34,8 @@ class LogToServer : public base::NonThreadSafe, public HostStatusObserver, public SignalStrategy::Listener { public: - explicit LogToServer(SignalStrategy* signal_strategy); + explicit LogToServer(ChromotingHost* host, + SignalStrategy* signal_strategy); virtual ~LogToServer(); // Logs a session state change. Currently, this is either |