ref(traces): Remove hiddenAttributes prop from AttributeTree

[gggritso]

Small refactor to simplify some upcoming changes. AttributeTree accepts an attributes prop, and a hiddenAttributes prop. Any attribute whose name is included in hiddenAttributes is visually filtered out. This seems strange:

1. It's strange that the tree component has to iterate all the attributes and then decide which ones to hide one-by-one. It's much more efficient to just omit those attributes from the component, so it only iterates attributes that actually show up
2. The hidden attributes are redundant. The AttributeTree in the trace view, for example, both passes HIDDEN_ATTRIBUTES and manually omits those fields from attributes, I'm not sure why
3. The logic that hides attributes is very deep inside AttributeTree, so it's not clear from looking at the component why some things are missing
4. In the near future, we'll need to omit attributes by some dynamic property (e.g., omit anything that starts with "__sentry" but not for Staff users), which cannot be expressed via hiddenAttributes

It seems easier and more consistent to remove the hiddenAttributes prop, and ensure that the parent component only passes whichever attributes should be omitted.

Am I missing something?

Changes

• Add test case for hidden attributes
• Stop passing hiddenAttributes
• Remove unused hidden attributes prop

[gggritso]

@k-fish @Abdkhan14 what do you think of this one?

[Abdkhan14]

On board 🚢

[k-fish]

Aside from mentioned doc updates offline, looks good to me.