823869 - Test which checks that the accept cookie dialog is not displayed in private browsing mode

Status: RESOLVED INVALID

(Mozilla QA Graveyard :: Mozmill Tests, defect, P3)

Description

[Daniela Petrovici]

We need a new test to check the cookies modal dialog when in private browsing mode (that it does not appear) and also in normal mode (that it appears). 

This is per comment #67 in bug 818456 (https://bugzilla.mozilla.org/show_bug.cgi?id=818456#c67)

Comment 1

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

As of how the initial comment is written it's unclear for me what this test has to do. Please make sure to file new bugs with a clear summary and clear steps. What is the cookie modal dialog? We have a lot of different ones. Please be explicit what this test should check.

Comment 2

[Daniela Petrovici]

The test will need to verify the following:
1) Have the Accept cookies from sites option set in Firefox for ask me every time
2) Open New Private Window
3) Verify that the Confirm setting cookie dialog is not displayed
4) Open New Window
5) Verify that the Confirm setting cookie dialog is displayed

Comment 3

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

Updating summary accordingly to to the test specifications. Does it mean that we don't have any already existent Mochitests for those tests? If not why can't this be implemented as a mochitest?

Comment 4

[Daniela Petrovici]

There were two mochitests (see below) that partially cover the functionality, but they do not explicitly verify that the cookie modal dialog does not appear in PB mode:
http://mxr.mozilla.org/mozilla-central/source/browser/components/privatebrowsing/test/browser/global/browser_privatebrowsing_popupblocker.js
http://mxr.mozilla.org/mozilla-central/source/browser/components/privatebrowsing/test/browser/global/browser_privatebrowsing_cookieacceptdialog.js

Comment 5

[Daniela Petrovici]

Based on the email thread the existing mochitest needs to be updated with the steps in comment #2

Comment 6

[Daniela Petrovici]

Logged bug 826311 for extending the existing mochitest.

Comment 7

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

Thanks. Closing this bug in favor of the Mochitest.