Bug 1578783 - Cleanup fennec references in marionette tests; r=whimboo,ato

Differential Revision: https://phabricator.services.mozilla.com/D44753

--HG--
extra : moz-landing-system : lando

testing/marionette/harness/marionette_harness/__init__.py

@@ -17,7 +17,6 @@ from .marionette_test import (
     skip_if_chrome,
     skip_if_desktop,
     skip_if_e10s,
-    skip_if_mobile,
     SkipTest,
     skip_unless_protocol,
 )

testing/marionette/harness/marionette_harness/marionette_test/__init__.py

@@ -19,7 +19,6 @@ from .decorators import (
     skip_if_chrome,
     skip_if_desktop,
     skip_if_e10s,
-    skip_if_mobile,
     skip_unless_browser_pref,
     skip_unless_protocol,
     with_parameters,

testing/marionette/harness/marionette_harness/marionette_test/decorators.py

@@ -139,21 +139,6 @@ def skip_if_e10s(reason):
     return decorator
 
 
-def skip_if_mobile(reason):
-    """Decorator which skips a test if run on mobile."""
-    def decorator(test_item):
-        if not isinstance(test_item, types.FunctionType):
-            raise Exception('Decorator only supported for functions')
-
-        @functools.wraps(test_item)
-        def skip_wrapper(self, *args, **kwargs):
-            if self.marionette.session_capabilities.get('browserName') == 'fennec':
-                raise SkipTest(reason)
-            return test_item(self, *args, **kwargs)
-        return skip_wrapper
-    return decorator
-
-
 def skip_unless_browser_pref(reason, pref, predicate=bool):
     """Decorator which skips a test based on the value of a browser preference.
 

testing/marionette/harness/marionette_harness/tests/unit/test_click.py

@@ -12,7 +12,6 @@ from marionette_driver.marionette import Alert
 from marionette_harness import (
     MarionetteTestCase,
     run_if_e10s,
-    skip_if_mobile,
     WindowManagerMixin,
 )
 
@@ -453,7 +452,6 @@ class TestClickCloseContext(WindowManagerMixin, MarionetteTestCase):
         self.marionette.navigate(self.test_page)
         self.marionette.find_element(By.ID, "close-window").click()
 
-    @skip_if_mobile("Fennec doesn't support other chrome windows")
     def test_click_close_window(self):
         new_tab = self.open_window()
         self.marionette.switch_to_window(new_tab)

testing/marionette/harness/marionette_harness/tests/unit/test_execute_sandboxes.py

@@ -6,7 +6,7 @@ from __future__ import absolute_import
 
 from marionette_driver.errors import JavascriptException
 
-from marionette_harness import MarionetteTestCase, skip_if_mobile
+from marionette_harness import MarionetteTestCase
 
 
 class TestExecuteSandboxes(MarionetteTestCase):
@@ -41,7 +41,6 @@ class TestExecuteSandboxes(MarionetteTestCase):
             "return foo", sandbox="2", new_sandbox=False)
         self.assertEqual(foo, 2)
 
-    @skip_if_mobile("Intermittent on Android - bug 1526914")
     def test_execute_new_sandbox(self):
         # test that clearing a sandbox does not affect other sandboxes
         self.marionette.execute_script("foo = 1", sandbox="1")

testing/marionette/harness/marionette_harness/tests/unit/test_execute_script.py

@@ -7,7 +7,7 @@ from marionette_driver import By, errors
 from marionette_driver.marionette import Alert, HTMLElement
 from marionette_driver.wait import Wait
 
-from marionette_harness import MarionetteTestCase, skip_if_mobile, WindowManagerMixin
+from marionette_harness import MarionetteTestCase, WindowManagerMixin
 
 
 def inline(doc):
@@ -351,7 +351,6 @@ class TestExecuteContent(MarionetteTestCase):
     def test_comment_in_last_line(self):
         self.marionette.execute_script(" // comment ")
 
-    @skip_if_mobile("Modal dialogs not supported in Fennec")
     def test_return_value_on_alert(self):
         res = self.marionette._send_message("WebDriver:ExecuteScript", {"script": "alert()"})
         self.assertIn("value", res)
@@ -372,7 +371,6 @@ class TestExecuteChrome(WindowManagerMixin, TestExecuteContent):
         self.marionette.execute_script(
             "Components.classes['@mozilla.org/preferences-service;1']")
 
-    @skip_if_mobile("New windows not supported in Fennec")
     def test_unmarshal_element_collection(self):
         try:
             win = self.open_chrome_window("chrome://marionette/content/test.xul")

testing/marionette/harness/marionette_harness/tests/unit/test_get_current_url_chrome.py

@@ -6,7 +6,7 @@ from __future__ import absolute_import
 
 from marionette_driver.errors import NoSuchWindowException
 
-from marionette_harness import MarionetteTestCase, WindowManagerMixin, skip_if_mobile
+from marionette_harness import MarionetteTestCase, WindowManagerMixin
 
 
 class TestGetCurrentUrlChrome(WindowManagerMixin, MarionetteTestCase):
@@ -29,7 +29,6 @@ class TestGetCurrentUrlChrome(WindowManagerMixin, MarionetteTestCase):
         chrome_url = self.marionette.execute_script("return window.location.href;")
         self.assertEqual(self.marionette.get_url(), chrome_url)
 
-    @skip_if_mobile("Fennec doesn't support other chrome windows")
     def test_no_browser_window(self):
         win = self.open_chrome_window("chrome://marionette/content/test.xul")
         self.marionette.switch_to_window(win)

testing/marionette/harness/marionette_harness/tests/unit/test_marionette.py

@@ -9,7 +9,7 @@ import time
 
 from marionette_driver import errors
 from marionette_driver.marionette import Marionette
-from marionette_harness import MarionetteTestCase, run_if_manage_instance, skip_if_mobile
+from marionette_harness import MarionetteTestCase, run_if_manage_instance
 
 
 class TestMarionette(MarionetteTestCase):
@@ -25,7 +25,6 @@ class TestMarionette(MarionetteTestCase):
         self.assertIn(expected_test_name, self.marionette.test_name)
 
     @run_if_manage_instance("Only runnable if Marionette manages the instance")
-    @skip_if_mobile("Bug 1322993 - Missing temporary folder")
     def test_raise_for_port_non_existing_process(self):
         """Test that raise_for_port doesn't run into a timeout if instance is not running."""
         self.marionette.quit()
@@ -61,7 +60,6 @@ class TestMarionette(MarionetteTestCase):
         self.assertEqual(current_socket_timeout,
                          self.marionette.client._sock.gettimeout())
 
-    @skip_if_mobile("No application update service available on Android")
     def test_application_update_disabled(self):
         # Updates of the application should always be disabled by default
         with self.marionette.using_context("chrome"):

testing/marionette/harness/marionette_harness/tests/unit/test_mouse_action.py

@@ -9,7 +9,7 @@ import urllib
 from marionette_driver import By, errors, Wait
 from marionette_driver.keys import Keys
 
-from marionette_harness import MarionetteTestCase, skip_if_mobile
+from marionette_harness import MarionetteTestCase
 
 
 def inline(doc):
@@ -78,7 +78,6 @@ class TestPointerActions(BaseMouseAction):
         event_count = self.marionette.execute_script("return window.eventCount", sandbox=None)
         self.assertEqual(event_count, 2)
 
-    @skip_if_mobile("There is no context menu available on mobile")
     def test_context_click_action(self):
         test_html = self.marionette.absolute_url("clicks.html")
         self.marionette.navigate(test_html)
@@ -151,7 +150,6 @@ class TestNonSpecCompliantPointerOrigin(BaseMouseAction):
         self.assertAlmostEqual(click_position["x"], elem_center_point["x"], delta=1)
         self.assertAlmostEqual(click_position["y"], elem_center_point["y"], delta=1)
 
-    @skip_if_mobile("Bug 1534291 - Missing MoveTargetOutOfBoundsException")
     def test_click_element_larger_than_viewport_with_center_point_outside(self):
         self.marionette.navigate(inline("""
           <div id="div" style="width: 300vw; height: 300vh; background: green;"

testing/marionette/harness/marionette_harness/tests/unit/test_navigation.py

@@ -15,7 +15,6 @@ from marionette_harness import (
     MarionetteTestCase,
     run_if_e10s,
     run_if_manage_instance,
-    skip_if_mobile,
     WindowManagerMixin,
 )
 
@@ -258,14 +257,12 @@ class TestNavigate(BaseNavigationTestCase):
         self.marionette.go_back()
         self.assertEqual(self.marionette.get_url(), self.test_page_push_state)
 
-    @skip_if_mobile("Test file is only located on host machine")
     def test_navigate_file_url(self):
         self.marionette.navigate(self.test_page_file_url)
         self.marionette.find_element(By.ID, "file-url")
         self.marionette.navigate(self.test_page_remote)
 
     @run_if_e10s("Requires e10s mode enabled")
-    @skip_if_mobile("Test file is only located on host machine")
     def test_navigate_file_url_remoteness_change(self):
         self.marionette.navigate("about:robots")
         self.assertFalse(self.is_remote_tab)
@@ -283,7 +280,6 @@ class TestNavigate(BaseNavigationTestCase):
         self.marionette.navigate("about:blank")
 
     @run_if_manage_instance("Only runnable if Marionette manages the instance")
-    @skip_if_mobile("Bug 1322993 - Missing temporary folder")
     def test_focus_after_navigation(self):
         self.marionette.restart()
 
@@ -299,7 +295,6 @@ class TestNavigate(BaseNavigationTestCase):
         self.marionette.switch_to_window(self.new_tab)
         self.marionette.navigate(self.test_page_remote)
 
-    @skip_if_mobile("Interacting with chrome elements not available for Fennec")
     def test_type_to_non_remote_tab(self):
         self.marionette.navigate(self.test_page_not_remote)
         self.assertFalse(self.is_remote_tab)
@@ -315,7 +310,6 @@ class TestNavigate(BaseNavigationTestCase):
             message="'about:support' hasn't been loaded")
         self.assertFalse(self.is_remote_tab)
 
-    @skip_if_mobile("Interacting with chrome elements not available for Fennec")
     @run_if_e10s("Requires e10s mode enabled")
     def test_type_to_remote_tab(self):
         self.assertTrue(self.is_remote_tab)
@@ -431,7 +425,6 @@ class TestBackForwardNavigation(BaseNavigationTestCase):
         ]
         self.run_bfcache_test(test_pages)
 
-    @skip_if_mobile("Test file is only located on host machine")
     def test_file_url(self):
         test_pages = [
             {"url": self.test_page_remote},
@@ -521,7 +514,6 @@ class TestBackForwardNavigation(BaseNavigationTestCase):
         ]
         self.run_bfcache_test(test_pages)
 
-    @skip_if_mobile("Bug 1333209 - Process killed because of connection loss")
     def test_non_remote_about_pages(self):
         test_pages = [
             {"url": "about:preferences", "is_remote": False},
@@ -620,7 +612,6 @@ class TestRefresh(BaseNavigationTestCase):
         self.marionette.refresh()
         self.marionette.find_element(By.NAME, "third")
 
-    @skip_if_mobile("Test file is only located on host machine")
     def test_file_url(self):
         self.marionette.navigate(self.test_page_file_url)
         self.assertEqual(self.test_page_file_url, self.marionette.get_url())

testing/marionette/harness/marionette_harness/tests/unit/test_prefs.py

@@ -10,8 +10,6 @@ from marionette_driver.errors import JavascriptException
 from marionette_harness import (
     MarionetteTestCase,
     run_if_manage_instance,
-    skip_if_desktop,
-    skip_if_mobile
 )
 
 
@@ -43,7 +41,6 @@ class TestPreferences(MarionetteTestCase):
                 self.assertEqual(self.marionette.get_pref(key), value,
                                  "Preference {} hasn't been set to {}".format(key, repr(value)))
 
-    @skip_if_mobile("Only runnable with Firefox")
     def test_desktop_instance_preferences(self):
         required_prefs = geckoinstance.DesktopInstance.desktop_prefs
 
@@ -54,17 +51,6 @@ class TestPreferences(MarionetteTestCase):
             self.assertEqual(self.marionette.get_pref(key), value,
                              "Preference {} hasn't been set to {}".format(key, value))
 
-    @skip_if_desktop("Only runnable with Fennec")
-    def test_fennec_instance_preferences(self):
-        required_prefs = geckoinstance.FennecInstance.required_prefs
-
-        for key, value in required_prefs.iteritems():
-            if key in ["browser.tabs.remote.autostart"]:
-                return
-
-            self.assertEqual(self.marionette.get_pref(key), value,
-                             "Preference {} hasn't been set to {}".format(key, value))
-
     def test_clear_pref(self):
         self.assertIsNone(self.marionette.get_pref(self.prefs["bool"]))
 

testing/marionette/harness/marionette_harness/tests/unit/test_screen_orientation.py

@@ -10,7 +10,6 @@ from marionette_harness import (
     MarionetteTestCase,
     parameterized,
     skip_if_desktop,
-    skip_if_mobile,
 )
 
 
@@ -22,12 +21,8 @@ class TestScreenOrientation(MarionetteTestCase):
 
     def setUp(self):
         MarionetteTestCase.setUp(self)
-        self.is_mobile = self.marionette.session_capabilities.get("rotatable", False)
 
     def tearDown(self):
-        if self.is_mobile:
-            self.marionette.set_orientation(default_orientation)
-            self.wait_for_orientation(default_orientation)
         MarionetteTestCase.tearDown(self)
 
     def wait_for_orientation(self, orientation, timeout=None):
@@ -75,7 +70,6 @@ class TestScreenOrientation(MarionetteTestCase):
                                      unknown_orientation.format("null")):
             self.marionette.set_orientation(None)
 
-    @skip_if_mobile("Specific test for Firefox")
     def test_unsupported_operation_on_desktop(self):
         with self.assertRaises(errors.UnsupportedOperationException):
             self.marionette.set_orientation("landscape-primary")

testing/marionette/harness/marionette_harness/tests/unit/test_screenshot.py

@@ -18,7 +18,6 @@ from marionette_driver.errors import NoSuchWindowException
 from marionette_harness import (
     MarionetteTestCase,
     skip,
-    skip_if_mobile,
     WindowManagerMixin,
 )
 
@@ -146,7 +145,6 @@ class TestScreenCaptureChrome(WindowManagerMixin, ScreenCaptureTestCase):
         screenshot_chrome = self.marionette.screenshot()
         self.assertNotEqual(screenshot_content, screenshot_chrome)
 
-    @skip_if_mobile("Fennec doesn't support other chrome windows")
     def test_capture_element(self):
         dialog = self.open_dialog()
         self.marionette.switch_to_window(dialog)
@@ -164,7 +162,6 @@ class TestScreenCaptureChrome(WindowManagerMixin, ScreenCaptureTestCase):
         self.marionette.close_chrome_window()
         self.marionette.switch_to_window(self.start_window)
 
-    @skip_if_mobile("Fennec doesn't support other chrome windows")
     def test_capture_full_area(self):
         dialog = self.open_dialog()
         self.marionette.switch_to_window(dialog)
@@ -185,7 +182,6 @@ class TestScreenCaptureChrome(WindowManagerMixin, ScreenCaptureTestCase):
         self.assertEqual(root_dimensions, self.get_image_dimensions(screenshot_full))
         self.assertEqual(screenshot_root, screenshot_full)
 
-    @skip_if_mobile("Fennec doesn't support other chrome windows")
     def test_capture_window_already_closed(self):
         dialog = self.open_dialog()
         self.marionette.switch_to_window(dialog)
@@ -194,7 +190,6 @@ class TestScreenCaptureChrome(WindowManagerMixin, ScreenCaptureTestCase):
         self.assertRaises(NoSuchWindowException, self.marionette.screenshot)
         self.marionette.switch_to_window(self.start_window)
 
-    @skip_if_mobile("Fennec doesn't support other chrome windows")
     def test_formats(self):
         dialog = self.open_dialog()
         self.marionette.switch_to_window(dialog)
@@ -233,7 +228,6 @@ class TestScreenCaptureContent(WindowManagerMixin, ScreenCaptureTestCase):
         self.assertRaises(NoSuchWindowException, self.marionette.screenshot)
         self.marionette.switch_to_window(self.start_tab)
 
-    @skip_if_mobile("Bug 1487124 - Android need its own maximum allowed dimensions")
     def test_capture_vertical_bounds(self):
         self.marionette.navigate(inline("<body style='margin-top: 32768px'>foo"))
         screenshot = self.marionette.screenshot()

testing/marionette/harness/marionette_harness/tests/unit/test_switch_window_content.py

@@ -9,7 +9,6 @@ from marionette_driver.keys import Keys
 
 from marionette_harness import (
     MarionetteTestCase,
-    skip_if_mobile,
     WindowManagerMixin,
 )
 
@@ -89,7 +88,6 @@ class TestSwitchToWindowContent(WindowManagerMixin, MarionetteTestCase):
         self.assertEqual(self.marionette.current_window_handle, self.start_tab)
         self.assertEqual(self.get_selected_tab_index(), self.selected_tab_index)
 
-    @skip_if_mobile("Fennec doesn't support other chrome windows")
     def test_switch_tabs_in_different_windows_with_focus_change(self):
         new_tab1 = self.open_tab(focus=True)
         self.assertEqual(self.marionette.current_window_handle, self.start_tab)

testing/marionette/harness/marionette_harness/tests/unit/test_timeouts.py

@@ -12,7 +12,7 @@ from marionette_driver.errors import (
 )
 from marionette_driver.marionette import HTMLElement
 
-from marionette_harness import MarionetteTestCase, run_if_manage_instance, skip_if_mobile
+from marionette_harness import MarionetteTestCase, run_if_manage_instance
 
 
 class TestTimeouts(MarionetteTestCase):
@@ -65,7 +65,6 @@ class TestTimeouts(MarionetteTestCase):
         self.assertRaises(NoSuchElementException, self.marionette.find_element, By.ID, "newDiv")
 
     @run_if_manage_instance("Only runnable if Marionette manages the instance")
-    @skip_if_mobile("Bug 1322993 - Missing temporary folder")
     def test_reset_timeout(self):
         timeouts = [getattr(self.marionette.timeout, f) for f in (
             'implicit', 'page_load', 'script',)]

testing/marionette/harness/marionette_harness/tests/unit/test_typing.py

@@ -10,7 +10,7 @@ from marionette_driver.by import By
 from marionette_driver.errors import ElementNotInteractableException
 from marionette_driver.keys import Keys
 
-from marionette_harness import MarionetteTestCase, skip, skip_if_mobile
+from marionette_harness import MarionetteTestCase, skip
 
 
 def inline(doc):
@@ -34,7 +34,6 @@ class TestTypingChrome(TypingTestCase):
         super(TestTypingChrome, self).setUp()
         self.marionette.set_context("chrome")
 
-    @skip_if_mobile("Interacting with chrome elements not available for Fennec")
     def test_cut_and_paste_shortcuts(self):
         with self.marionette.using_context("content"):
             test_html = self.marionette.absolute_url("keyboard.html")
@@ -215,7 +214,6 @@ class TestTypingContent(TypingTestCase):
         #  filled, we're a letter short here
         self.assertEqual(result.text, "I like chees")
 
-    @skip_if_mobile("Bug 1333069 - Assertion: 'down: 40' not found in u''")
     def test_should_report_key_code_of_arrow_keys_up_down_events(self):
         test_html = self.marionette.absolute_url("keyboard.html")
         self.marionette.navigate(test_html)

testing/marionette/harness/marionette_harness/tests/unit/test_window_close_content.py

@@ -7,7 +7,7 @@ from __future__ import absolute_import
 import urllib
 
 from marionette_driver.by import By
-from marionette_harness import MarionetteTestCase, skip_if_mobile, WindowManagerMixin
+from marionette_harness import MarionetteTestCase, WindowManagerMixin
 
 
 def inline(doc):
@@ -22,7 +22,6 @@ class TestCloseWindow(WindowManagerMixin, MarionetteTestCase):
 
         super(TestCloseWindow, self).tearDown()
 
-    @skip_if_mobile("Interacting with chrome windows not available for Fennec")
     def test_close_chrome_window_for_browser_window(self):
         with self.marionette.using_context("chrome"):
             new_window = self.open_window()
@@ -34,7 +33,6 @@ class TestCloseWindow(WindowManagerMixin, MarionetteTestCase):
         self.assertListEqual(self.start_windows, chrome_window_handles)
         self.assertNotIn(new_window, self.marionette.window_handles)
 
-    @skip_if_mobile("Interacting with chrome windows not available for Fennec")
     def test_close_chrome_window_for_non_browser_window(self):
         new_window = self.open_chrome_window("chrome://marionette/content/test.xul")
         self.marionette.switch_to_window(new_window)
@@ -46,7 +44,6 @@ class TestCloseWindow(WindowManagerMixin, MarionetteTestCase):
         self.assertListEqual(self.start_windows, chrome_window_handles)
         self.assertNotIn(new_window, self.marionette.window_handles)
 
-    @skip_if_mobile("Interacting with chrome windows not available for Fennec")
     def test_close_chrome_window_for_last_open_window(self):
         self.close_all_windows()
 
@@ -79,7 +76,6 @@ class TestCloseWindow(WindowManagerMixin, MarionetteTestCase):
         self.marionette.find_element(By.TAG_NAME, "input").send_keys("foo")
         self.marionette.close()
 
-    @skip_if_mobile("Interacting with chrome windows not available for Fennec")
     def test_close_window_for_browser_window_with_single_tab(self):
         new_tab = self.open_window()
         self.marionette.switch_to_window(new_tab)
@@ -98,7 +94,6 @@ class TestCloseWindow(WindowManagerMixin, MarionetteTestCase):
         self.assertListEqual([self.start_window], self.marionette.chrome_window_handles)
         self.assertIsNotNone(self.marionette.session)
 
-    @skip_if_mobile("discardBrowser is only available in Firefox")
     def test_close_browserless_tab(self):
         self.close_all_tabs()
 

testing/marionette/harness/marionette_harness/tests/unit/test_window_handles_content.py

@@ -9,7 +9,7 @@ import urllib
 
 from marionette_driver import errors
 
-from marionette_harness import MarionetteTestCase, skip_if_mobile, WindowManagerMixin
+from marionette_harness import MarionetteTestCase, WindowManagerMixin
 
 
 def inline(doc):
@@ -78,7 +78,6 @@ class TestWindowHandles(WindowManagerMixin, MarionetteTestCase):
         self.assert_window_handles()
         self.assertEqual(self.marionette.current_window_handle, self.start_tab)
 
-    @skip_if_mobile("Fennec doesn't support other chrome windows")
     def tst_window_handles_after_opening_new_non_browser_window(self):
         new_window = self.open_chrome_window(self.xul_dialog)
         self.assert_window_handles()

testing/marionette/harness/marionette_harness/tests/unit/test_window_management.py

@@ -7,7 +7,7 @@ from __future__ import absolute_import
 from marionette_driver import By
 from marionette_driver.errors import NoSuchWindowException
 
-from marionette_harness import MarionetteTestCase, WindowManagerMixin, skip_if_mobile
+from marionette_harness import MarionetteTestCase, WindowManagerMixin
 
 
 class TestNoSuchWindowContent(WindowManagerMixin, MarionetteTestCase):
@@ -19,7 +19,6 @@ class TestNoSuchWindowContent(WindowManagerMixin, MarionetteTestCase):
         self.close_all_tabs()
         super(TestNoSuchWindowContent, self).tearDown()
 
-    @skip_if_mobile("Fennec doesn't support other chrome windows")
     def test_closed_chrome_window(self):
         with self.marionette.using_context("chrome"):
             new_window = self.open_window()
@@ -39,7 +38,6 @@ class TestNoSuchWindowContent(WindowManagerMixin, MarionetteTestCase):
         with self.assertRaises(NoSuchWindowException):
             self.marionette.switch_to_window(new_window)
 
-    @skip_if_mobile("Fennec doesn't support other chrome windows")
     def test_closed_chrome_window_while_in_frame(self):
         new_window = self.open_chrome_window("chrome://marionette/content/test.xul")
         self.marionette.switch_to_window(new_window)

testing/marionette/harness/marionette_harness/tests/unit/test_window_status_content.py

@@ -7,7 +7,7 @@ from __future__ import absolute_import
 from marionette_driver import By
 from marionette_driver.errors import NoSuchWindowException
 
-from marionette_harness import MarionetteTestCase, WindowManagerMixin, skip_if_mobile
+from marionette_harness import MarionetteTestCase, WindowManagerMixin
 
 
 class TestNoSuchWindowContent(WindowManagerMixin, MarionetteTestCase):
@@ -19,7 +19,6 @@ class TestNoSuchWindowContent(WindowManagerMixin, MarionetteTestCase):
         self.close_all_windows()
         super(TestNoSuchWindowContent, self).tearDown()
 
-    @skip_if_mobile("Fennec doesn't support other chrome windows")
     def test_closed_chrome_window(self):
         with self.marionette.using_context("chrome"):
             new_window = self.open_window()
@@ -39,7 +38,6 @@ class TestNoSuchWindowContent(WindowManagerMixin, MarionetteTestCase):
         with self.assertRaises(NoSuchWindowException):
             self.marionette.switch_to_window(new_window)
 
-    @skip_if_mobile("Fennec doesn't support other chrome windows")
     def test_closed_chrome_window_while_in_frame(self):
         new_window = self.open_chrome_window("chrome://marionette/content/test.xul")
         self.marionette.switch_to_window(new_window)

testing/marionette/harness/marionette_harness/tests/unit/unit-tests.ini

@@ -1,7 +1,7 @@
 [test_marionette.py]
 [test_transport.py]
 [test_cli_arguments.py]
-skip-if = manage_instance == false || appname == 'fennec' # Bug 1298921
+skip-if = manage_instance == false
 [test_geckoinstance.py]
 [test_data_driven.py]
 [test_session.py]
@@ -12,20 +12,15 @@ skip-if = manage_instance == false || appname == 'fennec' # Bug 1298921
 expected = fail
 [test_click.py]
 [test_click_chrome.py]
-skip-if = appname == 'fennec'
 [test_checkbox.py]
 [test_checkbox_chrome.py]
-skip-if = appname == 'fennec'
 [test_elementsize.py]
 [test_elementsize_chrome.py]
-skip-if = appname == 'fennec'
 [test_position.py]
 [test_rendered_element.py]
 [test_chrome_element_css.py]
-skip-if = appname == 'fennec'
 [test_element_state.py]
 [test_element_state_chrome.py]
-skip-if = appname == 'fennec'
 [test_text.py]
 
 [test_typing.py]
@@ -34,36 +29,27 @@ skip-if = appname == 'fennec'
 [test_execute_script.py]
 [test_element_retrieval.py]
 [test_findelement_chrome.py]
-skip-if = appname == 'fennec'
 
 [test_get_current_url_chrome.py]
 [test_navigation.py]
 [test_timeouts.py]
 
 [test_anonymous_content.py]
-skip-if = appname == 'fennec'
 [test_switch_frame.py]
 [test_switch_frame_chrome.py]
-skip-if = appname == 'fennec'
 [test_switch_window_chrome.py]
-skip-if = appname == 'fennec'
 [test_switch_window_content.py]
 
 [test_pagesource.py]
 [test_pagesource_chrome.py]
-skip-if = appname == 'fennec'
 
 [test_visibility.py]
 [test_window_handles_chrome.py]
-skip-if = appname == 'fennec'
 [test_window_handles_content.py]
 [test_window_close_chrome.py]
-skip-if = appname == 'fennec'
 [test_window_close_content.py]
 [test_window_rect.py]
-skip-if = appname == 'fennec'
 [test_window_maximize.py]
-skip-if = appname == 'fennec'
 [test_window_status_content.py]
 [test_window_status_chrome.py]
 
@@ -71,9 +57,7 @@ skip-if = appname == 'fennec'
 [test_cookies.py]
 [test_title.py]
 [test_title_chrome.py]
-skip-if = appname == 'fennec'
 [test_window_type_chrome.py]
-skip-if = appname == 'fennec'
 [test_implicit_waits.py]
 [test_wait.py]
 [test_expected.py]
@@ -86,40 +70,35 @@ skip-if = true # Bug 925688
 [test_execute_isolate.py]
 [test_click_scrolling.py]
 [test_profile_management.py]
-skip-if = manage_instance == false || appname == 'fennec' || (debug && ((os == 'mac') || (os == 'linux'))) # Bug 1298921, Bug 1450355
+skip-if = manage_instance == false || (debug && ((os == 'mac') || (os == 'linux'))) # Bug 1450355
 [test_quit_restart.py]
-skip-if = manage_instance == false || appname == 'fennec' # Bug 1298921
+skip-if = manage_instance == false
 [test_context.py]
 
 [test_modal_dialogs.py]
-skip-if = appname == 'fennec' # Bug 1325738
 [test_unhandled_prompt_behavior.py]
-skip-if = appname == 'fennec' # Bug 1325738
 
 [test_key_actions.py]
 [test_mouse_action.py]
 
 [test_teardown_context_preserved.py]
 [test_file_upload.py]
-skip-if = appname == 'fennec' || os == "win" # http://bugs.python.org/issue14574
+skip-if = os == "win" # http://bugs.python.org/issue14574
 
 [test_execute_sandboxes.py]
 [test_prefs.py]
 [test_prefs_enforce.py]
-skip-if = manage_instance == false || appname == 'fennec' # Bug 1298921
+skip-if = manage_instance == false
 
 [test_shadow_dom.py]
 
 [test_chrome.py]
-skip-if = appname == 'fennec'
 
 [test_addons.py]
-skip-if = appname == 'fennec' # Bug 1330598
 
 [test_select.py]
 [test_crash.py]
-skip-if = asan || manage_instance == false || appname == 'fennec' # Bug 1298921
+skip-if = asan || manage_instance == false
 [test_localization.py]
 
 [test_reftest.py]
-skip-if = appname == 'fennec' # Bug 1519552