Port |Bug 422700 – Get bug 376238 fixed or disable images on drag feedback| to SeaMonkey

RESOLVED FIXED in seamonkey2.0a1

Status

RESOLVED FIXED
11 years ago
10 years ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

({dogfood})

Trunk
seamonkey2.0a1
x86
Linux
dogfood

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

11 years ago
Per bug 360488 dependency.
(Assignee)

Updated

11 years ago
Blocks: 360488
(Assignee)

Comment 1

11 years ago
Created attachment 316584 [details] [diff] [review]
(Av1-Bw) <mailnews.js> whitespace cleanup
Assignee: mail → sgautherie.bz
Status: NEW → ASSIGNED
(Assignee)

Comment 2

11 years ago
Created attachment 316585 [details] [diff] [review]
(Av1) <mailnews.js> whitespace cleanup
[Checkin: Comment 10]

*Converts tabs to spaces.
*Removes spaces at end of lines.
*Add spaces between preference names and values.

Use Av1-Bw to ease review.
Attachment #316585 - Flags: superreview?(bienvenu)
Attachment #316585 - Flags: review?(bienvenu)
(Assignee)

Comment 3

11 years ago
Created attachment 316587 [details] [diff] [review]
(Bv1) <mailnews.js> Add new preference
[Checkin: Comment 9]

It seems this file is still used/shared by Thunderbird:
<http://mxr.mozilla.org/mozilla/search?string=mailnews.js>
At least, it's packaged in
[Mozilla Thunderbird, version 3.0a1pre (2008032702)] (nightly) (W2Ksp4)

Then, should I (re)move the preference from <all-thunderbird.js> ? (see bug 422700)
Or add an |#ifdef MOZ_SUITE| here ?
Attachment #316587 - Flags: superreview?(bienvenu)
Attachment #316587 - Flags: review?(bienvenu)

Comment 4

11 years ago
Comment on attachment 316585 [details] [diff] [review]
(Av1) <mailnews.js> whitespace cleanup
[Checkin: Comment 10]

does this patch have anything to do with this bug? I can't see that it does...
(Assignee)

Comment 5

11 years ago
Comment on attachment 316585 [details] [diff] [review]
(Av1) <mailnews.js> whitespace cleanup
[Checkin: Comment 10]

No, this patch is unrelated to any "bug":
simple cleanup while there.

Comment 6

10 years ago
Comment on attachment 316585 [details] [diff] [review]
(Av1) <mailnews.js> whitespace cleanup
[Checkin: Comment 10]

ok, thx
Attachment #316585 - Flags: superreview?(bienvenu)
Attachment #316585 - Flags: superreview+
Attachment #316585 - Flags: review?(bienvenu)
Attachment #316585 - Flags: review+
(Assignee)

Updated

10 years ago
Keywords: checkin-needed
Whiteboard: [c-n: Av1 // Leave opened]

Updated

10 years ago
Attachment #316587 - Flags: superreview?(bienvenu)
Attachment #316587 - Flags: superreview+
Attachment #316587 - Flags: review?(bienvenu)
Attachment #316587 - Flags: review+
(Assignee)

Comment 7

10 years ago
David, what about my questions in comment 3 ?

Comment 8

10 years ago
better to remove it from all-thunderbird.js, I think, thx.
(Assignee)

Updated

10 years ago
Whiteboard: [c-n: Av1 // Leave opened] → [c-n: Av1, Bv1 // Leave opened]

Comment 9

10 years ago
Comment on attachment 316587 [details] [diff] [review]
(Bv1) <mailnews.js> Add new preference
[Checkin: Comment 9]

Checking in mailnews/mailnews.js;
/cvsroot/mozilla/mailnews/mailnews.js,v  <--  mailnews.js
new revision: 3.312; previous revision: 3.311
done
Attachment #316587 - Attachment filename: 429821-Bv1_mailnews.js.patch → [checked in] 429821-Bv1_mailnews.js.patch

Comment 10

10 years ago
Comment on attachment 316585 [details] [diff] [review]
(Av1) <mailnews.js> whitespace cleanup
[Checkin: Comment 10]

Checking in mailnews/mailnews.js;
/cvsroot/mozilla/mailnews/mailnews.js,v  <--  mailnews.js
new revision: 3.313; previous revision: 3.312
done
Attachment #316585 - Attachment filename: 429821-Av1_mailnews.js.patch → [checked in] 429821-Av1_mailnews.js.patch

Updated

10 years ago
Keywords: checkin-needed
(Assignee)

Updated

10 years ago
Attachment #316587 - Attachment description: (Bv1) <mailnews.js> Add new preference → (Bv1) <mailnews.js> Add new preference [Checkin: Comment 9]
Attachment #316587 - Attachment filename: [checked in] 429821-Bv1_mailnews.js.patch → 429821-Bv1_mailnews.js.patch
(Assignee)

Updated

10 years ago
Attachment #316585 - Attachment description: (Av1) <mailnews.js> whitespace cleanup → (Av1) <mailnews.js> whitespace cleanup [Checkin: Comment 10]
Attachment #316585 - Attachment filename: [checked in] 429821-Av1_mailnews.js.patch → 429821-Av1_mailnews.js.patch
(Assignee)

Updated

10 years ago
Attachment #316584 - Attachment is obsolete: true
(Assignee)

Updated

10 years ago
Blocks: 422700
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
No longer depends on: 422700
Resolution: --- → FIXED
Whiteboard: [c-n: Av1, Bv1 // Leave opened]
(Assignee)

Updated

10 years ago
No longer blocks: 360488
You need to log in before you can comment on or make changes to this bug.