AtomicString::show() and WTFString::show() should be const.

AtomicString::show() and WTFString::show are not const, so we have to use const_cast to call them occasionally.
These methods should be const.

Created an attachment (id=98870) [details]
Patch

(In reply to comment #0)
> AtomicString::show() and WTFString::show are not const, so we have to use const_cast to call them occasionally.

So you should remove such const_casts in this patch. Otherwise, the patch has no benefit.

Since this is a debug feature, there is no code to call show() in Sources (maybe). However, when doing something like printf-debug, the lack of const is painful.

This bug is related to 
https://bugs.webkit.org/show_bug.cgi?id=61149

(In reply to comment #3)
> Since this is a debug feature, there is no code to call show() in Sources (maybe). However, when doing something like printf-debug, the lack of const is painful.

Ok, I understand.  Please note such reasons to ChangeLog.

Created an attachment (id=98875) [details]
Patch

(From update of attachment 98875 [details])
ok.

(From update of attachment 98875 [details])
Clearing flags on attachment: 98875

Committed r89906: <http://trac.webkit.org/changeset/89906>

All reviewed patches have been landed.  Closing bug.