InstanceRegistry silent ignores errors

[pixelzoom]

For phetsims/tasks#1017.

InstanceRegistry silent ignores errors that it catches. Is that a problem? Is it accptable in production code?

        catch( e ) {

          // Ignore nodes that don't draw anything
          // TODO: is this masking a problem?
        }

[samreid]

This is used to render components for binder reports. @ariel-phet can you please prioritize and assign?

[ariel-phet]

@chrisklus you have been involved in binder more recently if I recall - mind looking into this issue? Marking as medium priority to look into, but feel free to downgrade if you deem it not to be a pressing issue.

[samreid]

As a first step, please remove the try/catch (or put a console.log) and run a full binder harvest, then see if any exceptions are thrown. If any are thrown, we should try to understand why a newly constructed Node cannot be rendered.