JSString::SafeView no longer achieves its intended goal to make it easier to handle strings safely. Its clients still need to do explicit exception checks in order to be correct. We'll remove it and replace its uses with StringViewWithUnderlyingString instead which serves to gets the a StringView (which is what we really wanted from SafeView) and keeps the backing String alive while the view is in use.
Created attachment 294845 [details] proposed patch.
Created attachment 294846 [details] proposed patch.
Comment on attachment 294846 [details] proposed patch. r=me
Thanks for the review. Landed in r208767: <http://trac.webkit.org/r208767>.