Correctly locate the browser frame window to prevent a crash.

chrome/browser/ui/views/first_run_bubble.cc

 // Copyright (c) 2011 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.
 
 #include "chrome/browser/ui/views/first_run_bubble.h"
 
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/first_run/first_run.h"
 #include "chrome/browser/search_engines/util.h"
 #include "chrome/browser/ui/browser.h"
@@ skipping 26 matching lines @@
 // provides in alternative OEM bubble.
 const int kOEMBubblePadding = 4;
 
 // Padding between parts of strings on the same line (for instance,
 // "New!" and "Search from the address bar!"
 const int kStringSeparationPadding = 2;
 
 // Margin around close button.
 const int kMarginRightOfCloseButton = 7;
 
+// The bubble's HWND is actually owned by the border widget, and it's the border
+// widget that's owned by the frame window the bubble is anchored to. This
+// function makes the two leaps necessary to go from the bubble contents HWND
+// to the frame HWND.
+HWND GetLogicalBubbleOwner(HWND bubble_hwnd) {
+  HWND border_widget_hwnd = GetWindow(bubble_hwnd, GW_OWNER);
+  return GetWindow(border_widget_hwnd, GW_OWNER);
+}
+
 }  // namespace
 
 // Base class for implementations of the client view which appears inside the
 // first run bubble. It is a dialog-ish view, but is not a true dialog.
 class FirstRunBubbleViewBase : public views::View,
                                public views::ButtonListener,
                                public views::FocusChangeListener {
  public:
   // Called by FirstRunBubble::Show to request focus for the proper button
   // in the FirstRunBubbleView when it is shown.
@@ skipping 450 matching lines @@
   GetWidget()->GetFocusManager()->RemoveFocusChangeListener(view_);
 }
 
 void FirstRunBubble::EnableParent() {
   ::EnableWindow(GetParent(), true);
   // The EnableWindow() call above causes the parent to become active, which
   // resets the flag set by Bubble's call to DisableInactiveRendering(), so we
   // have to call it again before activating the bubble to prevent the parent
   // window from rendering inactive.
   // TODO(beng): this only works in custom-frame mode, not glass-frame mode.
+  HWND bubble_owner = GetLogicalBubbleOwner(GetNativeView());
   views::NativeWidget* parent =
-      views::NativeWidget::GetNativeWidgetForNativeView(GetParent());
+      views::NativeWidget::GetNativeWidgetForNativeView(bubble_owner);
   if (parent)
-    parent->GetWidget()->GetTopLevelWidget()->DisableInactiveRendering();
+    parent->GetWidget()->DisableInactiveRendering();
   // Reactivate the FirstRunBubble so it responds to OnActivate messages.
   SetWindowPos(GetParent(), 0, 0, 0, 0,
                SWP_NOSIZE | SWP_NOMOVE | SWP_NOREDRAW | SWP_SHOWWINDOW);
 }
 
 void FirstRunBubble::OnActivate(UINT action, BOOL minimized, HWND window) {
   // Keep the bubble around for kLingerTime milliseconds, to prevent accidental
   // closure.
   const int kLingerTime = 3000;
 
@@ skipping 15 matching lines @@
   // Keep window from automatically closing until kLingerTime has passed.
   if (::IsWindowEnabled(GetParent()))
     Bubble::OnActivate(action, minimized, window);
 }
 
 void FirstRunBubble::BubbleClosing(Bubble* bubble, bool closed_by_escape) {
   // Make sure our parent window is re-enabled.
   if (!IsWindowEnabled(GetParent()))
     ::EnableWindow(GetParent(), true);
 }