Reenable most panel interactive_ui_tests on Linux

chrome/browser/ui/panels/panel_browsertest.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 "base/bind.h"
 #include "base/prefs/pref_service.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/devtools/devtools_window.h"
@@ skipping 503 matching lines @@
   // dragged.
   gfx::Point mouse_location(panel->GetBounds().origin());
   panel_testing->PressLeftMouseButtonTitlebar(mouse_location);
   panel_testing->DragTitlebar(mouse_location + gfx::Vector2d(-100, 5));
   EXPECT_FALSE(panel_testing->IsAnimatingBounds());
   panel_testing->FinishDragTitlebar();
 
   // Set bounds with animation.
   gfx::Rect bounds = gfx::Rect(10, 20, 150, 160);
   panel->SetPanelBounds(bounds);
-  // There is no animation on Linux, by design.
-#if !defined(OS_LINUX)

[pkotwicz, 2014/07/07 14:55:46]

It will take some time before we can disable animations on Linux. They do look
bad. There are some bad interactions with Linux changing bounds asynchronously
and synthetic mouse events. Hopefully, I can fix this in the next couple of
weeks.

   EXPECT_TRUE(panel_testing->IsAnimatingBounds());
   WaitForBoundsAnimationFinished(panel);
-#endif
   EXPECT_FALSE(panel_testing->IsAnimatingBounds());
   EXPECT_EQ(bounds, panel->GetBounds());
 
   // Set bounds without animation.
   bounds = gfx::Rect(30, 40, 200, 220);
   panel->SetPanelBoundsInstantly(bounds);
   EXPECT_FALSE(panel_testing->IsAnimatingBounds());
   EXPECT_EQ(bounds, panel->GetBounds());
 
   panel->Close();
@@ skipping 1181 matching lines @@
 IN_PROC_BROWSER_TEST_F(PanelBrowserTest,
                        HideDockedPanelCreatedBeforeFullScreenMode) {
   // Create a docked panel.
   Panel* panel = CreatePanel("PanelTest");
   scoped_ptr<NativePanelTesting> panel_testing(CreateNativePanelTesting(panel));
 
   // Panel should be visible at first.
   EXPECT_TRUE(panel_testing->IsWindowVisible());
 
   // Panel should become hidden when entering full-screen mode.
-  // Note that this is not needed on Linux because the full-screen window will
-  // be always placed above any other windows.
   mock_display_settings_provider()->EnableFullScreenMode(true);
-#if defined(OS_LINUX)
-  EXPECT_TRUE(panel_testing->IsWindowVisible());
-#else
   EXPECT_FALSE(panel_testing->IsWindowVisible());
-#endif
 
   // Panel should become visible when leaving full-screen mode.
   mock_display_settings_provider()->EnableFullScreenMode(false);
   EXPECT_TRUE(panel_testing->IsWindowVisible());
 
   PanelManager::GetInstance()->CloseAll();
 }
 
 IN_PROC_BROWSER_TEST_F(PanelBrowserTest,
                        HideDockedPanelCreatedOnFullScreenMode) {
@@ skipping 34 matching lines @@
 #else
 #define MAYBE_FocusChangeEventOnMinimize FocusChangeEventOnMinimize
 #endif
 IN_PROC_BROWSER_TEST_F(PanelExtensionApiTest,
                        MAYBE_FocusChangeEventOnMinimize) {
   // This is needed so the subsequently created panels can be activated.
   // On a Mac, it transforms background-only test process into foreground one.
   ASSERT_TRUE(ui_test_utils::BringBrowserWindowToFront(browser()));
   ASSERT_TRUE(RunExtensionTest("panels/focus_change_on_minimize")) << message_;
 }