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

Unified Diff: ui/gfx/compositor/compositor.h

Issue 10067034: RefCounted types should not have public destructors, printing/ and ui/ (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix 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 | « printing/backend/print_backend_win.cc ('k') | ui/gfx/gl/gl_context_cgl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/gfx/compositor/compositor.h
diff --git a/ui/gfx/compositor/compositor.h b/ui/gfx/compositor/compositor.h
index 5d8cc96b629023c0bfd2c6a2d150fe3179bbe509..9475cf62445acfe2de8b85b8450cbaba2fe1cf07 100644
--- a/ui/gfx/compositor/compositor.h
+++ b/ui/gfx/compositor/compositor.h
@@ -38,7 +38,7 @@ class Layer;
// a global object.
class COMPOSITOR_EXPORT ContextFactory {
public:
- virtual ~ContextFactory() { }
+ virtual ~ContextFactory() {}
// Gets the global instance.
static ContextFactory* GetInstance();
@@ -97,14 +97,18 @@ class COMPOSITOR_EXPORT DefaultContextFactory : public ContextFactory {
class COMPOSITOR_EXPORT Texture : public base::RefCounted<Texture> {
public:
Texture(bool flipped, const gfx::Size& size);
- virtual ~Texture();
unsigned int texture_id() const { return texture_id_; }
void set_texture_id(unsigned int id) { texture_id_ = id; }
bool flipped() const { return flipped_; }
gfx::Size size() const { return size_; }
+ protected:
+ virtual ~Texture();
+
private:
+ friend class base::RefCounted<Texture>;
+
unsigned int texture_id_;
bool flipped_;
gfx::Size size_;
« no previous file with comments | « printing/backend/print_backend_win.cc ('k') | ui/gfx/gl/gl_context_cgl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698