Port browser_ConsoleStoragePBTest.js to the new per-tab PB APIs

RESOLVED FIXED in Firefox 20

Status

()

defect
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: Ehsan, Assigned: marioalv)

Tracking

(Blocks 1 bug)

unspecified
Firefox 20
x86
macOS
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

http://mxr.mozilla.org/mozilla-central/source/dom/tests/browser/browser_ConsoleStoragePBTest.js

In order to port this test, the file needs to be copied to the same directory (perhaps with "_perwindowpb" appended to its file name), and then instead of setting privateBrowsingEnabled, we need to open a new private browsing window and then run the test on that window.  Note that the original test should only be added to the list of test files in Makefile.in ifndef MOZ_PER_WINDOW_PRIVATE_BROWSING, and the new test file should be added to the list with the reverse condition.
Removed this test from global PB builds: https://hg.mozilla.org/mozilla-central/rev/fd19c6212b41
s/global/per-window/
Assignee: nobody → marioalv.mozilla
(Assignee)

Updated

7 years ago
Status: NEW → ASSIGNED
Comment on attachment 690638 [details] [diff] [review]
Patch to migrate the test to the new per window private mode

Review of attachment 690638 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/tests/browser/browser_ConsoleStoragePBTest.js
@@ +17,5 @@
>  
> +  function getInnerWindowId(aWindow) {
> +    return aWindow.QueryInterface(Ci.nsIInterfaceRequestor)
> +      .getInterface(Ci.nsIDOMWindowUtils)
> +        .currentInnerWindowID;

Nit: indentation.
Attachment #690638 - Flags: review?(ehsan) → review+
https://hg.mozilla.org/mozilla-central/rev/3ce22cb51e56
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 20
You need to log in before you can comment on or make changes to this bug.