If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

use gBrowser instead of getBrowser() in private browsing code

RESOLVED FIXED in Firefox 3.7a1

Status

()

Firefox
Private Browsing
--
trivial
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: dao, Assigned: dao)

Tracking

Trunk
Firefox 3.7a1
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
getBrowser() is deprecated and exists only for extension compatibility.
(Assignee)

Comment 1

8 years ago
Created attachment 401648 [details] [diff] [review]
patch
Attachment #401648 - Flags: review?(ehsan.akhgari)
Comment on attachment 401648 [details] [diff] [review]
patch

r=me.
Attachment #401648 - Flags: review?(ehsan.akhgari) → review+
(Assignee)

Comment 3

8 years ago
http://hg.mozilla.org/mozilla-central/rev/d93f6e968a19
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a1
(Assignee)

Comment 4

8 years ago
I reverted the test changes, as I realized that getBrowser() was called on a view-source window rather than a browser window:

http://hg.mozilla.org/mozilla-central/rev/14b3ba7b71ae
(Assignee)

Updated

8 years ago
Blocks: 448936
You need to log in before you can comment on or make changes to this bug.