319356 - Apply consistent camel casing in nsIEditorSpellCheck.idl

Status: NEW

(Core :: DOM: Editor, defect)

Description

[timeless]

Hi brett. this bug is not optional as you did not get an editor sign off and no sr nor editor peer should ever have granted any mark for your changes.

deadline is 1.9a. if you can't arrange for this to be done, i'll find an editor owner to agree to revert your changes.

In short, you faield to get a module owner/peer review for editor on your changes.

The editor interface changes are unacceptable and the superreviewer should have blocked all of them.
 
At the very least things should have proper idl case.

converting a property (inlinespellchecker) into a function with a maybe init is uncool. instead add a new property that indicates whether accessing the property will create it as needed.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Er... timeless, did you read the mail I sent about this?

1) Brett DID NOT CHANGE THE CASE OF ANY IDL METHODS OR PROPERTIES IN EDITOR.
2) The CanSpellCheck method that was added was uppercase simply because every
   single other method in that interface is.  If you want that interface cleaned
   up, that's not really brett's job.
3) Your proposal for the inline spell checker property doesn't handle the case
   we need to handle here -- getting the existing spellchecker without creating
   it if it hasn't been created yet.

Maybe I'm missing something, but what are the exact review comments on https://bugzilla.mozilla.org/attachment.cgi?id=201876 from you as an editor peer?  Please quote the parts of that patch that you're commenting on?

Comment 2

[Brett Wilson]

I haven't heard of any specific regressions other than the title of this bug, and I'll certainly fix anything that is broken. Timeless, could you please be more specific?

> 1) Brett DID NOT CHANGE THE CASE OF ANY IDL METHODS OR PROPERTIES IN EDITOR.

I did change one: inlineSpellChecker. I thought I had the proper signoffs. All my new IDL has the proper formatting (as far as I know) as bz pointed out.

WRT the lazy creation, some code assumes that it will always autocreate. I added some code that wants no auto create because creation can be very expensive. It seems dumb to say "SetAutoCreate(false); GetSpellchecker(blah); SetAutoCreate(true);" It also seems dumb to add a new attribute MaybeGetInlineSpellchecker. Is there some consensus that this should work differently?

Comment 3

[timeless]

he added nsIEditorSpellCheck.CanSpellCheck

Comment 4

[Boris Zbarsky [:bzbarsky]]

See comment 1 item 2.

Comment 5

[Simon Fraser [no longer active]]

Heh, the first checkin of the idl file (with leading uppercase and methods rather than attributes) is mine, back in 1999.

Comment 6

[Brett Wilson]

I'm closing this bug in lieu of specific regressions or suggestions. If you find any actual problems or regressions with this feature, feel free to assign them to me.

Comment 7

[Simon Fraser [no longer active]]

This should not be closed.

Comment 8

[timeless]

so, bz and i actually came up w/ an approach a couple of days ago.