[AxPlugin] Remove old ActiveScript support from plugin

RESOLVED FIXED

Status

Core Graveyard
Embedding: ActiveX Wrapper
RESOLVED FIXED
15 years ago
6 years ago

People

(Reporter: Adam Lock, Assigned: Adam Lock)

Tracking

Trunk
x86
Windows XP

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

3.71 KB, patch
David Bradley
: review+
Alec Flett
: superreview+
Details | Diff | Splinter Review
(Assignee)

Description

15 years ago
The plugin has some stub code from a time when it was going to invoke the
ActiveScript engine for .axs files. It should all be removed.
(Assignee)

Comment 1

15 years ago
Created attachment 121005 [details] [diff] [review]
Patch

Patch removes stub ActiveScript support
(Assignee)

Comment 2

15 years ago
Comment on attachment 121005 [details] [diff] [review]
Patch

Requesting r/sr on this simple patch to remove some old stub code from the
plugin.
Attachment #121005 - Flags: superreview?(alecf)
Attachment #121005 - Flags: review?(dbradley)

Comment 3

15 years ago
Comment on attachment 121005 [details] [diff] [review]
Patch

r=dbradley
Attachment #121005 - Flags: review?(dbradley) → review+

Comment 4

15 years ago
Comment on attachment 121005 [details] [diff] [review]
Patch

sr=alecf
Attachment #121005 - Flags: superreview?(alecf) → superreview+
(Assignee)

Comment 5

15 years ago
Fix is checked in
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
Component: Embedding: ActiveX Wrapper → Embedding: ActiveX Wrapper
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.