Remove WTF::textAtom and WTF::commentAtom

Remove WTF::textAtom and WTF::commentAtom

These are only used in implementations of Node::nodeName(), where
CDATASection, Document and DocumentFragment already use raw strings.

Added in <https://trac.webkit.org/changeset/9639>;, at which point both
atoms were used in multiple places. Previously raw strings were used.

BUG=none

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=163281

[jochen (gone - plz use gerrit), 2013-12-05 13:00:16]

why did you chose to remove the atomic strings here instead of adding them in
the other classes? is there a performance difference?

[philipj_slow, 2013-12-05 13:17:53]

On 2013/12/05 13:00:16, jochen wrote:
> why did you chose to remove the atomic strings here instead of adding them in
> the other classes? is there a performance difference?

I've tracked this back to <https://trac.webkit.org/changeset/9639>, where both
atoms were added and used in multiple places. Before a simple string was used.

I doubt that this is performance critical, but if it were I think that
DEFINE_STATIC_LOCAL would make more sense than global atoms. Is there any
performance measurement you would like me to do, or can we trust the perf bots?

[jochen (gone - plz use gerrit), 2013-12-05 14:44:19]

no, trusting the perf bots is fine

lgtm

[commit-bot: I haz the power, 2013-12-05 14:52:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/philipj@opera.com/106483002/1

[commit-bot: I haz the power, 2013-12-05 18:11:15]

Retried try job too often on win_blink_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_blink_...

[commit-bot: I haz the power, 2013-12-05 18:37:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/philipj@opera.com/106483002/1

[commit-bot: I haz the power, 2013-12-05 19:28:42]

Change committed as 163281