1374094 - Update SeaMonkey debugQA extension

Status: RESOLVED FIXED

(SeaMonkey :: General, enhancement)

Description

[Frank-Rainer Grahl (:frg)]

debugQA should be compatible with SeaMonkey 2.49.1 when a new release is put on AMO. The current debugQA is only comaptible with SeaMonkey 2.50 and up because of Bug 1322414 (remove content-primary). Bug 1324899 fixed the backwards comaptibility for DOMi so the same workaround can be applied here. Also moztrap has been discontinued and needs to be removed from the menu. 

For the editor tests to work Bug 1364772 needs to be fixed too.

Comment 1

[Frank-Rainer Grahl (:frg)]

Attachment: 1374094-debugQA245.patch

Tested with Bug 1364772 fixed 2.53a1 and 2.49.1.

Comment 2

[Ian Neal]

Comment on attachment 8878921 [details] [diff] [review]
1374094-debugQA245.patch

>+  // Fix up content primary for older versions.
>+  // See bug 1324899.
>+  if (Services.vc.compare(Services.appinfo.platformVersion, "53.0a1") < 0) {
>+    document.getElementById("content-frame").setAttribute("type",
>+                                                          "content-primary");
>+  }
As TextEditorOnLoad() is only used by debugQA then the whole of this could be moved to the top of that function, thus saving adding another file and simplifying the patch. You may find Services.jsm already loaded in the context.

f+ for the moment

Comment 3

[Frank-Rainer Grahl (:frg)]

> As TextEditorOnLoad() is only used by debugQA

Do you mean moving TextEditorOnLoad() into debugQA? This might break other add-ons which use this function? 

The setAttribute needs to stay in debugQA obviously. It could probably be moved into debugQAEditorOverlay.js but this would all muddle it a bit.

Comment 4

[Ian Neal]

(In reply to Frank-Rainer Grahl (:frg) from comment #3)
> > As TextEditorOnLoad() is only used by debugQA
> 
> Do you mean moving TextEditorOnLoad() into debugQA? This might break other
> add-ons which use this function? 
No, I mean don't create a new file, just add the new code to TextEditorOnLoad()

Comment 5

[Frank-Rainer Grahl (:frg)]

> No, I mean don't create a new file, just add the new code to TextEditorOnLoad()

Can't be done. editingOverlay.js is part of SeaMonkey. The code needs to be in the add-on and must be called during load to set the attribute only when debugQA is used with SeaMonkey 2.49.x and older versions. Only needed so that a new AMO version works with them. Check bug 1324899 for DOMi.

Comment 6

[Ian Neal]

Comment on attachment 8878921 [details] [diff] [review]
1374094-debugQA245.patch

(In reply to Frank-Rainer Grahl (:frg) from comment #5)
> > No, I mean don't create a new file, just add the new code to TextEditorOnLoad()
> 
> Can't be done. editingOverlay.js is part of SeaMonkey. The code needs to be
> in the add-on and must be called during load to set the attribute only when
> debugQA is used with SeaMonkey 2.49.x and older versions. Only needed so
> that a new AMO version works with them. Check bug 1324899 for DOMi.

How do we tell if any Add-ons are using TextEditorOnLoad()?
If none, merge into new function in debugQATextEditorShell.js

>+++ b/suite/debugQA/content/debugQATextEditorShell.js
>@@ -0,0 +1,17 @@
>+/* -*- Mode: Java; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*- *
>+/* This Source Code Form is subject to the terms of the Mozilla Public
>+ * License, v. 2.0. If a copy of the MPL was not distributed with this
>+ * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
>+
>+Components.utils.import("resource://gre/modules/Services.jsm");
>+
>+function debugQATextEditorShell_load()
Maybe debugQATextEditorOnLoad() instead

>+{
>+  // Fix up content primary for older versions.
>+  // See bug 1324899.
>+  if (Services.vc.compare(Services.appinfo.platformVersion, "53.0a1") < 0) {
>+    document.getElementById("content-frame").setAttribute("type",
>+                                                          "content-primary");
>+  }
>+  TextEditorOnLoad();
>+}

r=me either way.

Comment 7

[Frank-Rainer Grahl (:frg)]

Jorg,

would you be ok with removing TextEditorOnLoad() from editor/ui/composer/content/editingOverlay.js?

Seems to be only needed in the SeaMonkey debugQA addon. Still need to check if there will be any fallout in the addon but doesn't think so. Everone else seems to use EditorOnLoad().

Do you think other addons use it? Dind't find any references searching for it.

Comment 8

[Jorg K (CEST = GMT+2)]

No problem.

Comment 9

[Frank-Rainer Grahl (:frg)]

Attachment: 1374094-debugQA.patch

The old patch has overstayed its welcome. debugQA is now integrated into SeaMonkey and no longer tied to the Nightlies. At this point in time we no longer care about aynthing below 2.53.
New patch is compatible with 2.53 and up. Trunk would need other additonal patches. We deal with this or not later.

Basically clean up now. Some outdated web sites removed and some added. Editor change removed and new routine no longer neccesary. If we ever come to Nightly we can clean it up here.

Comment 10

[Ian Neal]

Comment on attachment 9073860 [details] [diff] [review]
1374094-debugQA.patch

[Triage Comment]
What's the reason behind removing www.mozilla.org and quality.mozilla.org

r/a=me with that answered

Comment 11

[Frank-Rainer Grahl (:frg)]

What's the reason behind removing www.mozilla.org and quality.mozilla.org

Discussed on irc a while ago. A follow-up bug should be filed for a new developer menu. On quality.mozilla.org there is also nothing left of value for SeaMonkey devs or users.

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/a41dab1ff796
Clean up debugQA. r=IanN

Comment 13

[Frank-Rainer Grahl (:frg)]

https://hg.mozilla.org/releases/comm-esr60/rev/7c134fa3cd9fa430ce46202db2dee45b889323ef

Comment 14

[Frank-Rainer Grahl (:frg)]

Target 2.53.1
https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/16c62a6d97c6061daba0b40709ad45400a5322f6