The default bug view has changed. See this FAQ.

Would like to to override nsIInlineSpellChecker with javascript but [noscript] methods are blocking it.

RESOLVED FIXED in mozilla12

Status

()

Core
Editor
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: josh, Assigned: khuey)

Tracking

7 Branch
mozilla12
x86_64
Windows 7
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug][mentor=ehsan])

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0.1) Gecko/20110928 Firefox/7.0.1 SeaMonkey/2.4.1
Build ID: 20110928161145

Steps to reproduce:

I have a project that overrides nsIInlineSpellChecker in C++ code.
I would like to convert it to Javascript, but there are two [noscript] methods in nsIInlineSpellChecker that are blocking the creation of a Javascript object that can replace mozInlineSpellChecker successfully.

  [noscript] void init(in nsIEditor aEditor);
  [noscript] void cleanup(in boolean aDestroyingFrames);

Is [noscript] required on these methods?  Is it possible to remove it?
Looks like it was this way in the initial checkin http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/editor/txtsvc/public/nsIInlineSpellChecker.idl&rev=1.1&root=/cvsroot

I don't see any reason why we can't drop [noscript] here.  Ehsan?
Status: UNCONFIRMED → NEW
Component: General → Editor
Ever confirmed: true
Product: Firefox → Core
QA Contact: general → editor
Yeah, we can do it.
Whiteboard: [good first bug][mentor=ehsan]
Created attachment 586053 [details] [diff] [review]
Just do it
Assignee: nobody → khuey
Status: NEW → ASSIGNED
Attachment #586053 - Flags: review?(ehsan)

Updated

5 years ago
Attachment #586053 - Flags: review?(ehsan) → review+
https://hg.mozilla.org/mozilla-central/rev/65983b0e4a76
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
You need to log in before you can comment on or make changes to this bug.