Closed Bug 1348319 Opened 3 years ago Closed 3 years ago

Port bug 1343821 to SeaMonkey [Remove Dynamic Skin Switching (DSS) support]

Categories

(SeaMonkey :: Themes, enhancement)

enhancement
Not set

Tracking

(seamonkey2.52 fixed)

RESOLVED FIXED
seamonkey2.52
Tracking Status
seamonkey2.52 --- fixed

People

(Reporter: frg, Assigned: frg)

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1348032 +++

Bug 1343821 removes the pref extensions.dss.enabled.

SM has additional places of this in suite/debugQA/content/pref-debug.xul which is better a SM developer removes them too and can check that all still works.
Attachment #8848544 - Flags: review?(iann_bugzilla)
Comment on attachment 8848544 [details] [diff] [review]
1348319-dssremove.patch

>+++ b/suite/debugQA/install.rdf
>@@ -5,17 +5,17 @@
> 
> 
> #filter substitution
> 
> <RDF xmlns="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
>      xmlns:em="http://www.mozilla.org/2004/em-rdf#">
>   <Description about="urn:mozilla:install-manifest">
>     <em:id>debugQA@mozilla.org</em:id>
>-    <em:version>2.0.2</em:version>
>+    <em:version>2.0.3</em:version>
> 
>     <em:targetApplication>
>       <!-- Suite -->
>       <Description>
>         <em:id>{92650c4d-4b8e-4d2a-b7eb-24ecf4f6b63a}</em:id>
>         <em:minVersion>2.45</em:minVersion>
Do we need to bump the minVersion too?

r=me with that answered
Attachment #8848544 - Flags: review?(iann_bugzilla) → review+
>> Do we need to bump the minVersion too?

The usual problem with the extension not updating itself if the min version doesn't change. Reminds me to ask Callek if he found the password for AMO yet :)

FRG
https://hg.mozilla.org/comm-central/rev/4cc7e241a7d02dd1a662f37cd2b674a1566f2306
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.52
You need to log in before you can comment on or make changes to this bug.