--- books/www/corrections.html 2003/03/13 17:17:50 1.87 +++ books/www/corrections.html 2003/04/01 16:24:25 1.113 @@ -1,6 +1,6 @@

Corrections @@ -12,7 +12,7 @@ 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 @@ -83,15 +83,30 @@ changed to 'selecting F 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, +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):

    + +New
+Figure 2-5

    +
  • 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 + + +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. + + @@ -106,6 +121,16 @@ to 'We've packed a lot
  • 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) + + + 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( );" >'

    @@ -113,6 +138,9 @@ 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 seperate values) @@ -140,6 +168,11 @@ changed to '<b class="fixed">&lt;descrip </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.'

    + @@ -151,8 +184,8 @@ changed to '<descrip
  • 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'

    +
  • 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'

    @@ -173,9 +206,25 @@ class="fixed">The chrome URL that the -m
  • On page 120: 'dump(l+'\n');' has been changed to 'dump(len+'\n');'

  • -
  • On page 135: Example 5-11 has been changed to +
  • On page 123: Example 5-3 + +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 +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 + <head> <title>Sound Service Play Example</title> <script> @@ -184,10 +233,10 @@ class="fixed">The chrome URL that the -m 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> @@ -196,7 +245,6 @@ class="fixed">The chrome URL that the -m <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";' @@ -215,7 +263,7 @@ load('chrome/jslib/jslib.js');'
    <
      -
    • On page 145: Example 6-1 has been changed to
      +
    • On page 145: Example 6-1 has been changed to var myFile = "xFly.jar"; @@ -225,60 +273,141 @@ initInstall( // ini "0.0.1", // version of install 1); // flags - an optional argument, // reserved for future use -f = getFolder("Chrome"); // specify a target directory +f = getFolder("Chrome"); // specify a target directory setPackageFolder(f); -addFile(myFile); // add software to the installation +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. +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):

      + +New Figure 7-2 + +
    • + +
    + +

    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 + +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; }, - - 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 + + NS_INIT_REFCNT( ); // has been depricated use NS_INIT_ISUPPORTS() + mValue = PL_strdup("default value"); + +has been changed to + + // 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
    • DEPTH = ../../.. @@ -326,22 +455,109 @@ install:: $(TARGETS) <ul> +On page 247: '<b class="broken">When it appears inside the &lt;template&gt;, the &lt;listbox&gt; tag has two special attributes.</b>' +has been changed to '<b class="fixed">The &lt;listbox&gt; gains two special attributes when it contains a &lt;template&gt;.</b>' +<br><br> + +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> + +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 + + +' <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 + + + <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 + + + <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 - <triple subject="?listitem" - predicate=" http://xfly.mozdev.org/fly-rdf#location" - object="?location"/> + <triple subject="?listitem" + predicate=" http://xfly.mozdev.org/fly-rdf#location" + object="?location"/> has been changed to <triple subject="?listitem" - predicate="fly-location#location" + 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>' +

      +
    @@ -349,12 +565,36 @@ has been changed to
      +
    • 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: + + +'containerUtils = Components.classes['@mozilla.org/rdf/container-utils;1' +getService(Components.interfaces.nsIRDFContainerUtils);' + + +has been changed to: + + +'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.

      +
    @@ -406,21 +646,59 @@ href="http://certs.mozdev.org">http://ce
  • 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)
    + +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) + + +1. Set the CVSROOT variable: + $ export CVSROOT=:pserver:anonymous@cvs-mirror.mozilla.org:/cvsroot +2. Log in using the password "anonymous": + + +Windows Instructions (p368)
    + +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) + + +1. Set the CVSROOT variable: + > set CVSROOT=:pserver:anonymous@cvs-mirror.mozilla.org:/cvsroot +2. Log in using the password "anonymous": + + +
  • +

    Index



    - - +