Commit a9c3c50d authored by pkotwicz@chromium.org's avatar pkotwicz@chromium.org

Remove unused SkipTestIfIceWM() and SkipTestIfCompizWM()

BUG=None
TEST=None

Review URL: https://codereview.chromium.org/366383002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@284449 0039d316-1c4b-4281-b951-d872f2087c98
parent f0e90cf9
......@@ -245,22 +245,6 @@ BasePanelBrowserTest::BasePanelBrowserTest()
BasePanelBrowserTest::~BasePanelBrowserTest() {
}
bool BasePanelBrowserTest::SkipTestIfIceWM() {
#if defined(OS_LINUX) && defined(USE_X11)
return ui::GuessWindowManager() == ui::WM_ICE_WM;
#else
return false;
#endif
}
bool BasePanelBrowserTest::SkipTestIfCompizWM() {
#if defined(OS_LINUX) && defined(USE_X11)
return ui::GuessWindowManager() == ui::WM_COMPIZ;
#else
return false;
#endif
}
void BasePanelBrowserTest::SetUpCommandLine(CommandLine* command_line) {
command_line->AppendSwitch(switches::kEnablePanels);
}
......
......@@ -42,19 +42,6 @@ class BasePanelBrowserTest : public InProcessBrowserTest {
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. [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(base::CommandLine* command_line) OVERRIDE;
virtual void SetUpOnMainThread() OVERRIDE;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment