Fix use of enum and other reserved words since Bug 497869 landed

RESOLVED FIXED in seamonkey2.1b2

Status

SeaMonkey
General
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Philip Chee, Assigned: Philip Chee)

Tracking

Trunk
seamonkey2.1b2
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b10pre) Gecko/20110125 SeaMonkey/2.1b2pre

Error: enum is a reserved identifier
Source file: resource://gre/components/smileApplication.js
Line: 365, Column: 8
Source code:
    var enum = Utilities.windowMediator.getEnumerator("navigator:browser"); 

Error: Factory not registered = NS_ERROR_FACTORY_NOT_REGISTERED
Source file: chrome://communicator/content/bindings/notification.xml
Line: 1175
(Assignee)

Updated

6 years ago
Depends on: 497869
(Assignee)

Comment 1

6 years ago
Created attachment 507029 [details] [diff] [review]
Patch Ev1.0 fix enum strict errors
[Checked in: Comment 2]
Attachment #507029 - Flags: review?(bugspam.Callek)

Updated

6 years ago
Attachment #507029 - Flags: review?(bugspam.Callek) → review+
(Assignee)

Comment 2

6 years ago
Comment on attachment 507029 [details] [diff] [review]
Patch Ev1.0 fix enum strict errors
[Checked in: Comment 2]

Pushed to comm-central
http://hg.mozilla.org/comm-central/rev/75488924c92f

Leaving open in case other strict errors crop up.
(Assignee)

Comment 3

6 years ago
Jens, You might want to take note of this:

> +++ b/browser/base/content/syncSetup.js
> @@ -875,19 +875,19 @@ var gSyncSetup = {
>    _setFeedback: function (element, success, string) {
>      element.hidden = success || !string;
> -    let class = success ? "success" : "error";
> +    let classname = success ? "success" : "error";
>      let image = element.getElementsByAttribute("class", "statusIcon")[0];
> -    image.setAttribute("status", class);
> +    image.setAttribute("status", classname);
(In reply to comment #3)
> Jens, You might want to take note of this:

TUVM! Will take care of it in bug 618709 / bug 576970.
Depends on: 576970
Flags: in-testsuite-
Bustage-fix for bug 576970 comment 178:
http://hg.mozilla.org/comm-central/rev/0f313f7b192e
(AAv1) Fix xpcshell bustage from bug 576970, copied from bug 497869
Attachment #507029 - Attachment description: Patch Ev1.0 fix enum strict errors. → Patch Ev1.0 fix enum strict errors [Checked in: Comment 2]
Ftr, from bug 497869:
*syncSetup.js: Jens did it.
*nsBrowserGlue.js: I did it.
*fuelApplication.js: Philip did it
;-)

Let's reopen if need be.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b2
You need to log in before you can comment on or make changes to this bug.