Don't hardcode defaultChromeURI in BrowserCLH.js

RESOLVED FIXED in Firefox 42

Status

()

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: fabrice, Assigned: fabrice)

Tracking

Trunk
Firefox 42
Points:
---

Firefox Tracking Flags

(firefox42 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Currently the value of the toolkit.defaultChromeURI pref is ignored in fennec.
(Assignee)

Comment 1

3 years ago
Created attachment 8629676 [details] [diff] [review]
fennec-browserCLH.patch

Use the pref while keeping the same default value.
Assignee: nobody → fabrice
Attachment #8629676 - Flags: review?(mark.finkle)
Comment on attachment 8629676 [details] [diff] [review]
fennec-browserCLH.patch


>         let flags = "chrome,dialog=no,all";
>-        browserWin = openWindow(null, "chrome://browser/content/browser.xul", "_blank", flags, args);
>+        let url = "chrome://browser/content/browser.xul";

Just to reduce any potential naming issues, can you switch this to | chromeURL | ?
Attachment #8629676 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/mozilla-central/rev/0bc555946d99
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox42: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
You need to log in before you can comment on or make changes to this bug.