1325047 - Test test_ssl_status_after_restart.py TestSSLStatusAfterRestart.test_ssl_status_after_restart fails in e10s mode

Status: RESOLVED DUPLICATE of bug 1573641

(Testing :: Firefox UI Tests, defect, P3)

Description

[Henrik Skupin [:whimboo][⌚️UTC+1]]

This test is marked as skipped for e10s, but there is no bug filed about this issue yet. Doing that now so we have a reference.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Lets wait on bug 1433873 which has a known race during in-app restarts.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+1]]

I tried to run this test locally and the problem is here:

> 1539690620194	Marionette	TRACE	1 -> [0,191,"WebDriver:ExecuteScript",{"scriptTimeout":null,"newSandbox":true,"args":[{"element-6066-11e4-a52e-4f735466cecf":"f44e ... n) {\n            return null;\n          }\n          return win.outerWindowID.toString();","sandbox":"default","line":211}]
> 1539690620200	Marionette	TRACE	1 <- [1,191,null,{"value":"6442450945"}]
> 1539690620209	Marionette	TRACE	1 -> [0,192,"WebDriver:ExecuteScript",{"scriptTimeout":null,"newSandbox":true,"args":[{"element-6066-11e4-a52e-4f735466cecf":"6447 ... n) {\n            return null;\n          }\n          return win.outerWindowID.toString();","sandbox":"default","line":211}]
> 1539690620228	Marionette	TRACE	1 <- [1,192,{"error":"javascript error","message":"TypeError: win.outerWindowID is undefined; can't access its \"toString\" proper ... et@chrome://marionette/content/server.js:236:8\n_onJSONObjectReady/<@chrome://marionette/content/transport.js:490:9\n"},null]

We run this `WebDriver:ExecuteScript` command several times, but then once the outerWindowID is undefined. And this is happening in `testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/browser/tabbar.py:216:11`. As such enabling this test would depend on bug 1351940.

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+1]]

When I'm running the test now with bug 1351940 fixed, I see the following error:

>  0:28.31 TEST_END: ERROR, expected PASS - TimeoutException: Timed out after 5.1 seconds with message: Identity popup has not been opened.
> Traceback (most recent call last):
>   File "/Volumes/data/code/gecko/testing/marionette/harness/marionette_harness/marionette_test/testcases.py", line 159, in run
>     testMethod()
>   File "/Volumes/data/code/gecko/testing/firefox-ui/tests/functional/security/test_ssl_status_after_restart.py", line 63, in test_ssl_status_after_restart
>     self.verify_certificate_status(item)
>   File "/Volumes/data/code/gecko/testing/firefox-ui/tests/functional/security/test_ssl_status_after_restart.py", line 75, in verify_certificate_status
>     self.locationbar.open_identity_popup()
>   File "/Volumes/data/code/gecko/testing/marionette/puppeteer/firefox/firefox_puppeteer/ui/browser/toolbars.py", line 245, in open_identity_popup
>     message='Identity popup has not been opened.')
>   File "/Volumes/data/code/gecko/testing/marionette/client/marionette_driver/wait.py", line 154, in until
>     cause=last_exc)

The problem is still related to this execute script call, which which should return the outerWindowID but always returns `null` as if the browser would never exists. But I can clearly see the web page being loaded.

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Looks like that as long as we cannot have a fixed handle, there is nothing we can do here. We will have to wait for bug 1311041 or get rid of the Tab abstraction in Firefox Puppeteer, which I don't wanna do at this point.

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+1]]

With the simplification on bug 1573641 we can re-enable it for e10n.

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Bug 1573641 actually fixed it already.