Closed Bug 274866 Opened 20 years ago Closed 19 years ago

hook up mailnews prefs for the permissions extension

Categories

(SeaMonkey :: MailNews: Message Display, defect)

Product:
SeaMonkey
Component:
MailNews: Message Display
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: mvl, Assigned: mvl)

References

Details

Attachments

(1 file)

patch v1
1.11 KB, patch
dwitte
: review+
darin.moz
: superreview+
Details | Diff | Splinter Review 
after fixing bug 240070, there is now a way to block all content in mailnews.
But the pref used there to block content in mailnews doesn't have UI yet.
I suggest to just make it use the current pref, so that we don't have to update
the UI and don't have to write pref migration code.
Attached patch patch v1 — — Splinter Review 
This patch looks simple, but it does mean that all remote content will be
blocked when the pref is set, like images, iframes and stylesheets.
Attachment #168854 - Flags: review?(mscott)
Attachment #168854 - Flags: review?(mscott) → review+
Comment on attachment 168854 [details] [diff] [review]
patch v1

see comment 1 for what this patch actually does
Attachment #168854 - Flags: superreview?(bienvenu)
http://lxr.mozilla.org/mozilla/source/configure.in#3504

do we need to add this to the list of extensions for thunderbird? what about
other apps in that list, e.g. camino?

if we do, the code will need some massaging since nsContentBlocker is built
always, and we don't want that.

also, for the record, there exists profile migration code for firefox - see e.g.
http://lxr.mozilla.org/mozilla/source/browser/base/content/browser.js#5071
Comment on attachment 168854 [details] [diff] [review]
patch v1

I'm going to pass the buck to mscott on this one.
Attachment #168854 - Flags: superreview?(bienvenu) → superreview?(mscott)
(In reply to comment #3)
> http://lxr.mozilla.org/mozilla/source/configure.in#3504
> 
> do we need to add this to the list of extensions for thunderbird? what about
> other apps in that list, e.g. camino?

thunderbird has its own contentpolicy, so doesn't need this. Camino might need
it. I don't know if that currentlt allows blocking of images.

> if we do, the code will need some massaging since nsContentBlocker is built
> always, and we don't want that.

Only browsers need this extension, not thunderbird. No need for the massaging.

> also, for the record, there exists profile migration code for firefox - see e.g.
> http://lxr.mozilla.org/mozilla/source/browser/base/content/browser.js#5071

I already covered migration of firefox prefs. somewhere.
And this patch doesn't need any migration, because it makes new code read the
old, existing pref.
camino doesn't use image blocking as of 0.8+. so i guess all my questions are
answered :)

mscott, any chance of getting to this before beta? it would be a nice little
codesize win to get rid of old code, if we can land this.
Comment on attachment 168854 [details] [diff] [review]
patch v1

Darin, can you look at this? It only affects seamonkey mailnews, not
thunderbird.
With this patch, the old img mananger can be removed. codesize win :)
Attachment #168854 - Flags: superreview?(mscott) → superreview?(darin)
Attachment #168854 - Flags: superreview?(darin) → superreview+
patch checked in
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Blocks: 287672
Hi
I just found this bug which still doesn't seem to be fixed...?
I'm using Mozilla 1.7.8 (20050511) and i have remote images disabled in the prefs.
I received a html mail with a stylesheet and first it did was ask me if the
certificate was correct.. that's when i saw that something isn't working as it's
supposed to...?
The Content type is text/html and the mail begins with:
<html>
<head>
<title>citydisc.ch - Infos</title>
<link href="https://212.59.151.34/css/dm/win.css" rel=stylesheet type=text/css>
</head>

So it appears as if that stylesheet got loaded...?!
it's fixed on trunk, not on the 1.8 branch.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: