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

Unified Diff: remoting/host/screen_recorder.h

Issue 10066042: Refcounted types should not have public destructors, remoting/ edition (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Few fixes Created 8 years, 8 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 | « remoting/host/gaia_oauth_client.cc ('k') | remoting/host/screen_recorder.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/screen_recorder.h
diff --git a/remoting/host/screen_recorder.h b/remoting/host/screen_recorder.h
index 0f364f2e0bd74394fe6ddea74a2f5f59a7186e6c..64e079f8c64a2d22801afb9cb4cf2394170b9f2a 100644
--- a/remoting/host/screen_recorder.h
+++ b/remoting/host/screen_recorder.h
@@ -84,8 +84,6 @@ class ScreenRecorder : public base::RefCountedThreadSafe<ScreenRecorder> {
Capturer* capturer,
Encoder* encoder);
- virtual ~ScreenRecorder();
-
// Start recording.
void Start();
@@ -106,6 +104,9 @@ class ScreenRecorder : public base::RefCountedThreadSafe<ScreenRecorder> {
void UpdateSequenceNumber(int64 sequence_number);
private:
+ friend class base::RefCountedThreadSafe<ScreenRecorder>;
+ virtual ~ScreenRecorder();
+
// Getters for capturer and encoder.
Capturer* capturer();
Encoder* encoder();
« no previous file with comments | « remoting/host/gaia_oauth_client.cc ('k') | remoting/host/screen_recorder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698