Minor cleanup of opaque-keys library

Is there a situation (now or in the future) where the `CourseLocator` might be created from a deprecated string but we might want to create a non-deprecated `UsageKey`? This change is better than what was there before, but I figure it's good to ask the question now rather than later.