Commit fdf30d4d authored by ananta@chromium.org's avatar ananta@chromium.org

Enable the DesktopAuraTopLevelOwnedPopupResizeTest test for Linux AURA.

Added implementation for the DesktopRootWindowHostX11::SetSize function.

BUG=330596
R=erg@chromium.org, sky@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@243740 0039d316-1c4b-4281-b951-d872f2087c98
parent ebced5fa
...@@ -356,8 +356,10 @@ bool DesktopWindowTreeHostX11::IsVisible() const { ...@@ -356,8 +356,10 @@ bool DesktopWindowTreeHostX11::IsVisible() const {
} }
void DesktopWindowTreeHostX11::SetSize(const gfx::Size& size) { void DesktopWindowTreeHostX11::SetSize(const gfx::Size& size) {
// TODO(erg): bool size_changed = bounds_.size() != size;
NOTIMPLEMENTED(); XResizeWindow(xdisplay_, xwindow_, size.width(), size.height());
if (size_changed)
NotifyHostResized(size);
} }
void DesktopWindowTreeHostX11::StackAtTop() { void DesktopWindowTreeHostX11::StackAtTop() {
......
...@@ -1218,10 +1218,6 @@ TEST_F(WidgetTest, DesktopAuraTopLevelOwnedPopupTest) { ...@@ -1218,10 +1218,6 @@ TEST_F(WidgetTest, DesktopAuraTopLevelOwnedPopupTest) {
RunPendingMessages(); RunPendingMessages();
} }
#if defined(OS_WIN)
// TODO(ananta)
// Fix this test to work on Linux Aura. Need to implement the
// views::DesktopWindowTreeHostX11::SetSize function
// This test validates that when a top level owned popup Aura window is // This test validates that when a top level owned popup Aura window is
// resized, the widget is resized as well. // resized, the widget is resized as well.
TEST_F(WidgetTest, DesktopAuraTopLevelOwnedPopupResizeTest) { TEST_F(WidgetTest, DesktopAuraTopLevelOwnedPopupResizeTest) {
...@@ -1239,7 +1235,6 @@ TEST_F(WidgetTest, DesktopAuraTopLevelOwnedPopupResizeTest) { ...@@ -1239,7 +1235,6 @@ TEST_F(WidgetTest, DesktopAuraTopLevelOwnedPopupResizeTest) {
ASSERT_NO_FATAL_FAILURE(popup_window.DestroyOwnedWindow()); ASSERT_NO_FATAL_FAILURE(popup_window.DestroyOwnedWindow());
RunPendingMessages(); RunPendingMessages();
} }
#endif
// Test to ensure that the aura Window's visiblity state is set to visible if // Test to ensure that the aura Window's visiblity state is set to visible if
// the underlying widget is hidden and then shown. // the underlying widget is hidden and then shown.
......
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