Move all the AtomicString table into AtomicStringTable.

Move all the AtomicString hash logic into AtomicStringTable.

This makes the code more clear by separating it from AtomicString itself. It will
also allow us to do some clever things like storing a reference to the table in
the PerIsolateData and skipping the TLS lookups in the bindings.

Committed: https://crrev.com/ef596a22eb27ec50b348c01684a6cf3940bca560
Cr-Commit-Position: refs/heads/master@{#403066}

[esprehn, 2016-06-29 21:46:42]

The CQ bit was checked by esprehn@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-29 21:49:20]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-29 22:09:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-29 22:09:04]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[esprehn, 2016-06-29 22:58:56]

The CQ bit was checked by esprehn@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-29 23:00:09]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[esprehn, 2016-06-29 23:02:20]

Description was changed from

==========
Move all the AtomicString hash logic into AtomicStringTable.

BUG=
==========

to

==========
Move all the AtomicString hash logic into AtomicStringTable.

This makes the code more clear by separating it from AtomicString itself. It
will
also allow us to do some clever things like storing a reference to the table in
the PerIsolateData and skipping the TLS lookups in the bindings.
==========

[esprehn, 2016-06-29 23:43:05]

esprehn@chromium.org changed reviewers:
+ haraken@chromium.org, jbroman@chromium.org, yutak@chromium.org

[haraken, 2016-06-29 23:50:02]

LGTM

https://codereview.chromium.org/2110813003/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/wtf/text/AtomicStringTable.h (right):

https://codereview.chromium.org/2110813003/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/wtf/text/AtomicStringTable.h:53: // Froming a
StringImpl. This is for ~StringImpl, and should not be used

Froming => Removing

[esprehn, 2016-06-29 23:53:29]

The CQ bit was checked by esprehn@chromium.org

[esprehn, 2016-06-29 23:53:29]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2110813003/#ps40001
(title: "fix comments.")

[commit-bot: I haz the power, 2016-06-29 23:54:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-30 02:17:19]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-30 02:19:14]

Description was changed from

==========
Move all the AtomicString hash logic into AtomicStringTable.

This makes the code more clear by separating it from AtomicString itself. It
will
also allow us to do some clever things like storing a reference to the table in
the PerIsolateData and skipping the TLS lookups in the bindings.
==========

to

==========
Move all the AtomicString hash logic into AtomicStringTable.

This makes the code more clear by separating it from AtomicString itself. It
will
also allow us to do some clever things like storing a reference to the table in
the PerIsolateData and skipping the TLS lookups in the bindings.

Committed: https://crrev.com/ef596a22eb27ec50b348c01684a6cf3940bca560
Cr-Commit-Position: refs/heads/master@{#403066}
==========

[commit-bot: I haz the power, 2016-06-30 02:19:15]

Patchset 3 (id:??) landed as
https://crrev.com/ef596a22eb27ec50b348c01684a6cf3940bca560
Cr-Commit-Position: refs/heads/master@{#403066}