A couple more improvements to chrome_frame_tests.exe:

Add a useful return value to LaunchIEAndNavigate, indicating which step failed.

Also fix a crash in the HungCOMCallDetector if LaunchIEAndNavigate() fails.

BUG=NONE
TEST=NONE

Review URL: https://chromiumcodereview.appspot.com/9160029

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@119070 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 17 insertions, 4 deletions

diff --git a/chrome_frame/test/ie_event_sink.cc b/chrome_frame/test/ie_event_sink.cc
index dde9ebe..bdff4d2 100644
--- a/chrome_frame/test/ie_event_sink.cc
+++ b/chrome_frame/test/ie_event_sink.cc
@@ -453,15 +453,28 @@ HWND IEEventSink::GetRendererWindowSafe() {
   return renderer_window;
 }
 
-HRESULT IEEventSink::LaunchIEAndNavigate(
-    const std::wstring& navigate_url, IEEventListener* listener) {
+HRESULT IEEventSink::LaunchIEAndNavigate(const std::wstring& navigate_url,
+                                         IEEventListener* listener) {
   listener_ = listener;
   HRESULT hr = LaunchIEAsComServer(web_browser2_.Receive());
   if (SUCCEEDED(hr)) {
     web_browser2_->put_Visible(VARIANT_TRUE);
-    Attach(web_browser2_);
-    hr = Navigate(navigate_url);
+    hr = Attach(web_browser2_);
+    if (SUCCEEDED(hr)) {
+      hr = Navigate(navigate_url);
+      if (FAILED(hr)) {
+        LOG(ERROR) << "Failed to navigate IE to " << navigate_url << ", hr = 0x"
+                   << std::hex << hr;
+      }
+    } else {
+      LOG(ERROR) << "Failed to attach to web browser event sink for "
+                 << navigate_url << ", hr = 0x" << std::hex << hr;
+    }
+  } else {
+    LOG(ERROR) << "Failed to Launch IE for " << navigate_url << ", hr = 0x"
+               << std::hex << hr;
   }
+
   return hr;
 }