Schedule posttask to decide Uma optin option offering

chrome/browser/ui/views/session_crashed_bubble_view.h

 // Copyright 2014 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_SESSION_CRASHED_BUBBLE_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_SESSION_CRASHED_BUBBLE_VIEW_H_
 
 #include "chrome/browser/ui/tabs/tab_strip_model_observer.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "content/public/browser/web_contents_observer.h"
 #include "ui/views/bubble/bubble_delegate.h"
 #include "ui/views/controls/button/button.h"
 #include "ui/views/controls/styled_label_listener.h"
 
 namespace views {
 class Checkbox;
 class GridLayout;
 class LabelButton;
 class Widget;
 }
 
 namespace content {
 class WebContents;
 class RenderViewHost;
 }
 
+namespace {
+  class BrowserRemovalObserver;

[Alexei Svitkine (slow), 2014/05/20 15:52:42]

I think you can make it an inner class inside SessionCrashedBubbleView and then
just declare it (i.e. "class BrowserRemovalObserver;") in the private section in
the header, while still keeping the definition in the .cc.

[yao, 2014/05/20 16:25:44]

Done.

+}
+
 class Browser;
 
 // It creates a session restore request bubble when the previous session has
 // crashed. It also presents an option to enable metrics reporting, if it not
 // enabled already.
 class SessionCrashedBubbleView
     : public views::BubbleDelegateView,
       public views::ButtonListener,
       public views::StyledLabelListener,
       public content::WebContentsObserver,
       public content::NotificationObserver,
       public TabStripModelObserver {
  public:
   static void Show(Browser* browser);
 
  private:
   SessionCrashedBubbleView(views::View* anchor_view,
                            Browser* browser,
-                           content::WebContents* web_contents);
+                           content::WebContents* web_contents,
+                           bool offer_uma_optin);
   virtual ~SessionCrashedBubbleView();
 
+  // Creates and shows the session crashed bubble, with |offer_uma_optin|
+  // indicating whether the UMA opt-in checkbox should be shown. Called
+  // by Show() after checking whether the UMA option should be presented.
+  static void ShowForReal(BrowserRemovalObserver* browser_observer,
+                          Browser* browser,
+                          bool offer_uma_optin);
+
   // WidgetDelegateView methods.
   virtual views::View* GetInitiallyFocusedView() OVERRIDE;
 
   // views::BubbleDelegateView methods.
   virtual void Init() OVERRIDE;
 
   // views::ButtonListener methods.
   virtual void ButtonPressed(views::Button* sender,
                              const ui::Event& event) OVERRIDE;
 
@@ skipping 46 matching lines @@
 
   // Button for the user to confirm a session restore.
   views::LabelButton* restore_button_;
 
   // Button for the user to close this bubble.
   views::LabelButton* close_;
 
   // Checkbox for the user to opt-in to UMA reporting.
   views::Checkbox* uma_option_;
 
+  // Whether or not the UMA opt-in option should be shown.
+  bool offer_uma_optin_;
+
   // Whether or not a navigation has started on current tab.
   bool started_navigation_;
 
   DISALLOW_COPY_AND_ASSIGN(SessionCrashedBubbleView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_SESSION_CRASHED_BUBBLE_VIEW_H_