ForgetAboutSite.jsm is included with the wrong url

RESOLVED INVALID

Status

()

Toolkit
Data Sanitization
RESOLVED INVALID
6 years ago
6 years ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(firefox18 affected)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
It's included as resource://gre/modules/ForgetAboutSite.jsm while it's a browser module.
(Assignee)

Updated

6 years ago
status-firefox18: --- → affected
(Assignee)

Comment 1

6 years ago
Created attachment 672236 [details] [diff] [review]
Include ForgetAboutSite.jsm from resource:///modules instead of resource://gre/modules
(Assignee)

Comment 2

6 years ago
In fact, this was indirectly fixed by bug 799229.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → INVALID

Comment 3

6 years ago
Oops, sorry.  Just to make sure that I don't get this wrong in the future, what is the difference between resource://gre/modules and resource://modules?
(Assignee)

Comment 4

6 years ago
There are two parts of a xul application: the app part, and the gecko (gre) part. resource:/// and resource://app/ point to the app part, resource://gre/ to the gecko part.
In Firefox, at least until bug 755724, and unless you build it as a xulrunner application, both resource:/// and resource://gre/ point to the same location, so discrepancies go unnoticed.
In the past, this was only making problems to xulrunner applications, mostly, but now, this is going to be a problem for metro builds.

Updated

6 years ago
Component: Private Browsing → Forget About Site
Product: Firefox → Toolkit
You need to log in before you can comment on or make changes to this bug.