Commit 830c0312 authored by David Bertoni's avatar David Bertoni Committed by Commit Bot

[Extensions] Re-enable a test that's no longer broken on Mac.

The WindowOpenApiTest.WindowOpenSized was disabled on Mac but
it is no longer broken.

Note: This test was NOT disabled because it was flaky.

Bug: 225601
Change-Id: I6d6193f7fa745f6189887e849d596855a0fc5b91
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2252928Reviewed-by: default avatarArchana Simha <archanasimha@chromium.org>
Commit-Queue: David Bertoni <dbertoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#780515}
parent 67f29ffa
...@@ -221,15 +221,9 @@ IN_PROC_BROWSER_TEST_F(WindowOpenApiTest, WindowOpener) { ...@@ -221,15 +221,9 @@ IN_PROC_BROWSER_TEST_F(WindowOpenApiTest, WindowOpener) {
ASSERT_TRUE(RunExtensionTest("window_open/opener")) << message_; ASSERT_TRUE(RunExtensionTest("window_open/opener")) << message_;
} }
#if defined(OS_MACOSX)
// Extension popup windows are incorrectly sized on OSX, crbug.com/225601
#define MAYBE_WindowOpenSized DISABLED_WindowOpenSized
#else
#define MAYBE_WindowOpenSized WindowOpenSized
#endif
// Ensure that the width and height properties of a window opened with // Ensure that the width and height properties of a window opened with
// chrome.windows.create match the creation parameters. See crbug.com/173831. // chrome.windows.create match the creation parameters. See crbug.com/173831.
IN_PROC_BROWSER_TEST_F(WindowOpenApiTest, MAYBE_WindowOpenSized) { IN_PROC_BROWSER_TEST_F(WindowOpenApiTest, WindowOpenSized) {
ASSERT_TRUE(RunExtensionTest("window_open/window_size")) << message_; ASSERT_TRUE(RunExtensionTest("window_open/window_size")) << message_;
EXPECT_TRUE(WaitForTabsPopupsApps(browser(), 0, 0, 1)); EXPECT_TRUE(WaitForTabsPopupsApps(browser(), 0, 0, 1));
} }
......
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