wm: Fix an uncommon memory leak in the compositor code.

compositor/gles/opengles_visitor.h

 // Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef WINDOW_MANAGER_COMPOSITOR_GLES_GLES_VISITOR_H_
 #define WINDOW_MANAGER_COMPOSITOR_GLES_GLES_VISITOR_H_
 
 #include <GLES2/gl2.h>
 #include <EGL/egl.h>
 #include <EGL/eglext.h>
@@ skipping 18 matching lines @@
 // This class vists an actor tree and draws it using OpenGLES
 class OpenGlesDrawVisitor : virtual public RealCompositor::ActorVisitor {
  public:
   OpenGlesDrawVisitor(Gles2Interface* gl,
                       RealCompositor* compositor,
                       Compositor::StageActor* stage);
   virtual ~OpenGlesDrawVisitor();
   void set_has_fullscreen_actor(bool has_fullscreen_actor) {
     has_fullscreen_actor_ = has_fullscreen_actor;
   }
-  void set_damaged_region(Rect damaged_region) {
+  void set_damaged_region(const Rect& damaged_region) {
     damaged_region_ = damaged_region;
   }
 
   void BindImage(const ImageContainer* container,
                  RealCompositor::QuadActor* actor);
 
   virtual void VisitActor(RealCompositor::Actor* actor) {}
   virtual void VisitStage(RealCompositor::StageActor* actor);
   virtual void VisitContainer(RealCompositor::ContainerActor* actor);
   virtual void VisitImage(RealCompositor::ImageActor* actor);
@@ skipping 11 matching lines @@
   // scissoring if the stack is now empty.
   void PushScissorRect(const Rect& scissor);
   void PopScissorRect();
 
  private:
   Gles2Interface* gl_;  // Not owned.
   RealCompositor* compositor_;  // Not owned.
   Compositor::StageActor* stage_;  // Not owned.
   XConnection* x_connection_;  // Not owned.
 
-  TexColorShader* tex_color_shader_;
-  TexShadeShader* tex_shade_shader_;
-  NoAlphaColorShader* no_alpha_color_shader_;
-  NoAlphaShadeShader* no_alpha_shade_shader_;
+  scoped_ptr<TexColorShader> tex_color_shader_;
+  scoped_ptr<TexShadeShader> tex_shade_shader_;
+  scoped_ptr<NoAlphaColorShader> no_alpha_color_shader_;
+  scoped_ptr<NoAlphaShadeShader> no_alpha_shade_shader_;
 
   EGLDisplay egl_display_;
   EGLSurface egl_surface_;
   bool egl_surface_is_capable_of_partial_updates_;
   EGLContext egl_context_;
 
   // Matrix state
   Matrix4 projection_;
 
   // Scissor rect data.
@@ skipping 67 matching lines @@
   // Not owned.
   Gles2Interface* gl_;
 
   // EGLImage
   EGLImageKHR egl_image_;
 };
 
 }  // namespace window_manager
 
 #endif  // WINDOW_MANAGER_COMPOSITOR_GLES_GLES_VISITOR_H_