Update the Ready Mode UI in response to feedback from UI leads, Chrome Frame ...

chrome_frame/ready_mode/internal/ready_prompt_content.cc

Index: chrome_frame/ready_mode/internal/ready_prompt_content.cc
===================================================================
--- chrome_frame/ready_mode/internal/ready_prompt_content.cc	(revision 72310)
+++ chrome_frame/ready_mode/internal/ready_prompt_content.cc	(working copy)
@@ -4,15 +4,15 @@
 
 #include "chrome_frame/ready_mode/internal/ready_prompt_content.h"
 

[erikwright (departed), 2011/01/24 06:58:39]

Oops. This is missing atlbase.h, atlwin.h, and Exdisp.h.

[robertshield, 2011/01/24 16:10:21]

Add them? ;-)

[erikwright (departed), 2011/01/24 17:14:01]

Yeah. Wanted to get this out for initial feedback ASAP. My workstation crashed
before I discovered this.

Anyway, not needed now.

-#include <atlbase.h>
-#include <atlwin.h>
-
 #include "base/logging.h"
 #include "chrome_frame/ready_mode/internal/ready_mode_state.h"
 #include "chrome_frame/ready_mode/internal/ready_prompt_window.h"
 
-ReadyPromptContent::ReadyPromptContent(ReadyModeState* ready_mode_state)
+ReadyPromptContent::ReadyPromptContent(ReadyModeState* ready_mode_state,
+                                       IWebBrowser2* web_browser)
     : ready_mode_state_(ready_mode_state) {
+  web_browser->AddRef();
+  web_browser_.Attach(web_browser);
 }
 
 ReadyPromptContent::~ReadyPromptContent() {
@@ -29,7 +29,8 @@
 
   // The window owns itself upon call to Initialize.
   ReadyPromptWindow* new_window_ = new ReadyPromptWindow();

[robertshield, 2011/01/24 16:10:21]

new_window_ is not a member so shouldn't have the trailing underscore.

-  window_ = new_window_->Initialize(frame, ready_mode_state_.release());
+  window_ = new_window_->Initialize(frame, ready_mode_state_.release(),
+                                    web_browser_);
 
   return window_ != NULL;
 }