We have a lot of duplicate atoms (and string buffers, etc.) that could be shared between the HTML5 parser and nsGkAtomList. We should share those atoms, which would save ~50-60K of libxul size on a 64-bit platform, and make startup a wee bit faster by not having to intern everything.

Comment on attachment 8859984 [details] [diff] [review] Translator changes Review of attachment 8859984 [details] [diff] [review]: ----------------------------------------------------------------- ::: translator-src/nu/validator/htmlparser/cpptranslate/CppTypes.java @@ +146,5 @@ > + String line; > + while ((line = atomReader.readLine()) != null) { > + manualAtoms.append(line); > + manualAtoms.append('\n'); > + if (line.startsWith("// BEGIN GENERATED")) { The list is generated atoms is long and the "// BEGIN GENERATED" warning block is far from the end of the file so it seems likely that someone will append to the middle or end of the file when adding a new atom. At the least we should add a comment at the end, something like "// END GENERATED ATOMS, DO NOT ADD CODE BELOW THIS LINE" and maybe also add a comment after each generated atoms saying "// ATOM GENERATED BY HTML PARSER TRANSLATOR" to make it even more clear so that we don't get atoms accidentally added to the middle of the list. Alternatively, I think it would be even better if we generated the parser atoms to another file that we include in nsGkAtomList.h so then we don't have to worry about atoms manually added in the wrong place.

Comment on attachment 8859927 [details] Bug 1269490 - Merge nsHtml5Atoms into nsGkAtoms. https://reviewboard.mozilla.org/r/131988/#review138668

Comment on attachment 8859927 [details] Bug 1269490 - Merge nsHtml5Atoms into nsGkAtoms. https://reviewboard.mozilla.org/r/131988/#review138980

Pushed by hsivonen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/355a23c6c2bd Merge nsHtml5Atoms into nsGkAtoms. r=wchen

https://hg.mozilla.org/projects/htmlparser/rev/9e783e21265f51e3ac96feca70dd025f03c3cf8e Mozilla bug 1269490 - Merge nsHtml5Atoms into nsGkAtoms. r=wchen.