Schedule posttask to decide Uma optin option offering

chrome/browser/ui/views/session_crashed_bubble_view.h

Index: chrome/browser/ui/views/session_crashed_bubble_view.h
diff --git a/chrome/browser/ui/views/session_crashed_bubble_view.h b/chrome/browser/ui/views/session_crashed_bubble_view.h
index 8986da6c983bff5b01a1727d9b83bd3db93ab84b..61e649c1fde298ced30d13aa765b19dd67c23f21 100644
--- a/chrome/browser/ui/views/session_crashed_bubble_view.h
+++ b/chrome/browser/ui/views/session_crashed_bubble_view.h
@@ -25,6 +25,10 @@ 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
@@ -43,9 +47,17 @@ class SessionCrashedBubbleView
  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;
 
@@ -112,6 +124,9 @@ class SessionCrashedBubbleView
   // 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_;