Closed
Bug 806685
Opened 12 years ago
Closed 12 years ago
Port browser_privatebrowsing_downloadLastDir.js and its friends to the new per-window PB APIs
Categories
(Firefox :: Private Browsing, defect)
Tracking
()
RESOLVED
FIXED
Firefox 19
People
(Reporter: ehsan.akhgari, Assigned: andreshm)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
48.48 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
http://mxr.mozilla.org/mozilla-central/source/browser/components/privatebrowsing/test/browser/global/browser_privatebrowsing_downloadLastDir.js
http://mxr.mozilla.org/mozilla-central/source/browser/components/privatebrowsing/test/browser/global/browser_privatebrowsing_downloadLastDir_c.js
http://mxr.mozilla.org/mozilla-central/source/browser/components/privatebrowsing/test/browser/global/browser_privatebrowsing_downloadLastDir_toggle.js
http://mxr.mozilla.org/mozilla-central/source/browser/components/privatebrowsing/test/browser/global/browser_privatebrowsing_DownloadLastDirWithCPS.js
In order to port this test, the files need to be copied to the perwindow/ directory, and then instead of setting privateBrowsingEnabled sequentially, the tests need to open new private browsing windows and then run the test on that window. This involves rewriting the test structure to make it asynchronous.
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → andres
Assignee | ||
Updated•12 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #680926 -
Flags: review?(ehsan)
Reporter | ||
Comment 2•12 years ago
|
||
Comment on attachment 680926 [details] [diff] [review]
Patch v1
Review of attachment 680926 [details] [diff] [review]:
-----------------------------------------------------------------
Looks great, thanks!
Attachment #680926 -
Flags: review?(ehsan) → review+
Reporter | ||
Comment 3•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 19
You need to log in
before you can comment on or make changes to this bug.
Description
•