506041 - Part 2: Correct misspellings in source code

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Michael Kohler [:mkohler]]

Since bug 106386 gets bigger and bigger I decided to open a new bug for newer misspellings.

http://mxr.mozilla.org/mozilla-central/search?string=differnet

Comment 1

[Michael Kohler [:mkohler]]

to do by someone who knows how to do cvs patches:

http://mxr.mozilla.org/mozilla-central/search?string=loose&find=%2Fsecurity
http://mxr.mozilla.org/mozilla-central/search?string=mutliple&find=%2Fsecurity

Comment 2

[Alex Vincent [:WeirdAl]]

http://mxr.mozilla.org/mozilla1.9.1/search?string=ouput

Note a patch fixing this is not just comments-only - nsIAccessibleProvider.idl requires a change in a property name, so the UUID must be updated at the same time.

Comment 3

[Alex Vincent [:WeirdAl]]

http://mxr.mozilla.org/mozilla-central/search?string=implment

Comment 4

[Jesse Ruderman]

http://mxr.mozilla.org/mozilla-central/search?string=occured

Comment 5

[Alex Vincent [:WeirdAl]]

http://mxr.mozilla.org/mozilla-central/search?string=mainted

Comment 6

[Tanner M. Young [:tmyoung]]

http://mxr.mozilla.org/mozilla-central/search?string=+teh+&case=on&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

This gives the result of "teh" without the Arabic letter of the same name.

Comment 7

[Tanner M. Young [:tmyoung]]

http://mxr.mozilla.org/mozilla-central/search?string=untill&find=%2F&findi=&filter=&hitlimit=&tree=mozilla-central

Comment 8

[Tanner M. Young [:tmyoung]]

This is probably where you say, "I'm sorry I asked!"

I went through the list of 100 most misspelled words to see what I could come up with.  (I apologize if there are duplicates from the other comments.)

http://mxr.mozilla.org/mozilla-central/search?string=threshhold&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

http://mxr.mozilla.org/mozilla-central/search?string=supercede&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

http://mxr.mozilla.org/mozilla-central/search?string=seperate&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

http://mxr.mozilla.org/mozilla-central/search?string=+refrence&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

http://mxr.mozilla.org/mozilla-central/search?string=refered&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

http://mxr.mozilla.org/mozilla-central/search?string=recieve&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

http://mxr.mozilla.org/mozilla-central/search?string=priviledge&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

http://mxr.mozilla.org/mozilla-central/search?string=preceed&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

http://mxr.mozilla.org/mozilla-central/search?string=occurrance&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

http://mxr.mozilla.org/mozilla-central/search?string=noticable&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

http://mxr.mozilla.org/mozilla-central/search?string=judgement&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

http://mxr.mozilla.org/mozilla-central/search?string=independant&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

http://mxr.mozilla.org/mozilla-central/search?string=existance&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

http://mxr.mozilla.org/mozilla-central/search?string=changable&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

Comment 9

[Jeff Walden [:Waldo]]

http://mxr.mozilla.org/mozilla-central/search?string=phots

Comment 10

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

http://mxr.mozilla.org/mozilla-central/search?string=canoincail

Comment 11

[Jeff Walden [:Waldo]]

http://mxr.mozilla.org/mozilla-central/search?string=supress

Comment 12

[Jeff Walden [:Waldo]]

http://mxr.mozilla.org/mozilla-central/search?string=catestro

Comment 13

[Simon Montagu :smontagu]

http://mxr.mozilla.org/mozilla-central/search?string=elipse

Comment 14

[David Humphrey (:humph)]

57    * Initialize the reader with the edia stream.

s/edia/media/

http://mxr.mozilla.org/mozilla-central/source/content/media/ogg/nsChannelReader.h#57

Comment 15

[Jeff Walden [:Waldo]]

http://mxr.mozilla.org/mozilla-central/search?string=bahaviour

Comment 16

[[e-mail]]

http://mxr.mozilla.org/mozilla-central/search?string=existant

Comment 17

[Philip Chee]

Bug 241181

http://mxr.mozilla.org/mozilla-central/search?string=canceled

Comment 18

[Michael Kohler [:mkohler]]

I will create a first patch tomorrow.

Comment 19

[Michael Kohler [:mkohler]]

http://mxr.mozilla.org/mozilla-central/search?find=/&string=sepcify

Comment 20

[Michael Kohler [:mkohler]]

Attachment: accessible - misspellings

This and the following patches are splitted up per component (or a few components together). These patches change the following misspellings:

differnet -> different
loose -> lose
ouput -> output
implment -> implement
occured -> occurred
mainted -> maintained
teh -> the
untill -> until
seperated -> separated
refrence -> reference
refered -> referred
prefered -> preferred
priviledge -> privilege
preceeding -> preceding
preceeds -> preceds
occurrance -> occurrence
noticable -> noticeable
independant -> independent
changable -> changeable
existance -> existence
existant -> existent
elipse -> ellipse
bahaviour -> behaviour
supress -> suppress
supercede -> supersede
recieve -> receive

Comment 21

[Michael Kohler [:mkohler]]

Attachment: browser - misspellings

Comment 22

[Michael Kohler [:mkohler]]

Attachment: builds / caps /config - misspellings

Comment 23

[Michael Kohler [:mkohler]]

Attachment: content - misspellings

Comment 24

[Michael Kohler [:mkohler]]

Attachment: db / dom - misspellings

Comment 25

[Michael Kohler [:mkohler]]

Attachment: intl / ipc - misspellings

Comment 26

[Michael Kohler [:mkohler]]

Attachment: js - misspellings

Comment 27

[Michael Kohler [:mkohler]]

Attachment: layout/media - misspellings

Comment 28

[Michael Kohler [:mkohler]]

Attachment: modules/netwerk - misspellings

Comment 29

[Michael Kohler [:mkohler]]

Attachment: nsprpub/parser - misspellings

Comment 30

[Michael Kohler [:mkohler]]

Attachment: security - misspellings

Comment 31

[Michael Kohler [:mkohler]]

Attachment: testing/toolkit - misspellings

Comment 32

[Michael Kohler [:mkohler]]

Attachment: tools/view/widget/xpcom - misspellings

Comment 33

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

(In reply to comment #20)
> supercede -> supersede

What's wrong with supercede?

Comment 34

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

(In reply to comment #33)
> (In reply to comment #20)
> > supercede -> supersede
> 
> What's wrong with supercede?

Meh, ignore that.  I suppose it's technically an error, and it's certainly just bikeshedding :-)

Comment 35

[timeless]

Comment on attachment 441373 [details] [diff] [review]
db / dom - misspellings

please exclude sqlite, they have their own bug tracker. i've been meaning to send them a patch for spelling stuff (based on work i did on Symbian).

+          this.assertNull(transport.getHeader('X-Inexistent'));

this should be nonexistent

(the word shouldn't be hyphentated.)

otherwise everything else is ok.

Comment 36

[timeless]

Comment on attachment 441374 [details] [diff] [review]
intl / ipc - misspellings

chromium has an upstream, please send this patch to them.

Comment 37

[timeless]

Comment on attachment 441375 [details] [diff] [review]
js - misspellings

drop js/src/ctypes/libffi/ChangeLog

libffi should have an upstream, so a patch should be sent to it.

otherwise this looks fine.

Comment 38

[timeless]

Comment on attachment 441376 [details] [diff] [review]
layout/media - misspellings

libfishsound/liboggz/libtheora/libvorbis have upstreams, please file bugs to them - i think it's all the same upstream.

Comment 39

[timeless]

Comment on attachment 441377 [details] [diff] [review]
modules/netwerk - misspellings

freetype should have an upstream (although it's possibly defunct)

Comment 40

[timeless]

Comment on attachment 441378 [details] [diff] [review]
nsprpub/parser - misspellings

split nspr from parser, nspr requires a distinct committer.

occuring(sp)

Comment 41

[timeless]

Comment on attachment 441379 [details] [diff] [review]
security - misspellings

this looks good to me, but nelson has to do the commit (he can probably do the nspr commit too).

Comment 42

[timeless]

Comment on attachment 441382 [details] [diff] [review]
testing/toolkit - misspellings

google-breakpad has an upstream. please file it and then exclude. otherwise it looks good to me.

Comment 43

[Nelson Bolyard (seldom reads bugmail)]

Attachment: PSM part of above "security" patch

The above "security" patch includes patches to files in two Mozilla modules,
which are: PSM and NSS.  I have separated out the PSM part into this file.
Timeless, I'll let you review and/or commit this, if you wish.

I will convert the NSS part of the above security patch to a CVS diff and 
attach it to this bug, and then review it.

Comment 44

[Nelson Bolyard (seldom reads bugmail)]

Attachment: NSS part of the above "security" patch, converted to CVS diff (partially committed)

This is the NSS part of the above (now obsoleted) "security" patch.
I will review this.

Comment 45

[Nelson Bolyard (seldom reads bugmail)]

Comment on attachment 441404 [details] [diff] [review]
NSS part of the above "security" patch, converted to CVS diff (partially committed)

This patch is not "atomic".  It need not be accepted or rejected 
as a whole.  Parts of it can be accepted, and other parts rejected.
That's what I'm going to do.  I have accepted and committed parts 
of this patch (as noted below) and so will mark this r+.  But I will 
soon attach another patch which is the remnant of this patch that 
I did not accept, and I will explain why.

Bug 506041: Correct misspellings in source code comments 
Patch contributed by Michael Kohler <[e-mail]>, r=nelson

coreconf/jniregen.pl;                               new: 1.5;  previous: 1.4
coreconf/outofdate.pl;                              new: 1.3;  previous: 1.2
coreconf/rules.mk;                                  new: 1.83; previous: 1.82
nss/cmd/bltest/blapitest.c;                         new: 1.59; previous: 1.58
nss/cmd/crlutil/crlutil.c;                          new: 1.33; previous: 1.32
nss/cmd/lib/SECerrs.h;                              new: 1.20; previous: 1.19
nss/cmd/pk11mode/pk11mode.c;                        new: 1.28; previous: 1.27
nss/lib/certdb/alg1485.c;                           new: 1.40; previous: 1.39
nss/lib/ckfw/capi/crsa.c;                           new: 1.4;  previous: 1.3
nss/lib/ckfw/nssmkey/mobject.c;                     new: 1.5;  previous: 1.4
nss/lib/crmf/crmf.h;                                new: 1.4;  previous: 1.3
nss/lib/libpkix/pkix_pl_nss/system/pkix_pl_mutex.c; new: 1.7;  previous: 1.6
nss/lib/libpkix/pkix/top/pkix_build.c               new: 1.58; previous: 1.57
nss/lib/nss/nss.h;                                  new: 1.80; previous: 1.79
nss/lib/pk11wrap/pk11merge.c;                       new: 1.7;  previous: 1.6
nss/lib/pk11wrap/pk11nobj.c;                        new: 1.12; previous: 1.11
nss/lib/pk11wrap/pk11pub.h;                         new: 1.32; previous: 1.31
nss/lib/pk11wrap/pk11util.c;                        new: 1.58; previous: 1.57
nss/lib/pkcs12/p12d.c;                              new: 1.46; previous: 1.45
nss/lib/pkcs12/p12dec.c;                            new: 1.7;  previous: 1.6
nss/lib/pki/nsspkit.h;                              new: 1.7;  previous: 1.6
nss/lib/smime/cms.h;                                new: 1.23; previous: 1.22
nss/lib/smime/cmssiginfo.c;                         new: 1.32; previous: 1.31
nss/lib/ssl/sslcon.c;                               new: 1.40; previous: 1.39
nss/lib/ssl/sslgathr.c;                             new: 1.12; previous: 1.11
nss/lib/ssl/sslsock.c;                              new: 1.67; previous: 1.66
nss/lib/util/portreg.c;                             new: 1.7;  previous: 1.6
nss/lib/util/secdig.h;                              new: 1.9;  previous: 1.8
nss/lib/zlib/deflate.h;                             new: 1.6;  previous: 1.5
nss/tests/cert/cert.sh;                             new: 1.57; previous: 1.56
nss/tests/dbtests/dbtests.sh;                       new: 1.23; previous: 1.22
nss/tests/pkcs11/netscape/trivial/trivial.c;        new: 1.5;  previous: 1.4

Comment 46

[Nelson Bolyard (seldom reads bugmail)]

Attachment: Unaccepted remnant of above NSS patch

This patch serves to document the part of the above NSS patch that cannot be
checked into the NSS trunk in its present form.  There are three different 
reasons:

1) We cannot change the names of members of structures declared in public 
header files once those have been released.  So, we cannot rename the members
from "priviledge" to "privilege", much as we might wish.  What we CAN do is 
add a #define to add a synonym with the correct spelling.  

2) Some of the files patched here are part of a body of code that has been 
certified by the US government as cryptographically sound, and *ANY* changes
to those files (even to comments) invalidate that certification, necessitating
a new certification.  So, we make those changes on a branch which we hold 
until the next time we plan to go through a certification.  We JUST finished
a certification earlier this month, so it will be some time before we do 
another.  This applies to the files in "softoken".

3. The sqlite3 file is a file we take from an upstream source.  We strongly 
prefer to make only the absolutely necessary changes to the source as received from that upstream, and keep all changes out unless they are absolutely necessary for the correct execution of the code, which clearly comments do 
not affect.  I'd suggest you send your corrections for the sqlite3 code to 
the upstream source.  If they're part of the next release we take from that
upstream source, then we'll get those changes that way.

I suggest you do as follows:
1) submit a new patch attached to this bug, to address the "priviledge" issue with a synonym, and ask me to review it.
2) File a new bug against product NSS about the proposed changes for the 
files in softoken.  We'll get that into the review process for that code.
3) send your sqlite3 changes to the sqlite3 upstream, whereever that is.

Comment 47

[Nelson Bolyard (seldom reads bugmail)]

Attachment: NSS patch for the alternative priviledge->privilege fix (committed)

This is the alternative I proposed above to fix the misspelled priviledge 
members without actually changing the names of the members in the struct.

Michael, does this satisfy you?
If so, Timeless, please review & I'll give credit for this patch to Michael.

Comment 48

[timeless]

Comment on attachment 441423 [details] [diff] [review]
NSS patch for the alternative priviledge->privilege fix (committed)

you should probably add a comment to keythi.h explaining the defines.

Comment 49

[Michael Kohler [:mkohler]]

Nelson, sure it satisfies me, well done!  And please feel free to take credits for yourself.

I will update the other patches in a few hours.

Comment 50

[Michael Kohler [:mkohler]]

Attachment: dom v2 (without db now)

Comment 51

[Michael Kohler [:mkohler]]

Attachment: intl v2 (without ipc now)

Comment 52

[Michael Kohler [:mkohler]]

Attachment: js v2

Comment 53

[Michael Kohler [:mkohler]]

Attachment: layout/media v2

Comment 54

[Michael Kohler [:mkohler]]

Attachment: parser v2 (without nsprpub)

Comment 55

[Michael Kohler [:mkohler]]

Attachment: nsprpub v2

Comment 56

[Michael Kohler [:mkohler]]

Attachment: parser v2.1

uploaded too early, I'm sorry for bugspam.

Comment 57

[Michael Kohler [:mkohler]]

Attachment: modules/netwerk v2

Comment 58

[Michael Kohler [:mkohler]]

Attachment: testing/toolkit v2

Comment 59

[Michael Kohler [:mkohler]]

Attachment: other-licenses - misspellings

Found another patch which you've not yet seen. Did I upload every updated patch of the r- patches? I think I didn't forget something.

Comment 60

[Michael Kohler [:mkohler]]

to do in a future patch:

http://mxr.mozilla.org/mozilla-central/search?string=judgement -> judgment
http://mxr.mozilla.org/mozilla-central/search?string=threshhold -> threshold?
http://mxr.mozilla.org/mozilla-central/search?string=sepcify -> specifiy
http://mxr.mozilla.org/mozilla-central/search?find=/&string=non-existent -> nonexistent

Comment 61

[timeless]

Comment on attachment 441571 [details] [diff] [review]
dom v2 (without db now)

didn't we agree on nonexistent instead of non-existent? :(

X-nonexistent
should be
X-Nonexistent
to match the style.

Comment 62

[timeless]

Comment on attachment 441574 [details] [diff] [review]
js v2

inexistent => nonexistent
non-existent => nonexistent

Comment 63

[timeless]

Comment on attachment 441576 [details] [diff] [review]
layout/media v2

:(, exclude libogg

Non-existent => Nonexistent

Comment 64

[timeless]

Comment on attachment 441579 [details] [diff] [review]
nsprpub v2

nonexistence

nelson: can you commit this?

Comment 65

[timeless]

Comment on attachment 441584 [details] [diff] [review]
modules/netwerk v2

sorry

libimg/png is a third party library http://sourceforge.net/tracker/?atid=105624&group_id=5624&func=browse

modules/zlib is http://www.gzip.org/zlib/zlib_faq.html#faq08

everything else is fine.

Comment 66

[timeless]

Comment on attachment 441588 [details] [diff] [review]
other-licenses - misspellings

http://sourceforge.net/tracker/?atid=373085&group_id=22049&func=browse

Comment 67

[Nelson Bolyard (seldom reads bugmail)]

Comment on attachment 441579 [details] [diff] [review]
nsprpub v2

I'm giving this an r+ in principle.  But the master source repository for NSPR is CVS, so I'll know for sure if this applies cleanly when I convert this patch to a CVS patch.

Comment 68

[Jeff Walden [:Waldo]]

"judgement" actually is a valid, if less-common, spelling, just so you know.  I don't mind the change, but quite strictly speaking it's unnecessary.

Comment 69

[Michael Kohler [:mkohler]]

(In reply to comment #61)
> (From update of attachment 441571 [details] [diff] [review])
> didn't we agree on nonexistent instead of non-existent? :(
> 
> X-nonexistent
> should be
> X-Nonexistent
> to match the style.

missed that, sorry.

(In reply to comment #65)
> (From update of attachment 441584 [details] [diff] [review])
> sorry
> 
> libimg/png is a third party library
> http://sourceforge.net/tracker/?atid=105624&group_id=5624&func=browse
> 
> modules/zlib is http://www.gzip.org/zlib/zlib_faq.html#faq08
> 
> everything else is fine.

I'll file bugs when I get home this evening (CET). Also I'll update the other patches. I thought I'd do the change from non-existent to nonexistent in a future patch since this is used over all components. But I'll update these patches to nonexistent, sorry for this misunderstanding.

Thanks for using that much time to review, timeless. :)

Nelson, could you please change the nit from comment 64 when converting to a CVS patch?

Comment 70

[Michael Kohler [:mkohler]]

Attachment: js v3

Comment 71

[Michael Kohler [:mkohler]]

Attachment: layout/media v3

Comment 72

[timeless]

Comment on attachment 441708 [details] [diff] [review]
js v3

for components which are hard to change (nss, nspr), it makes sense to roll non-existent => nonexistent changes together. for this patch, it's ok to get it later.

Comment 73

[timeless]

Comment on attachment 441710 [details] [diff] [review]
layout/media v3

sorry... liboggplay is part of the same xiph stuff

https://trac.xiph.org/

Comment 74

[Michael Kohler [:mkohler]]

I will file bugs with patches in the upstreams' tracker. Right now I can't do it because I'm heading to school.

BTW, is there any list of third-party-libraries I could use for future patches to save time for you, timeless?

Comment 75

[Nelson Bolyard (seldom reads bugmail)]

Do you have a list of misspelled words that you're planning to correct but
have not yet corrected?  If so, would you put it in a comment here?  
I'd rather fix the entire list all at once for NSS and NSPR, using the 
CVS trunk as the basis.

Comment 76

[Michael Kohler [:mkohler]]

(In reply to comment #75)
> Do you have a list of misspelled words that you're planning to correct but
> have not yet corrected?  If so, would you put it in a comment here?  
> I'd rather fix the entire list all at once for NSS and NSPR, using the 
> CVS trunk as the basis.

see comment 60. Is it easier for you to convert a hg patch to a cvs patch or do you do the patch yourself?

Comment 77

[Michael Kohler [:mkohler]]

Attachment: dom v3

Comment 78

[Michael Kohler [:mkohler]]

Attachment: modules/netwerk v3

Comment 79

[Michael Kohler [:mkohler]]

Attachment: layout v4

This patch was "layout/media v3" before the update, now there is no code from media in there any more.

Comment 80

[timeless]

Comment on attachment 441842 [details] [diff] [review]
dom v3

-* Seperated internal stuff from get() into set()
+* separated internal stuff from get() into set()

You case folded :(

Sorry, but i believe this is the last round of minuses :)

Comment 81

[Michael Kohler [:mkohler]]

Attachment: dom v4

oh, missed that.

Comment 82

[Michael Kohler [:mkohler]]

Attachment: final misspellings patch [fixes described in comment 20]

Final review. I really hope I didn't screw up doing this patch (at least it applies).

The part from /content/media/ogg/nsOggDecoder.h I corrected has been moved to /content/media/ogg/nsOggPlayStateMachine.h in bug 560708. I have fixed it in the nsOggPlayStateMachine.h file.

For all the fixes please see comment 20.

Comment 83

[timeless]

Comment on attachment 442554 [details] [diff] [review]
final misspellings patch [fixes described in comment 20]

i was hoping you'd fold in the nonexistent collapsing change too, but oh well.

+    this.assertElementsMatch(Selector.matchElements($('fixtures').descendants(), '.inexistent, a.internal'), '#link_1', '#link_2');

should be nonexistent

dom/tests/mochitest/dom-level2-core

this somehow came from something in w3. We should be sending it upstream. konigsberg has some w3 stuff, i'm not sure if it has an index of the right thing for this.

http://mxr-test.konigsberg.mozilla.org/dev-w3/source/2001/DOM-Test-Suite/tests/level2/html/

seems close. Sorry for not spotting it earlier. Can you split the mochitest changes that seem to be w3 derived into its own patch and try to offer it to w3.org? hsivonen should be able to help.

+    // Check that annotation to a not existent page has been removed

an annotation/non-existing

+    // Add a bookmark using a not existent keyword

non-existing

actually for that whole file that's generally the right replacement.

sorry about this. you might as well make the full set of changes here. 

i'm stuck reading the whole thing start to finish anyway (and it's my fault, since i should have caught all of this much earlier)

Comment 84

[Nelson Bolyard (seldom reads bugmail)]

regarding comment 83, 
surely "nonexistent" is preferable to "non-existing", no?

Comment 85

[Nelson Bolyard (seldom reads bugmail)]

Comment on attachment 441423 [details] [diff] [review]
NSS patch for the alternative priviledge->privilege fix (committed)

Checking in certhigh/certvfy.c; new revision: 1.70; previous revision: 1.69
Checking in cryptohi/keythi.h;  new revision: 1.12; previous revision: 1.11
Checking in cryptohi/seckey.c;  new revision: 1.53; previous revision: 1.52

Comment 86

[Nelson Bolyard (seldom reads bugmail)]

In comment 69, Timeless wrote:
> Nelson, could you please change the nit from comment 64 when converting 
> to a CVS patch?

That comment (64) was too terse for me.  Please tell me
- what file
- old string
- new string
and I'll do it

Comment 87

[Michael Kohler [:mkohler]]

(In reply to comment #83)
> (From update of attachment 442554 [details] [diff] [review])
> i was hoping you'd fold in the nonexistent collapsing change too, but oh well.
>
> +   
> this.assertElementsMatch(Selector.matchElements($('fixtures').descendants(),
> '.inexistent, a.internal'), '#link_1', '#link_2');
> 
> should be nonexistent

oh, yes. And I will change all non-existent to nonexistent in this patch if they are in a line that is edited anyway. The other cases I'll replace in another patch next week.You wrote in this comment "nonexistent" and "non-existent", which should I use? I'd like to use one in every file and not component-specific.


> dom/tests/mochitest/dom-level2-core
> 
> this somehow came from something in w3. We should be sending it upstream.
> konigsberg has some w3 stuff, i'm not sure if it has an index of the right
> thing for this.
> 
> http://mxr-test.konigsberg.mozilla.org/dev-w3/source/2001/DOM-Test-Suite/tests/level2/html/
> 
> seems close. Sorry for not spotting it earlier. Can you split the mochitest
> changes that seem to be w3 derived into its own patch and try to offer it to
> w3.org? hsivonen should be able to help.

I will check with him and make a new patch on Monday.

Comment 88

[Michael Kohler [:mkohler]]

(In reply to comment #86)
> In comment 69, Timeless wrote:
> > Nelson, could you please change the nit from comment 64 when converting 
> > to a CVS patch?
> 
> That comment (64) was too terse for me.  Please tell me
> - what file
> - old string
> - new string
> and I'll do it

nevermind ;) I thought the "occuring" was in NSPR, but it was in the parser, so it was my assignment. Thanks anyway.

Comment 89

[timeless]

http://mxr.mozilla.org/mozilla-central/search?string=associtation

Comment 90

[Philip Chee]

philor: Why aren't your changesets part of this bug?

http://hg.mozilla.org/mozilla-central/rev/a7cbd999c2f5
http://hg.mozilla.org/mozilla-central/rev/67b93e5e8f5d
http://hg.mozilla.org/mozilla-central/rev/6ffa228a5897

Perhaps you should coordinate with Michael so you don't bitrot him.

Comment 91

[Boris Zbarsky [:bzbarsky]]

I don't think we should be messing with the third-party-imported mochitests at all.  That includes all of the w3c test suites, the jquery stuff, etc, etc.

Comment 92

[Michael Kohler [:mkohler]]

Attachment: misspellings

* changed to nits from comment 83
* deleted already checked in changes (see comment 90)
* deleted all changes to dom/tests/mochitest/ (see comment 91)

Comment 93

[Michael Kohler [:mkohler]]

Attachment: misspellings part 2

this changes:

* judgement -> judgment
* associtation -> association
* sepcify/sepcifying -> specify/specifying

Comment 94

[Michael Kohler [:mkohler]]

Attachment: judgement->judgment in nsprpub

Comment 95

[timeless]

Comment on attachment 443782 [details] [diff] [review]
misspellings part 2

drop:
toolkit/components/feeds/test/xml/rss1/full_feed_not_bozo.xml

it's based on a real blog, we can't really revise history.

Comment 96

[Jeff Walden [:Waldo]]

http://mxr.mozilla.org/mozilla-central/search?string=exlude

Comment 97

[Michael Kohler [:mkohler]]

Attachment: misspellings part 1 [checked in]

Comment 98

[Michael Kohler [:mkohler]]

Attachment: misspellings part 2 [checked in]

Comment 99

[Michael Kohler [:mkohler]]

Comment on attachment 444690 [details] [diff] [review]
misspellings part 1 [checked in]

http://hg.mozilla.org/mozilla-central/rev/7ef2a4732afe

Comment 100

[Michael Kohler [:mkohler]]

Comment on attachment 444691 [details] [diff] [review]
misspellings part 2 [checked in]

http://hg.mozilla.org/mozilla-central/rev/11ef0c724251

Comment 101

[timeless]

http://mxr.mozilla.org/mozilla-central/search?string=paramter

Comment 102

[Michael Kohler [:mkohler]]

http://mxr.mozilla.org/mozilla-central/search?string=paltform
http://mxr.mozilla.org/mozilla-central/search?string=shoud

Comment 103

[Michael Kohler [:mkohler]]

Attachment: Misspellings Part 3

shoud -> should
paltform -> platform
paramter -> parameter
exlude -> exclude

Comment 104

[Michael Kohler [:mkohler]]

Attachment: NSS misspellings

Changes see comment above.

Comment 105

[timeless]

Comment on attachment 452291 [details] [diff] [review]
Misspellings Part 3

>+++ b/uriloader/exthandler/win/nsOSHelperAppService.cpp	Fri >+// Strip a handler command string of it's quotes and parameters.

s/it's/its/

Comment 106

[Michael Kohler [:mkohler]]

Attachment: Misspellings Part 3 v2

Corrected: s/it's/its

Also pushed to try server as http://hg.mozilla.org/try/rev/f57476abee09 to be sure I don't break anything with this. BTW, is this try-checkin correct or should I do something different next time? (was my first push)

Comment 107

[timeless]

Comment on attachment 456000 [details] [diff] [review]
Misspellings Part 3 v2

sorry, please exclude libvpx. I'm not sure where the upstream is, but i'm sure there is one. you can split out a bug to video/audio and get one of the video/audio people to deal w/ it, or find the upstream directly.

Comment 108

[Michael Kohler [:mkohler]]

Attachment: Patch 3.3

(In reply to comment #107)
> (From update of attachment 456000 [details] [diff] [review])
> sorry, please exclude libvpx. I'm not sure where the upstream is, but i'm sure
> there is one. you can split out a bug to video/audio and get one of the
> video/audio people to deal w/ it, or find the upstream directly.

As far as I see the upstream lives at http://www.webmproject.org/code/ . Filed a bug there with the patch attached.

Comment 109

[Michael Kohler [:mkohler]]

Please report new misspellings in bug 577266 . If they are only in nspr or nss present, please report them to bug 577267 or bug 577268. Thanks.

Comment 110

[Michael Kohler [:mkohler]]

Pushed as http://hg.mozilla.org/mozilla-central/rev/a9356e0479d1 .

Comment 111

[Michael Kohler [:mkohler]]

libvpx fixed: http://review.webmproject.org/#change,316