Fix flaky linux panel tests related to active state checks

chrome/browser/ui/panels/base_panel_browser_test.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_PANELS_BASE_PANEL_BROWSER_TEST_H_
 #define CHROME_BROWSER_UI_PANELS_BASE_PANEL_BROWSER_TEST_H_
 #pragma once
 
 #include "base/values.h"
 #include "base/memory/ref_counted.h"
@@ skipping 23 matching lines @@
     virtual void SetDesktopBarThickness(DesktopBarAlignment alignment,
                                         int thickness) = 0;
   };
 
   BasePanelBrowserTest();
   virtual ~BasePanelBrowserTest();
 
   // Linux bots use icewm which activate windows in ways that break
   // certain panel tests. Skip those tests when running on the bots.
   // We do not disable the tests to make it easy for developers to run
-  // them locally.
+  // them locally. [Icewm always activates a window when shown.]
   bool SkipTestIfIceWM();
 
   // Gnome running compiz refuses to activate a window that was initially
   // created as inactive, causing certain panel tests to fail. These tests
   // pass fine on the bots, but fail for developers as Gnome running compiz
   // is the typical linux dev machine configuration. We do not disable the
   // tests to ensure we still have coverage on the bots.
   bool SkipTestIfCompizWM();
 
   virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE;
@@ skipping 67 matching lines @@
   static const FilePath::CharType* kTestDir;
 
  private:
   // Passed to and owned by PanelManager.
   MockDisplaySettingsProvider* mock_display_settings_provider_;
 
   bool mock_display_settings_enabled_;
 };
 
 #endif  // CHROME_BROWSER_UI_PANELS_BASE_PANEL_BROWSER_TEST_H_