Question about gagdet_site.js GadgetSite.close impl for close() function

[Henry Saputra]

In the gadget_site.js file there is this function:

/**
 * @inheritDoc
 */
osapi.container.GadgetSite.prototype.close = function() {
  osapi.container.Site.prototype.close.call(this); // super.close();
  if (this.loadingGadgetEl_ && this.loadingGadgetEl_.firstChild) {
    this.loadingGadgetEl_.removeChild(this.loadingGadgetEl_.firstChild);
  }

  // dispose of our other holder.
  var holder = this.getActiveSiteHolder();
  holder && holder.dispose();
};


The parent site.js has this:


/**
 * Closes this site.
 */
osapi.container.Site.prototype.close = function() {
  if (this.el_ && this.el_.firstChild) {
    this.el_.removeChild(this.el_.firstChild);
  }

  var holder = this.getActiveSiteHolder();
  holder && holder.dispose();
};


If am not sure what " holder && holder.dispose();" should do. I think
its trying to do:

if(holder) {
  holder.dispose();
}

And the GadgetSite should not repeat the code I suppose.

- Henry