Minimized panels should not open on Windows when activated by the OS

chrome/browser/ui/panels/panel_browser_window_gtk.cc

 // Copyright (c) 2012 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/panels/panel_browser_window_gtk.h"
 
 #include "base/bind.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/gtk/browser_titlebar.h"
 #include "chrome/browser/ui/panels/panel.h"
@@ skipping 79 matching lines @@
                    G_CALLBACK(OnTitlebarButtonReleaseEventThunk), this);
 
   g_signal_connect(window_, "enter-notify-event",
                    G_CALLBACK(OnEnterNotifyThunk), this);
   g_signal_connect(window_, "leave-notify-event",
                    G_CALLBACK(OnLeaveNotifyThunk), this);
 
   ui::WorkAreaWatcherX::AddObserver(this);
   registrar_.Add(
       this,
-      chrome::NOTIFICATION_PANEL_CHANGED_EXPANSION_STATE,
-      content::Source<Panel>(panel_.get()));
-  registrar_.Add(
-      this,
       chrome::NOTIFICATION_PANEL_CHANGED_LAYOUT_MODE,
       content::Source<Panel>(panel_.get()));
   registrar_.Add(
       this,
       chrome::NOTIFICATION_WINDOW_CLOSED,
       content::Source<GtkWindow>(window()));
 }
 
 bool PanelBrowserWindowGtk::GetWindowEdge(int x, int y, GdkWindowEdge* edge) {
   // Since panels are not resizable or movable by the user, we should not
@@ skipping 154 matching lines @@
   panel_->manager()->OnDisplayChanged();
 }
 
 void PanelBrowserWindowGtk::Observe(
     int type,
     const content::NotificationSource& source,
     const content::NotificationDetails& details) {
   switch (type) {
     case chrome::NOTIFICATION_PANEL_CHANGED_LAYOUT_MODE:
       titlebar()->UpdateCustomFrame(true);
-      // No break. Accept focus code should execute for both cases.
-    case chrome::NOTIFICATION_PANEL_CHANGED_EXPANSION_STATE:
-      gtk_window_set_accept_focus(window(), !panel_->IsMinimized());
       break;
     case chrome::NOTIFICATION_WINDOW_CLOSED:
       // Cleanup.
       if (bounds_animator_.get())
         bounds_animator_.reset();
 
       CleanupDragDrop();
       if (drag_widget_) {
         // Terminate the grab if we have it. We could do this using any widget,
         // |drag_widget_| is just convenient.
@@ skipping 89 matching lines @@
     browser_window->Activate();
   } else {
     Deactivate();
   }
 }
 
 bool PanelBrowserWindowGtk::IsPanelActive() const {
   return IsActive();
 }
 
+void PanelBrowserWindowGtk::PreventActivationByOS(bool prevent_activation) {
+  gtk_window_set_accept_focus(window(), !prevent_activation);
+  return;
+}
+
 gfx::NativeWindow PanelBrowserWindowGtk::GetNativePanelHandle() {
   return GetNativeHandle();
 }
 
 void PanelBrowserWindowGtk::UpdatePanelTitleBar() {
   UpdateTitleBar();
 }
 
 void PanelBrowserWindowGtk::UpdatePanelLoadingAnimations(bool should_animate) {
   UpdateLoadingAnimations(should_animate);
@@ skipping 477 matching lines @@
     MessageLoopForUI::current()->RunAllPending();
 }
 
 bool NativePanelTestingGtk::IsWindowSizeKnown() const {
   return panel_browser_window_gtk_->window_size_known_;
 }
 
 bool NativePanelTestingGtk::IsAnimatingBounds() const {
   return panel_browser_window_gtk_->IsAnimatingBounds();
 }