http://mxr.mozilla.org/mozilla-central/source/browser/components/privatebrowsing/test/browser/global/browser_privatebrowsing_placestitle.js In order to port this test, the file needs to be copied to the perwindow/ directory, 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 per-window version of this test can have both the private and non-private window simultaneously open as opposed to the current way that the test is structured.

The test is running fine. However, as I understand the test make sure that on private windows the history entries are not change. But, this is not true and the returned title is changed on the private window.

This requires the patches in bug 722850 to land as well.

Then probably the last condition should be: > case 4: > // The second time that the page is loaded but in a private window > is(aPageTitle, "No Cookie", > "The page should be loaded with a cookie for the second time"); Since, it doesn't change on private window.

(In reply to comment #3) > Then probably the last condition should be: > > > case 4: > > // The second time that the page is loaded but in a private window > > is(aPageTitle, "No Cookie", > > "The page should be loaded with a cookie for the second time"); > > Since, it doesn't change on private window. No, the cookie database is currently broken in per-window PB builds until bug 722850 gets fixed.

Andres, now that bug 722850 has been fixed, can you please test to see if this test works, so that we can land it? Thanks!

I have been trying to run it, but the onTitleChanged observer, doesn't seen to be registered (or get notified) for the private window. > PlacesUtils.history.addObserver(historyObserver, false); Do you know if any of the latest commits, change this to ignore PB windows?

Josh might know.

This is expected behaviour. We don't modify the places database for private windows, so no notifications occur. The original test did not attempt to observe this behaviour in PB mode either.

Updated patch.

Comment on attachment 683614 [details] [diff] [review] Patch v2 Review of attachment 683614 [details] [diff] [review]: ----------------------------------------------------------------- The original test goes into PB mode <http://mxr.mozilla.org/mozilla-central/source/browser/components/privatebrowsing/test/browser/global/browser_privatebrowsing_placestitle.js#45>, loads a tab, and this check here <http://mxr.mozilla.org/mozilla-central/source/browser/components/privatebrowsing/test/browser/global/browser_privatebrowsing_placestitle.js#57> makes sure that this tab load doesn't cause a places title change. The new version of the test doesn't seem to be doing anything with the private window that it opens. I think you should do something similar to the existing test there as well.

Fixed patch.

Comment on attachment 683698 [details] [diff] [review] Patch v3 Review of attachment 683698 [details] [diff] [review]: ----------------------------------------------------------------- Looks great, thanks!