--- books/www/corrections.html 2003/03/13 17:17:50 1.87 +++ books/www/corrections.html 2003/03/19 11:35:08 1.99 @@ -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,7 +83,7 @@ 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.'

    @@ -106,6 +106,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 +123,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 +153,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 +169,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'

    @@ -175,7 +193,6 @@ class="fixed">The chrome URL that the -m
  • On page 135: Example 5-11 has been changed to - <head> <title>Sound Service Play Example</title> <script> @@ -196,7 +213,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";' @@ -240,45 +256,112 @@ else // otherwi + +

    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):

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

      - -
    • On page 223: In Example 8-4 one of the duplicate 'if (outer != null)' lines has been removed

      +function NSGetModule(compMgr, fileSpec) { return Module; } + +
    • -
    • On page 239: Example 8-15 has been changed to
      +
    • On page 239: Example 8-15 has been changed to
    • DEPTH = ../../..