806686 - Port browser_privatebrowsing_geoprompt.js to the new per-window PB APIs

Status: RESOLVED FIXED

(Firefox :: Private Browsing, defect)

Description

[(no longer active)]

http://mxr.mozilla.org/mozilla-central/source/browser/components/privatebrowsing/test/browser/global/browser_privatebrowsing_geoprompt.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.

Comment 1

[Bellindira Castillo [:bellindira]]

Attachment: Patch v1

The test is ported to PB perwindow model as I think it should be. However, the former test (the one in global directory) doesn't pass, it has this error:
 JavaScript Error: "TypeError: notification is null" {file: "chrome://mochitests/content/browser/browser/components/privatebrowsing/test/browser/global/browser_privatebrowsing_geoprompt.js" line: 23}
So, the new test has the same error too:
JavaScript Error: "TypeError: notification is null" {file: "chrome://mochitests/content/browser/browser/components/privatebrowsing/test/browser/perwindow/browser_privatebrowsing_geoprompt.js" line: 25}

I tried to fix it by:
- setting an interval to wait for the notification
- importing the "resource://gre/modules/PopupNotifications.jsm" in case it wasn't loaded properly
- using executeSoon
However none of it solved the problem.

So, please let me know if you have any idea about it.

Comment 2

[(no longer active)]

Comment on attachment 680930 [details] [diff] [review]
Patch v1

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

::: browser/components/privatebrowsing/test/browser/global/browser_privatebrowsing_geoprompt.js
@@ +18,2 @@
>  
> +        let notification = PopupNotifications.getNotification("geolocation");

First of all, you need to access PopupNotifications from aWindow.

Also, there is a whole bunch of asynchronous stuff before the notification object gets created, so you need to "wait" until that happens.

I decided that I'll fix that and land the patch.  Here's the changes that I made to your patch: https://gist.github.com/4068961

Comment 3

[(no longer active)]

https://hg.mozilla.org/mozilla-central/rev/b9e99f5e8847