730840 - Use getBrowserURL() in Firefox tests to ease porting them, part 2

Status: VERIFIED FIXED

(Firefox :: General, defect)

Description

[Serge Gautherie (:sgautherie)]

+++ This bug was initially created as a clone of Bug #717963 +++

"Found 4 matching lines in 3 files"

Comment 1

[Raymond Lee [:raymondlee]]

Attachment: v1

Updated to use getBrowserURL in browser tests.

Comment 2

[Serge Gautherie (:sgautherie)]

Comment on attachment 618578 [details] [diff] [review]
v1

Could you fix new

/browser/components/privatebrowsing/test/browser/browser_privatebrowsing_lastpbcontextexited.js
    * line 40 -- let newWin = window.openDialog("chrome://browser/content/", "_blank", "chrome,all,dialog=no");

too?

Comment 3

[Raymond Lee [:raymondlee]]

Attachment: v2

(In reply to Serge Gautherie (:sgautherie) from comment #2)
> Comment on attachment 618578 [details] [diff] [review]
> v1
> 
> Could you fix new
> 
> /browser/components/privatebrowsing/test/browser/
> browser_privatebrowsing_lastpbcontextexited.js
>     * line 40 -- let newWin = window.openDialog("chrome://browser/content/",
> "_blank", "chrome,all,dialog=no");
> 
> too?

Done.

Comment 4

[Raymond Lee [:raymondlee]]

Attachment: Patch for checkin

Comment 5

[Raymond Lee [:raymondlee]]

Comment on attachment 618949 [details] [diff] [review]
Patch for checkin

Passed Try
https://tbpl.mozilla.org/?tree=Try&rev=ef5f8d617329

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e4f72e3b1081

Comment 7

[Ryan VanderMeulen [:RyanVM]]

http://hg.mozilla.org/mozilla-central/rev/e4f72e3b1081

Comment 8

[Serge Gautherie (:sgautherie)]

V.Fixed, per MXR search.