diff options
| author | jamesr@chromium.org <jamesr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-01-30 07:40:54 +0000 |
|---|---|---|
| committer | jamesr@chromium.org <jamesr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-01-30 07:40:54 +0000 |
| commit | 4514d4ed0303ca25eb184e842567f61ed8812823 (patch) | |
| tree | 6318274985c32dbc7856d52e1e8600cfc7dfb783 /media/tools | |
| parent | 876ba22e2898771c466e2bb84c3c7c83aa9c61cd (diff) | |
| download | chromium_src-4514d4ed0303ca25eb184e842567f61ed8812823.zip chromium_src-4514d4ed0303ca25eb184e842567f61ed8812823.tar.gz chromium_src-4514d4ed0303ca25eb184e842567f61ed8812823.tar.bz2 | |
Cleanups to enable clang warning about protected non-virtual RefCounted destructor
This makes refcounted classes that do not have derived types have private
destructors instead of protected. If they did have derived classes, they would
need protected *virtual* destructors.
BUG=
Review URL: https://chromiumcodereview.appspot.com/11896117
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179555 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media/tools')
| -rw-r--r-- | media/tools/player_x11/gl_video_renderer.h | 3 | ||||
| -rw-r--r-- | media/tools/player_x11/x11_video_renderer.h | 3 |
2 files changed, 2 insertions, 4 deletions
diff --git a/media/tools/player_x11/gl_video_renderer.h b/media/tools/player_x11/gl_video_renderer.h index b8818a1..dcda343 100644 --- a/media/tools/player_x11/gl_video_renderer.h +++ b/media/tools/player_x11/gl_video_renderer.h @@ -23,11 +23,10 @@ class GlVideoRenderer : public base::RefCountedThreadSafe<GlVideoRenderer> { void Paint(media::VideoFrame* video_frame); - protected: + private: friend class base::RefCountedThreadSafe<GlVideoRenderer>; ~GlVideoRenderer(); - private: // Initializes GL rendering for the given dimensions. void Initialize(gfx::Size coded_size, gfx::Rect visible_rect); diff --git a/media/tools/player_x11/x11_video_renderer.h b/media/tools/player_x11/x11_video_renderer.h index 05c624c..18b76ba 100644 --- a/media/tools/player_x11/x11_video_renderer.h +++ b/media/tools/player_x11/x11_video_renderer.h @@ -24,11 +24,10 @@ class X11VideoRenderer : public base::RefCountedThreadSafe<X11VideoRenderer> { void Paint(media::VideoFrame* video_frame); - protected: + private: friend class base::RefCountedThreadSafe<X11VideoRenderer>; ~X11VideoRenderer(); - private: // Initializes X11 rendering for the given dimensions. void Initialize(gfx::Size coded_size, gfx::Rect visible_rect); |