diff options
| author | ben@chromium.org <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-01-16 22:41:42 +0000 |
|---|---|---|
| committer | ben@chromium.org <ben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-01-16 22:41:42 +0000 |
| commit | b6ad1cab92ed9e9c1734d1801482cfa8ffd3b4b1 (patch) | |
| tree | 0d1a8598536ed29830e4e8a4cdbcd90b5f0dbd87 /chrome/browser/views/frame | |
| parent | 16f3bd95e3f2bd39f6d64f91326bb3cdf894cedc (diff) | |
| download | chromium_src-b6ad1cab92ed9e9c1734d1801482cfa8ffd3b4b1.zip chromium_src-b6ad1cab92ed9e9c1734d1801482cfa8ffd3b4b1.tar.gz chromium_src-b6ad1cab92ed9e9c1734d1801482cfa8ffd3b4b1.tar.bz2 | |
Pull app modal dialog handling out of the BrowserList to remove the ChromeViews dependency. (Required for porting).
TEST=Open two browser windows. In one, enter "javascript:alert('Foo');" into the address bar and hit enter. Click on the other window. The modal dialog should be focused and the window it was opened over should flash in the taskbar,
Review URL: http://codereview.chromium.org/18179
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8234 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views/frame')
| -rw-r--r-- | chrome/browser/views/frame/browser_view.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/views/frame/browser_view.cc b/chrome/browser/views/frame/browser_view.cc index 9c28a59..9930cc1 100644 --- a/chrome/browser/views/frame/browser_view.cc +++ b/chrome/browser/views/frame/browser_view.cc @@ -250,7 +250,7 @@ void BrowserView::AddViewToDropList(views::View* view) { bool BrowserView::ActivateAppModalDialog() const { // If another browser is app modal, flash and activate the modal browser. - if (BrowserList::IsShowingAppModalDialog()) { + if (AppModalDialogQueue::HasActiveDialog()) { Browser* active_browser = BrowserList::GetLastActive(); if (active_browser && (browser_ != active_browser)) { active_browser->window()->FlashFrame(); |