Closed
Bug 1115076
Opened 10 years ago
Closed 10 years ago
test_sts_privatebrowsing_perwindowpb.html fails when run as a standalone directory
Categories
(Core :: Security: PSM, defect)
Tracking
()
RESOLVED
FIXED
mozilla37
People
(Reporter: vaibhav1994, Assigned: ehsan.akhgari)
References
Details
Attachments
(1 file)
(deleted),
patch
|
jdm
:
review+
|
Details | Diff | Splinter Review |
in bug 1110982, we are looking to enable tests where we run a fresh browser instance per directory. Usually what happens is that a few tests fail because they accidentally depend on the state of the browser from an earlier test.
In the try run:
https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=1a5fa04f057a
In the linux opt/debug oth chunks, the test fails as following:
2349 INFO TEST-UNEXPECTED-FAIL | security/manager/ssl/tests/mochitest/stricttransportsecurity/test_sts_privatebrowsing_perwindowpb.html | Test timed out. - expected PASS
Comment 1•10 years ago
|
||
this is failing locally for me, I verified I have e10s turned off and I get:
./mach mochitest-chrome security/manager/ssl/tests/mochitest
.
:
135 INFO State of example.com: true
136 INFO TEST-PASS | security/manager/ssl/tests/mochitest/stricttransportsecurity/test_sts_privatebrowsing_perwindowpb.html | in ROUND plain, test alreadysecure
137 INFO DONE WITH ROUND plain
138 INFO Cleaning up STS data
139 INFO State of example.com: false
-*-*- UserCustomizations (parent): document created: http://mochi.test:8888/chrome/security/manager/ssl/tests/mochitest/stricttransportsecurity/page_blank.html
-*-*- UserCustomizations (parent): _injectInWindow
-*-*- UserCustomizations (parent): principal status: 0
140 INFO In private window
141 INFO State of example.com: false
142 INFO TESTS LEFT IN ROUND subdom: 4
WARNING: content window passed to PrivateBrowsingUtils.isWindowPrivate. Use isContentWindowPrivate instead (but only for frame scripts).
pbu_isWindowPrivate@resource://gre/modules/PrivateBrowsingUtils.jsm:25:14
@chrome://browser/content/aboutPrivateBrowsing.js:12:6
WARNING: content window passed to PrivateBrowsingUtils.isWindowPrivate. Use isContentWindowPrivate instead (but only for frame scripts).
pbu_isWindowPrivate@resource://gre/modules/PrivateBrowsingUtils.jsm:25:14
@chrome://browser/content/aboutPrivateBrowsing.js:37:8
^Cruntests.py | Received keyboard interrupt.
Comment 2•10 years ago
|
||
The test passes, if I comment out this line:
http://dxr.mozilla.org/mozilla-central/source/security/manager/ssl/tests/mochitest/stricttransportsecurity/test_sts_privatebrowsing_perwindowpb.html#217
of course this isn't testing private browsing really.
I did notice this has similar code to dom/tests/mochitest/localstorage/test_localStorageQuotaPrivateBrowsing_perwindowpb.html which is failing as well in bug 1115063.
:ehsan, you have been involved in both of these bugs, do you have more information why these specific tests might be failing when run as a single directory?
Flags: needinfo?(ehsan)
Assignee | ||
Comment 3•10 years ago
|
||
How do I reproduce this? For me, both ./mach mochitest-chrome security/manager/ssl/tests/mochitest/stricttransportsecurity/ and ./mach mochitest-chrome security/manager/ssl/tests/mochitest/ pass locally.
Flags: needinfo?(ehsan) → needinfo?(jmaher)
Comment 4•10 years ago
|
||
hmm, maybe there is a difference in our environments. I am running linux64 (ubuntu 14.04) with a local build in opt mode. Is there something I could provide to help determine why this fails for me locally as I outlined in comment #1 ?
Flags: needinfo?(jmaher)
Assignee | ||
Comment 5•10 years ago
|
||
Oh hmm, this seems to be Linux specific. I'll take a look on Linux, probably next week.
Flags: needinfo?(ehsan)
Assignee | ||
Comment 6•10 years ago
|
||
We end up loading about:privatebrowsing in the middle of this test, which is why it times out.
Assignee: nobody → ehsan
Flags: needinfo?(ehsan)
Assignee | ||
Comment 7•10 years ago
|
||
Attachment #8542669 -
Flags: review?(josh)
Updated•10 years ago
|
Attachment #8542669 -
Flags: review?(josh) → review+
Comment 8•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
You need to log in
before you can comment on or make changes to this bug.
Description
•