--- books/www/corrections.html 2003/01/20 16:37:40 1.68
+++ books/www/corrections.html 2003/04/22 15:00:25 1.122
@@ -1,9 +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,
@@ -12,31 +12,54 @@ section on the chapter pages, or send a
href="mailto:reviewers@mozdev.org">reviewers@mozdev.org mailing list.
Note: All errors in the text and examples are shown in red and all corrections are shown in green.
+class="fixed">blue.
-
+
Preface
+- 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 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 Firebird project (http://www.mozilla.org/projects/firebird/) 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
@@ -59,10 +82,38 @@ changed to 'selecting F
- 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: In Figure 2-5 there should be an en-US subfolder under the locale folder. 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/'
+
+ - On page 33: A warning note has been added before the last paragraph of the chapter
+
+<![CDATA[
+If you have trouble editing and relaunching the xFly application,
+there are a couple of platform specific culprits to look into. On
+Windows, the Quick Launch feature may prevent you from seeing changes
+to your files. On Unix, file permission conflicts can also cause
+problems viewing your files. Making sure that Quick Launch is disabled
+and double checking file permissions may help get xFly working
+on your system.
+]]>
+
-
+
Chapter 3
@@ -70,6 +121,19 @@ to 'CenterWindowOnScree
- 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( );" >'
@@ -77,9 +141,59 @@ id="printBtn" label="Print" oncommand="d
- 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 separate 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
@@ -95,9 +209,25 @@ changed to '<descrip
- 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[
-<html>
<head>
<title>Sound Service Play Example</title>
<script>
@@ -106,24 +236,22 @@ changed to '<b class="fixed"><descrip
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';
+ 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>
- <body>
+
<form name="form">
<input type="button" value="Play Sound" onclick="play();">
</form>
- </body>
-</html>
]]>
-
-- On page 137: 'var connection = Components.classes ["@mozilla.org/network/ldap-connection;1";' has been changed to
+
- 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"];'
@@ -133,44 +261,156 @@ 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 7
+
+
+
+- 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):
+
+
+
+
+
+
+
+
+
+
Chapter 8
-- On page 211: In 2nd sentence of paragraph 'QueryInterace' has been changed to 'QueryInterface'
+ - On page 211: In 2nd sentence of paragraph 'QueryInterace' has been
+changed to 'QueryInterface'
-- On page 222: In Example 8-4 'mName: "a default value"' has been changed to 'mName: "a default
-value",'
+ - On page 222: In Example 8-4 has been changed to:
-
- On page 222: In Example 8-4
+<![CDATA[
+function SimpleComponent(){}
- <xmp class="broken">
- this.myCID,
- "My JS Component",
- this.myProgID, fileSpec,
- location, true, true,
- type);
- ]]>
+SimpleComponent.prototype = {
-has been changed to
+ get yourName() { return this.mName; },
+ set yourName(aName) { return this.mName = aName; },
- <![CDATA[
- compMgr.registerComponentWithType(
- this.myCID,
- "My JS Component",
- this.myProgID, fileSpec,
- location, true, true,
- type);
- ]]>
+ 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
-
- On page 222: In Example 8-4 one of the duplicate 'dump(" ***** Registering: Simple JS component!
-****\n");' lines has been removed
+function NSGetModule(compMgr, fileSpec) { return Module; }
+
+
-- On page 223: In Example 8-4 one of the duplicate 'if (outer != null)' lines has been removed
+ - 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");
+]]>
+
+
-- On page 239: Example 8-15 has been changed to
+ - On page 239: Example 8-15 has been changed to
<![CDATA[
DEPTH = ../../..
@@ -213,29 +453,165 @@ install:: $(TARGETS)
</ul>
-
-<a name="ch10">
-<p><b>Chapter 10</b>
+<a name="ch9"></a>
+<p><b>Chapter 9</b>
<ul>
-<li>On page 297: '<b class="broken">http://www.jslib.mozdev.org</b>' has been changed
-to '<b class="fixed">http://jslib.mozdev.org</b>'<br><br></li>
-</ul>
+<li>On page 247: '<b class="broken">When it appears inside the <template>, the <listbox> tag has two special attributes.</b>'
+has been changed to '<b class="fixed">The <listbox> gains two special attributes when it contains a <template>.</b>'
+<br><br>
+
+<li>On page 250: '<b class="broken">"Using Data for Style,"</b>' has been changed to '<b class="fixed">"Using Data for Style",</b>'
+<br><br>
+
+<li>On page 254: Example 9-5
+
+<xmp class="broken">
+ <rdf:Description about="urn:root">
+ <fly:list</td>>
+ <rdf:Seq>
+ <rdf:li>
+ <rdf:Description ID="House">
+ <fly:label</td>>House</fly:label>
+ <fly:list</td>>
+ <rdf:Seq>
+ <rdf:li>
+ <rdf:Description about="musca_autumnalis"
+ fly:label</td>="Face Fly"/>
+ </rdf:li>
+ <rdf:Seq>
+ </fly:list>
+ </rdf:li>
+ </rdf:Seq>
+ </fly:list>
+ </rdf:Description>
+]]>
+
+has been changed to
+
+<![CDATA[
+ <rdf:Description about="urn:root">
+ <fly:list</td>>
+ <rdf:Seq>
+ <rdf:li>
+ <rdf:Description ID="House">
+ <fly:label</td>>House</fly:label>
+ </rdf:Description>
+ <fly:list</td>
+ <rdf:Seq>
+ <rdf:li>
+ <rdf:Description about="musca_autumnalis"
+ fly:label</td>="Face Fly"/>
+ </rdf:li>
+ <rdf:Seq>
+ </fly:list>
+ </rdf:li>
+ </rdf:Seq>
+ </fly:list>
+ </rdf:Description>
+]]>
+
+- On page 255: Example 9-6
+
+<![CDATA[
+ <treecols>
+ <treecol id="LabelCol" flex="1" sort="?label" label="Name"
+ primary="true" />
+ <treecol id="LoCol" flex="1" sort="?location" label="Location"/>
+ <treecol id="ColCol" flex="1" sort="?color" label="Color"/>
+ </treecols>
+]]>
+
+has been changed to
+
+<![CDATA[
+ <treecols>
+ <treecol id="LabelCol" flex="1"
+ sort="rdf:http://xfly.mozdev.org/fly-rdf#label"
+ class="sortDirectionIndicator" primary="true" label="Name"/>
+ <treecol id="LoCol" flex="1"
+ sort="rdf:http://xfly.mozdev.org/fly-rdf#location"
+ label="Location"/>
+ <treecol id="ColCol" flex="1"
+ sort="rdf:http://xfly.mozdev.org/fly-rdf#color"
+ label="Color"/>
+ </treecols>
+]]>
+
+
- On page 257: In Example 9-7
+
+<![CDATA[
+ <triple subject="?listitem"
+ predicate=" http://xfly.mozdev.org/fly-rdf#location"
+ object="?location"/>
+]]>
+
+has been changed to
+
+<![CDATA[
+ <triple subject="?listitem"
+ predicate="fly-location#location"
+ object="?location"/>
+]]>
+
+
- On page 258: 'The most important additions to Example 9-7 are the
+container="?uri" member="?listitem" attributes on the <template>' has been
+changed to 'The most important inclusions to Example 9-7 are the
+container="?uri" member="?listitem" attributes on the <template>'
+
+
+
-
+
Chapter 10
-- On page 314: In Example 11-5 'var linetext = document.createTextNode(messageParagraphs[i]);'
-has been changed to 'var linetext = document.createTextNode(lines[i]);'
- - On page 314: In Example 11-6 'var linetext = document.createTextNode(messageParagraphs[i]);'
-has been changed to 'var linetext = document.createTextNode(lines[i]);'
+ - 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: The code at the top of the page
+
+<![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: In Example 10-14 '// if(del)gRDF.removeSeq('types',false);' has been deleted. It is 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
@@ -256,25 +632,81 @@ has been changed to (note the difference
print ' ' % form['op3'].value
-- On page 339: A new 'Certificate Authorities and Digital
-Signatures' section is being written with drafts available on http://certs.mozdev.org
-
+- On page 341: The following sentence has been deleted 'You can delete the key3.db and secmod.db files
+because they do not pertain to certificate creation or modification.'
+
+
+
NOTE: This last fix in chapter 12 is a
short/quick fix
+for some major flaws in the security section of this chapter.
+There is a more desirable fix,
outlined in the bug monitoring it,
+but it would require a substantial rewrite to the section.
+Please consider both of these options for a second printing.
+
-
+
+
+
Appendix A
- On page 365: 'http://www.mozilla.org/source.tml' has been changed to 'http://www.mozilla.org/source.html'
+
+
+- On page 367: In the Linux CVS instructions
+
+<![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 addition of '=' 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":
+]]>
+
+
- On page 368: In the Windows CVS instructions
+
+<![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 addition of '=' 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'
+
+
-
-
+
+
+