Bug 1319024 - Remove usage of browser.newtab.url from Firefox Puppeteer. draft
authorHenrik Skupin <mail@hskupin.info>
Mon, 21 Nov 2016 22:27:45 +0100
changeset 442529 1f1adde2544ee5aa135691e6612cf13fd3e9e170
parent 442499 1a3194836cb4c3da6ba3a9742a2d25cf26669b55
child 442530 9687ea0ff91c33b101842569aacef9fbdc422fad
push id36720
push userbmo:hskupin@gmail.com
push dateTue, 22 Nov 2016 17:26:19 +0000
bugs1319024
milestone53.0a1
Bug 1319024 - Remove usage of browser.newtab.url from Firefox Puppeteer. MozReview-Commit-ID: LwMJgQMgJoP
testing/puppeteer/firefox/firefox_puppeteer/mixins.py
--- a/testing/puppeteer/firefox/firefox_puppeteer/mixins.py
+++ b/testing/puppeteer/firefox/firefox_puppeteer/mixins.py
@@ -78,19 +78,21 @@ class PuppeteerMixin(object):
 
         self._start_handle_count = len(self.marionette.window_handles)
         self._init_tab_handles = self.marionette.window_handles
         self.marionette.set_context('chrome')
 
         self.puppeteer = Puppeteer(self.marionette)
         self.browser = self.puppeteer.windows.current
         self.browser.focus()
+
         with self.marionette.using_context(self.marionette.CONTEXT_CONTENT):
-            # Ensure that we have a default page opened
-            self.marionette.navigate(self.puppeteer.prefs.get_pref('browser.newtab.url'))
+            # Bug 1312674 - Navigating to about:blank twice can cause a hang in
+            # Marionette. So try to always have a known default page loaded.
+            self.marionette.navigate('about:')
 
     def tearDown(self, *args, **kwargs):
         self.marionette.set_context('chrome')
 
         try:
             # This code should be run after all other tearDown code
             # so that in case of a failure, further tests will not run
             # in a state that is more inconsistent than necessary.