Closed
Bug 806725
Opened 12 years ago
Closed 12 years ago
Port test_localStorageBasePrivateBrowsing.html to the new per-tab PB APIs
Categories
(Firefox :: Private Browsing, defect)
Tracking
()
RESOLVED
FIXED
Firefox 20
People
(Reporter: ehsan.akhgari, Assigned: bellindira)
References
(Blocks 1 open bug)
Details
Attachments
(2 files, 1 obsolete file)
25.95 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
1.65 KB,
patch
|
jdm
:
review+
|
Details | Diff | Splinter Review |
http://mxr.mozilla.org/mozilla-central/source/dom/tests/mochitest/localstorage/test_localStorageBasePrivateBrowsing.html 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.
Reporter | ||
Comment 1•12 years ago
|
||
Removed this test from per-window PB builds: https://hg.mozilla.org/mozilla-central/rev/3c0945f95ceb
Updated•12 years ago
|
Assignee: nobody → bellindira
Assignee | ||
Updated•12 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•12 years ago
|
||
The test is ported however it looks like localStorage works different on private browsing per window than on previous private browsing mode. Because once a new private browsing window is opened and closed, it is not possible to get the values of previous saved items in a normal window. As you will see when running this test, it will throw this error: 113 ERROR TEST-UNEXPECTED-FAIL | chrome://mochitests/content/chrome/dom/tests/mochitest/localstorage/test_localStorageBasePrivateBrowsing_perwindowpb.html | back in normal mode - got null, expected persistent1 make: *** [mochitest-chrome] Error 1 Please let me know what I should do on this check. Thanks
Attachment #691428 -
Flags: feedback?(ehsan)
Reporter | ||
Comment 3•12 years ago
|
||
That seems wrong. Josh is looking into this.
Comment 4•12 years ago
|
||
The test passes with these changes.
Attachment #691548 -
Flags: review?(ehsan)
Updated•12 years ago
|
Attachment #691428 -
Attachment is obsolete: true
Attachment #691428 -
Flags: feedback?(ehsan)
Reporter | ||
Comment 5•12 years ago
|
||
Comment on attachment 691548 [details] [diff] [review] patch Review of attachment 691548 [details] [diff] [review]: ----------------------------------------------------------------- I have some indentation nits which I fixed myself. :-) Thanks everyone!
Attachment #691548 -
Flags: review?(ehsan) → review+
Reporter | ||
Comment 6•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/fd919eb97465
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 20
Updated•12 years ago
|
Attachment #691548 -
Attachment description: BugÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃÂÃ → patch
Comment 7•12 years ago
|
||
This is a typo in this patch. http://mxr.mozilla.org/mozilla-central/source/dom/tests/mochitest/localstorage/test_localStorageBasePrivateBrowsing_perwindowpb.html?force=1#42 win.removeEventListener("DOMConten/tLoaded", onInnerLoad, true);
Assignee | ||
Comment 8•12 years ago
|
||
Attachment #693639 -
Flags: review?(ehsan)
Updated•12 years ago
|
Attachment #693639 -
Flags: review?(ehsan) → review+
Updated•12 years ago
|
Keywords: checkin-needed
Comment 9•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/3e9f8f3605bb
Keywords: checkin-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•