Attempt at fixing crash in balloon code. The current code does a

chrome/browser/ui/views/notifications/balloon_view_views.h

 // Copyright (c) 2012 The Chromium 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 CHROME_BROWSER_UI_VIEWS_NOTIFICATIONS_BALLOON_VIEW_VIEWS_H_
 #define CHROME_BROWSER_UI_VIEWS_NOTIFICATIONS_BALLOON_VIEW_VIEWS_H_
 
 #include "base/basictypes.h"
 #include "base/memory/scoped_ptr.h"
-#include "base/memory/weak_ptr.h"
 #include "chrome/browser/notifications/balloon.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "ui/base/animation/animation_delegate.h"
 #include "ui/gfx/rect.h"
 #include "ui/views/controls/button/button.h"
 #include "ui/views/controls/button/menu_button_listener.h"
 #include "ui/views/widget/widget_delegate.h"
 
 class BalloonCollection;
 class BalloonViewHost;
 class NotificationOptionsMenuModel;
 
 namespace gfx {
 class Path;
 }
 
 namespace ui {
 class SlideAnimation;
 }
 
 namespace views {
 class ImageButton;
 class Label;
 class MenuButton;
 class MenuRunner;
 }
 
-// A balloon view is the UI component for a desktop notification toasts.
+// A balloon view is the UI component for a desktop notification toast.
 // It draws a border, and within the border an HTML renderer.
 class BalloonViewImpl : public BalloonView,
                         public views::MenuButtonListener,
                         public views::WidgetDelegateView,
                         public views::ButtonListener,
                         public content::NotificationObserver,
                         public ui::AnimationDelegate {
  public:
   explicit BalloonViewImpl(BalloonCollection* collection);
   virtual ~BalloonViewImpl();
@@ skipping 14 matching lines @@
   virtual void OnBoundsChanged(const gfx::Rect& previous_bounds) OVERRIDE;
   virtual gfx::Size GetPreferredSize() OVERRIDE;
 
   // views::MenuButtonListener interface.
   virtual void OnMenuButtonClicked(views::View* source,
                                    const gfx::Point& point) OVERRIDE;
 
   // views::WidgetDelegate interface.
   virtual void OnDisplayChanged() OVERRIDE;
   virtual void OnWorkAreaChanged() OVERRIDE;
+  virtual void DeleteDelegate() OVERRIDE;
 
   // views::ButtonListener interface.
   virtual void ButtonPressed(views::Button* sender,
                              const ui::Event&) OVERRIDE;
 
   // content::NotificationObserver interface.
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
   // ui::AnimationDelegate interface.
   virtual void AnimationProgressed(const ui::Animation* animation) OVERRIDE;
 
   // Initializes the options menu.
   void CreateOptionsMenu();
 
   // Masks the contents to fit within the frame.
   void GetContentsMask(const gfx::Rect& contents_rect, gfx::Path* path) const;
 
   // Masks the frame for the rounded corners of the shadow-bubble.
   void GetFrameMask(const gfx::Rect&, gfx::Path* path) const;
 
   // Adjust the contents window size to be appropriate for the frame.
   void SizeContentsWindow();
 
-  // Do the delayed close work.
-  void DelayedClose(bool by_user);
-
   // The height of the balloon's shelf.
   // The shelf is where is close button is located.
   int GetShelfHeight() const;
 
   // The height of the part of the frame around the balloon.
   int GetBalloonFrameHeight() const;
 
   int GetTotalWidth() const;
   int GetTotalHeight() const;
 
@@ skipping 18 matching lines @@
   // Pointer owned by the View subclass.
   views::Widget* frame_container_;
 
   // The window that contains the contents of the notification.
   // Pointer owned by the View subclass.
   views::Widget* html_container_;
 
   // The renderer of the HTML contents.
   scoped_ptr<BalloonViewHost> html_contents_;
 
-  // The following factory is used to call methods at a later time.
-  base::WeakPtrFactory<BalloonViewImpl> method_factory_;
-
   // Pointer to sub-view is owned by the View sub-class.
   views::ImageButton* close_button_;
 
   // Pointer to sub-view is owned by View class.
   views::Label* source_label_;
 
   // An animation to move the balloon on the screen as its position changes.
   scoped_ptr<ui::SlideAnimation> animation_;
   gfx::Rect anim_frame_start_;
   gfx::Rect anim_frame_end_;
 
   // The options menu.
   scoped_ptr<NotificationOptionsMenuModel> options_menu_model_;
   scoped_ptr<views::MenuRunner> menu_runner_;
   views::MenuButton* options_menu_button_;
 
   content::NotificationRegistrar notification_registrar_;
 
   // Set to true if this is browser generate web UI.
   bool enable_web_ui_;
 
+  // Most recent value passed to Close().
+  bool closed_by_user_;
+
   DISALLOW_COPY_AND_ASSIGN(BalloonViewImpl);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_NOTIFICATIONS_BALLOON_VIEW_VIEWS_H_