--- books/www/corrections.html 2002/12/10 16:22:55 1.39
+++ books/www/corrections.html 2003/03/29 10:07:58 1.110
@@ -1,4 +1,9 @@
-Corrections
+
+
+
Corrections
This page contains a list of corrections to be made in the next printing of
the book. If you have suggestions for changes or find errors in the text,
@@ -6,132 +11,398 @@ feel free open a bug
section on the chapter pages, or send a message to the reviewers@mozdev.org mailing list.
+
Note: All errors in the text and examples are shown in red and all corrections are shown in blue.
-
+
Preface
-- On page xiv: 'http://www.oreilly.com/mozilla/' has been changed
-to 'http://www.oreilly.com/catalog/mozilla/'
+ - On page ix: 'other browsers such as Galeon and Chimera' has been changed to 'other browsers
+such as Galeon and Camino'
+
+ - On page xiv: 'http://www.oreilly.com/mozilla/' has been changed
+to 'http://www.oreilly.com/catalog/mozilla/'
+
+ - On page xvii: 'David McNamara, and Mark Hammond' has been changed to 'David
+McNamara, Mark Hammond, Gary Frederick, and Andreas Halter'
+
+ - On page xvii: 'David Hyatt and Mike Pinkerton for the Chimera browser' has been changed to 'David Hyatt and Mike Pinkerton for the Camino browser'
+
+ - On page xvii: 'including Kim Steckler, Rachel Oeschger, and Priscilla, Dylan, and Devon Collins' has been
+changed to 'including Kim Steckler, Rachel Oeschger, Marjana Molan, Lia Molan King, and Priscilla, Dylan, and Devon
+Collins'
-
+
Chapter 1
-- On page 7: 'applications web built using XPFE' has been
-changed to 'applications built using XPFE'
+ - On page 2: In sidebar 'like Chimera (an ultra-fast browser that works only on Mac OS X)' has been changed to
+'like Camino (an ultra-fast browser that works only on Mac OS X)'
+
+ - On page 7: In sidebar 'Another project, call m/b (short for mozilla/browser), addresses shortcomings' has
+been changed to 'The Phoenix project (http://www.mozilla.org/projects/phoenix/) also addresses
+shortcomings'
+
+ - On page 7: In sidebar 'Some examples include Chimera (http://chimera.mozdev.org)' has been changed to 'Some examples include Camino (http://www.mozilla.org/projects/camino/)'
+
+ - On page 7: 'applications web built using XPFE' has been
+changed to 'applications built using XPFE'
-
+
Chapter 2
-- On page 12: '<window
+
- On page 10: Added a new sentence to the end of the 2nd paragraph of the xFly examples sidebar: 'An installable version
+of the complete xFly application can be found at http://xfly.mozdev.org.'
+
+ - On page 12: '<window
xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
-/>' has been changed to '<window
-xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">'
+/>' has been changed to '<window
+xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">'
+
+ - On page 14: In first code sample from Windows features section, 'xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">' has been changed to 'xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"'
- - On page 14: Removed an extra '>' from example code
+ - On page 17: 'selecting File > Open File > from' has been
+changed to 'selecting File > Open File from'
- - On page 17: 'selecting File > Open File > from' has been
-changed to 'selecting File > Open File from'
+ - On page 20: 'CenterWindoOnScreen( ) function' has been changed
+to 'CenterWindowOnScreen( ) function'
- - On page 20: 'CenterWindoOnScreen( ) function' has been changed
-to 'CenterWindowOnScreen( ) function'
+ - On page 25: 'Under that directory, create the three new directories, content, locale, and skin, as
+shown in Figure 2-5.' has been changed to 'Under that directory, create the three new directories, content,
+locale, and skin, as shown in Figure 2-5. The locale directory will have the default 'en-US' language pack
+structure.'
+
+ - On page 25: There is 1 error in Figure 2-5 ...
+
+1) The locale folder should have an en-US subfolder.
+
+Here is a correct image (Click for bigger version):
+
+
+
+ - On page 31: In Example 2-14 'locale/' has been changed to 'locale/en-US/'
+
+ - On page 32: In Example 2-15 'locale,install,url,resource:/chrome/xfly/locale/' has been changed to 'locale,install,url,resource:/chrome/xfly/locale/en-US/'
-
+
Chapter 3
-- On page 40: '<toolbarbutton id="printBtn" label="Open"
-oncommand="doPrint( );" />' has been changed to '<toolbarbutton
-id="printBtn" label="Print" oncommand="doPrint( );" >'
+ - On page 34: 'We've packed a lot information in this chapter and it be a useful reference' has been changed
+to 'We've packed a lot of information in this chapter and it will be a useful reference'
+
+ - On page 35: 'secondLevelNodes[I] = docEl.childNodes[i];' has been changed to 'secondLevelNodes[I] = docEl.childNodes[I];'
+
+ - On page 38: In Example 3-3 the namespace declarations have been changed to (note the addition of quotes and the removal of italics)
+
+<![CDATA[
+ xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
+ xmlns:nc="http://home.netscape.com/NC-rdf#">
+]]>
+
+
- On page 38: In Example 3-3 '<image src="page1.png">' has been changed to '<image
+src="page1.png"/>'
+
+ - On page 40: '<toolbarbutton id="printBtn" label="Open"
+oncommand="doPrint( );" />' has been changed to '<toolbarbutton
+id="printBtn" label="Print" oncommand="doPrint( );" >'
+
+ - On page 51: '<decription>The mozdev.org site [...] your own development project</decription>' has been
+changed to '<description>The mozdev.org site [...] your own development project</description>'
+
+ - On page 54: 'Clicking on the box sets the clicked attribute' has been
+changed to 'Clicking on the box sets the checked attribute'
+
+ - On pages 58 and 59: Example 3-15 has been changed to (note the addition of spaces to seperate values)
+
+<![CDATA[
+<?xml version="1.0"?>
+<window id="hello-goodbye"
+ title="Hello Goodbye"
+ xmlns:html="http://www.w3.org/1999/xhtml"
+ xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
+ style="min-width:100px;min-height:100px;background-color:white;">
+ <broadcasterset id="broadcasterset">
+ <broadcaster id="cmd_hello" oncommand="alert('Hello There!');"/>
+ </broadcasterset>
+ <keyset id="keyset">
+ <key id="key_h" key="H" observes="cmd_hello" modifiers="accel,shift" />
+ <key id="key_g" key="G" command="cmd_goodbye" modifiers="accel,shift" />
+ </keyset>
+ <commandset id="commandset">
+ <command id="cmd_goodbye" oncommand="alert('Goodbye!');" />
+ </commandset>
+ <spacer flex="1"/>
+ <label value="hello/goodbye"/>
+ <textbox value="type ctl+shft+h"/>
+ <textbox value="type ctl+shft+g"/>
+ <spacer flex="1"/>
+</window>
+]]>
+
+
- On page 63: 'But there are also CSS versions of these properties that
+use the prefix box-pack becomes box-pack when it's defined in CSS, for example.' has been
+changed to 'But there are also CSS versions of these properties that
+use the prefix box-. pack becomes box-pack when it's defined in CSS, for example.'
+
+
+
+
+
+Chapter 4
+
+
+
+- On page 88: 'these are selectors are identical' has been changed to 'these selectors are
+identical'
+
+ - On page 109: 'The chome URL that the -moz-binding property points to' has been changed to 'The chrome URL that the -moz-binding property points to'
-
+
+
Chapter 5
- - (p 113)
- <![CDATA[
- Changed footnote link to
- http://developer.netscape.com/docs/manuals/index.html?content=javascript.html
- ]]>
-
- - (p 120)
- <![CDATA[
- dump(l+'\n');
- has been changed to
- dump(len+'\n');
- ]]>
-
- - (p 137)
- <![CDATA[
- var connection = Components.classes ["@mozilla.org/network/ldap-connection;1"; // create an object from the nsILDAPConnection interface;
- has been changed to
- var connection = Components.classes ["@mozilla.org/network/ldap-connection;1"]; // create an object from the nsILDAPConnection interface;
- ]]>
-
- - (p 140)
- <![CDATA[
- Changed load(`chrome/jslib/jslib.js');
- To load('chrome/jslib/jslib.js');
- ]]>
-
+
+- On page 113: Javascript 1.5 reference link in footnote has been changed from
+'http://developer.netscape.com/docs/manuals/js/core/jsref15/contents.html' to
+'http://developer.netscape.com/docs/manuals/index.html?content=javascript.html'
+
+
+- On page 120: '</script.' has been changed to '</script>'
+
+
+- On page 120: 'dump(l+'\n');' has been changed to 'dump(len+'\n');'
+
+
+- On page 123: Example 5-3
+<![CDATA[
+for (var i=0; i<chex.length; i++)
+ chex[i].setAttributte("checked", "false");
+for (var i=0; i<unchex.length; i++)
+ unchex[i].setAttributte("checked", "true");
+]]>
+has been changed to
+<![CDATA[
+for (var i=0; i<chex.length; i++)
+ chex[i].setAttribute("disabled", "false");
+for (var i=0; i<unchex.length; i++)
+ unchex[i].setAttribute("disabled", "true");
+]]>
+
+
+
+- On page 135: Example 5-11 has been changed to
+<![CDATA[
+ <head>
+ <title>Sound Service Play Example</title>
+ <script>
+ function play() {
+ netscape.security.PrivilegeManager.enablePrivilege("UniversalXPConnect");
+ var sample = Components.classes["@mozilla.org/sound;1"].createInstance();
+ sample = sample.QueryInterface(Components.interfaces.nsISound);
+ const SND_NETWORK_STD_CID = "@mozilla.org/network/standard-url;1";
+ const SND_I_URL = "nsIURL";
+ const SND_URL = new C.Constructor(SND_NETWORK_STD_CID, SND_I_URL);
+ var url = new SND_URL();
+ url.spec = 'http://jslib.mozdev.org/test.wav';
+ sample.play(url);
+ }
+ </script>
+ </head>
+
+ <form name="form">
+ <input type="button" value="Play Sound" onclick="play();">
+ </form>
+]]>
+
+
- On page 137: 'var connection = Components.classes ["@mozilla.org/network/ldap-connection;1";'
+ has been changed to
+'var connection = Components.classes ["@mozilla.org/network/ldap-connection;1"];'
+
+
+- On page 140: 'js> load(`chrome/jslib/jslib.js');' has been changed to 'js>
+load('chrome/jslib/jslib.js');'
+
+
+
+
+
+Chapter 6
+
+
+
+- On page 145: Example 6-1 has been changed to
+
+<![CDATA[
+var myFile = "xFly.jar";
+initInstall( // initialize the installation
+ "Install xFly", // display name of installation
+ "xFly", // package name
+ "0.0.1", // version of install
+ 1); // flags - an optional argument,
+ // reserved for future use
+f = getFolder("Chrome"); // specify a target directory
+setPackageFolder(f);
+addFile(myFile); // add software to the installation
+registerChrome(
+ PACKAGE | DELAYED_CHROME, // chrome switch (i.e., type)
+ getFolder("Chrome","xFly.jar"), // destination of package
+ "content/xFly/"); // location of manifest in package
+if (0 == getLastError( )) // if there have been no errors:
+ performInstall( ); // install "xfly.jar"
+else // otherwise
+ cancelInstall( ); // cancel the installation.
+]]>
+
-
-Chapter 8
+
+
Chapter 7
-- (p 211) Root Interfaces
- Changed 2nd sentence QueryInterace to QueryInterface.
+
+ - On page 181: There are 3 errors in Figure 7-2 ...
+
+1) In the XUL box, 'klabel="Input Field"' should be 'label="Input Field"'
+
+2) In the XBL box, '<binding id+"inputfield">' should be '<binding id="inputfield">'
+
+3) In the XBL box, '<xul:label xbl:inherits="value+label"?>' should be '<xul:label xbl:inherits="value=label"/>'
+
+Here is a correct image (Click for bigger version):
+
+
+
-- (p 222) Example 8-4
-
mName: "a default value" should be mName: "a default value",
+
+
+
+
+
+
+Chapter 8
+
+
+
+- On page 211: In 2nd sentence of paragraph 'QueryInterace' has been
+changed to 'QueryInterface'
+
+- On page 222: In Example 8-4 has been changed to:
+
+<![CDATA[
+function SimpleComponent(){}
+
+SimpleComponent.prototype = {
+
+ get yourName() { return this.mName; },
+ set yourName(aName) { return this.mName = aName; },
+
+ write: function () { dump("Hello " + this.mName + "\n"); },
+ change: function (aValue) { this.mName = aValue; },
+ mName: "a default value",
+
+ QueryInterface: function (iid) {
+ if (!iid.equals(Components.interfaces.nsISimple)
+ && !iid.equals(Components.interfaces.nsISupports))
+ {
+ throw Components.results.NS_ERROR_NO_INTERFACE;
+ }
+ return this;
+ }
+}
+
+var Module = {
+ firstTime: true,
+
+ registerSelf: function (compMgr, fileSpec, location, type) {
+ if (this.firstTime) {
+ dump("*** Deferring registration of simple JS components\n");
+ this.firstTime = false;
+ throw Components.results.NS_ERROR_FACTORY_REGISTER_AGAIN;
+ }
+ debug("*** Registering sample JS components\n");
+ compMgr =
+compMgr.QueryInterface(Components.interfaces.nsIComponentRegistrar);
+ compMgr.registerFactoryLocation(this.myCID,
+ "Simple JS Component",
+ this.myProgID,
+ fileSpec,
+ location,
+ type);
+ },
+
+ getClassObject : function (compMgr, cid, iid) {
+ if (!cid.equals(this.myCID))
+ throw Components.results.NS_ERROR_NO_INTERFACE
+ if (!iid.equals(Components.interfaces.nsIFactory))
+ throw Components.results.NS_ERROR_NOT_IMPLEMENTED;
+ return this.myFactory;
+ },
+
+ myCID: Components.ID("{98aa9afd-8b08-415b-91ed-01916a130d16}"),
+ myProgID: "@mozilla.org/js_simple_component;1",
+
+ myFactory: {
+ createInstance: function (outer, iid) {
+ dump("CI: " + iid + "\n");
+ if (outer != null)
+ throw Components.results.NS_ERROR_NO_AGGREGATION;
+ return (new SimpleComponent()).QueryInterface(iid);
+ }
+ },
+
+ canUnload: function(compMgr) {
+ dump("****** Unloading: Simple JS component! ****** \n");
+ return true;
+ }
+}; // END Module
+
+function NSGetModule(compMgr, fileSpec) { return Module; }
+]]>
-- (p 222) Example 8-4
- <![CDATA[
- this.myCID,
- "My JS Component",
- this.myProgID, fileSpec,
- location, true, true,
- type);
- ]]>
- Should be
- <![CDATA[
- compMgr.registerComponentWithType(
- this.myCID,
- "My JS Component",
- this.myProgID, fileSpec,
- location, true, true,
- type);
- ]]>
-
-- (p 222, p 223) Example 8-4
- <![CDATA[
- Removed double lines
- dump(" ***** Registering: Simple JS component! ****\n");
- and
- if (outer != null)
- ]]>
+
+
- On page 237: Example 8-13, line 9
+<![CDATA[
+ NS_INIT_REFCNT( ); // has been depricated use NS_INIT_ISUPPORTS()
+ mValue = PL_strdup("default value");
+]]>
+has been changed to
+<![CDATA[
+ // NS_INIT_REFCNT( ); // has been depricated use NS_INIT_ISUPPORTS()
+ NS_INIT_ISUPPORTS();
+ mValue = PL_strdup("default value");
+]]>
+
-- (p 239) Example 8-15 has been changed to
-<![CDATA[
+<li>On page 239: Example 8-15 has been changed to <br/></li>
+
+<xmp class="fixed">
DEPTH = ../../..
topsrcdir = ../../..
srcdir = .
@@ -169,55 +440,259 @@ EXTRA_DSO_LDOPTS += $(MOZ_COMPONENT_LIBS
install:: $(TARGETS)
]]>
-
+
+Chapter 9
+
+
+
+
Chapter 10
-- On page 297: 'http://www.jslib.mozdev.org' has been changed
-to 'http://jslib.mozdev.org'
+
+- One page 268, In Example 10-1 '<rdf:Description about="http://my.jar-of-flies.com">' has been
+changed to '<rdf:Description about="urn:root">'
+
+ - On page 279: 'The URI is also accessible as a datasource property:' has been
+changed to 'The URI, such as rdf:someName, is also accessible as a datasource
+property:'
+
+ - On page 280: In Table 10-3 'Mostly used for retrieving, datasources, resources, and literals.' has been
+changed to 'Mostly used for retrieving datasources, resources, and literals.'
+
+ - On page 288:
+
+<![CDATA[
+'containerUtils = Components.classes['@mozilla.org/rdf/container-utils;1'
+getService(Components.interfaces.nsIRDFContainerUtils);'
+]]>
+
+has been changed to:
+
+<![CDATA[
+'containerUtils = Components.classes['@mozilla.org/rdf/container-utils;1'].
+getService(Components.interfaces.nsIRDFContainerUtils);'
+]]>
+
+
- On page 297: 'http://www.jslib.mozdev.org' has been changed to 'http://jslib.mozdev.org'
+
+ - On page 297, Example 10-14: '// if(del)gRDF.removeSeq('types',false);' has been deleted. Not
+needed in this example.
+
+
+
+
+
+Chapter 11
+
+
+- On page 314: In Example 11-5 and Example 11-6 'var linetext =
+document.createTextNode(messageParagraphs[i]);' has been changed to 'var linetext =
+document.createTextNode(lines[i]);'
-
+
Chapter 12
-- On page 339: A new 'Certificate Authorities and Digital
-Signatures' section is being written with drafts available on http://certs.mozdev.org
-
+- On page 336: In Example 12-6
-
- (p 336) Use of backtics instead of single quote.
-<![CDATA[
+<xmp class="broken">
print ` <checkbox label="%s" />' % form['op1'].value
print ` <checkbox label="%s" />' % form['op2'].value
print ` <checkbox label="%s" />' % form['op3'].value
-Should be:
- print ' <checkbox label="%s" />' % form['op1'].value
- print ' <checkbox label="%s" />' % form['op2'].value
+]]>
+
+has been changed to (note the difference in indentation and in using a single quote versus a backtick)
+
+<![CDATA[
+ print ' <checkbox label="%s" />' % form['op1'].value
+ print ' <checkbox label="%s" />' % form['op2'].value
print ' <checkbox label="%s" />' % form['op3'].value
]]>
+
+
+
-
+
Appendix A
-- On page 365: 'http://www.mozilla.org/source.tml' has been
-changed to 'http://www.mozilla.org/source.html'
+ - On page 365: 'http://www.mozilla.org/source.tml' has been changed to 'http://www.mozilla.org/source.html'
+
+
+- On page 367 and 368: CVS Instructions ...
+
+Linux Instructions (p367)
+<![CDATA[
+1. Set the CVSROOT variable:
+ $ setenv CVSROOT :pserver:anonymous@cvs-mirror.mozilla.org:/cvsroot
+2. Log in as anonymous@cvs-mirror.mozilla.org using the password "anonymous":
+]]>
+
+has been changed to (note the adding to '=' after CVSROOT,
+the changing of 'setenv' to 'export',
+and the simplification of instruction 2)
+
+<![CDATA[
+1. Set the CVSROOT variable:
+ $ export CVSROOT=:pserver:anonymous@cvs-mirror.mozilla.org:/cvsroot
+2. Log in using the password "anonymous":
+]]>
+
+Windows Instructions (p368)
+<![CDATA[
+1. Set the CVSROOT variable:
+ > set CVSROOT :pserver:anonymous@cvs-mirror.mozilla.org:/cvsroot
+2. Log in as anonymous@cvs-mirror.mozilla.org using the password "anonymous":
+]]>
+
+has been changed to (note the adding to '=' after CVSROOT,
+and the simplification of instruction 2)
+
+<![CDATA[
+1. Set the CVSROOT variable:
+ > set CVSROOT=:pserver:anonymous@cvs-mirror.mozilla.org:/cvsroot
+2. Log in using the password "anonymous":
+]]>
+
+
+
+
+
+
+Index
+
+
+
+-
+On page 437: 'Chimera, 2' has been changed to 'Camino, 2'
+
+
-
-
+
+
+
+