| Index: chrome/browser/ui/views/chrome_views_delegate.cc
|
| diff --git a/chrome/browser/ui/views/chrome_views_delegate.cc b/chrome/browser/ui/views/chrome_views_delegate.cc
|
| index a3798aa122b766beb6fb80e3cc5f2f8d08b81fd5..920006ff591a2b5615e6fca1a64dade72c5eda15 100644
|
| --- a/chrome/browser/ui/views/chrome_views_delegate.cc
|
| +++ b/chrome/browser/ui/views/chrome_views_delegate.cc
|
| @@ -261,7 +261,9 @@ void ChromeViewsDelegate::OnBeforeWidgetInit(
|
|
|
| #if defined(USE_AURA) && !defined(OS_CHROMEOS)
|
| bool use_non_toplevel_window =
|
| - params->parent && params->type != views::Widget::InitParams::TYPE_MENU;
|
| + params->parent &&
|
| + params->type != views::Widget::InitParams::TYPE_MENU &&
|
| + params->type != views::Widget::InitParams::TYPE_TOOLTIP;
|
|
|
| #if defined(OS_WIN)
|
| // On desktop Linux Chrome must run in an environment that supports a variety
|
| @@ -320,10 +322,7 @@ void ChromeViewsDelegate::OnBeforeWidgetInit(
|
| // While the majority of the time, context wasn't plumbed through due to the
|
| // existence of a global WindowTreeClient, if this window is a toplevel, it's
|
| // possible that there is no contextual state that we can use.
|
| - if (params->parent == NULL &&
|
| - params->context == NULL &&
|
| - !params->child &&
|
| - params->type != views::Widget::InitParams::TYPE_TOOLTIP) {
|
| + if (params->parent == NULL && params->context == NULL && !params->child) {
|
| // We need to make a decision about where to place this window based on the
|
| // desktop type.
|
| switch (chrome::GetActiveDesktop()) {
|
| @@ -351,7 +350,7 @@ void ChromeViewsDelegate::OnBeforeWidgetInit(
|
| parent_profile);
|
| }
|
| params->native_widget = native_widget;
|
| - } else if (params->type != views::Widget::InitParams::TYPE_TOOLTIP) {
|
| + } else {
|
| // TODO(erg): Once we've threaded context to everywhere that needs it, we
|
| // should remove this check here.
|
| gfx::NativeView to_check =
|
|
|