I would like to see this whole if/else block refactored to something like the following:

```javascript
this.numGoodResults++;
this._addPopupTabContent(options.params.layer.getHtmlFormattedFeatureInfo());  // or some such
```

We have way to much `if (layer has this property) do such and such else do something else`.  We need to push this logic in to the inheritance hierarchy to avoid all this messy if/else logic in client code.