The issue appears to be one of whether the tab name defined in the XML (via `policy.json`) or that defined via Studio takes precedence. 

For courses which use the policy file to define the tab names (eg typically those imported from XML), the name defined by the policy file appears in `metadata`, and that is what should be used.  

We were seeing that`tab.get('name')` was returning the string "None" (versus the python `None`).  If that's still the case, then this change will break the usage needed for folks relying on the ability to edit tab names via `policy.json`.  If `tab.get('name')` is returning `None` then this change is fine.