From 7ad0dea94b930fdfaf3fb3de212c7321267156c8 Mon Sep 17 00:00:00 2001 From: "hbono@chromium.org" Date: Wed, 24 Mar 2010 05:47:17 +0000 Subject: A quick fix for Issue 38587. This issue is caused by my bonehead mistake that forgot setting the parent windows of AeroPeek windows. This change just sets the BrowserFrame object (which owns AeroPeek windows) to their parent window so we can minimize AeroPeek windows when we minimize the BrowserFrame. BUG=38597 TEST=Open two or more tabs, minimize Chrome, and verify there are not any garbage windows on Desktop. Review URL: http://codereview.chromium.org/1120011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@42433 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/aeropeek_manager.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'chrome') diff --git a/chrome/browser/aeropeek_manager.cc b/chrome/browser/aeropeek_manager.cc index 3830498..6385b4b 100644 --- a/chrome/browser/aeropeek_manager.cc +++ b/chrome/browser/aeropeek_manager.cc @@ -778,7 +778,7 @@ void AeroPeekWindow::Update(bool is_loading) { // thumbnail image of this window now. if (!hwnd()) { gfx::Rect bounds; - WindowImpl::Init(NULL, bounds); + WindowImpl::Init(frame_window_, bounds); return; } -- cgit v1.1