Add comments to wtf/AtomciString.h.

Add comments to wtf/AtomciString.h.

This CL has no behavior changes.

NOTRY=true

Committed: https://crrev.com/8c8664eebc5f132119492447844a66e49bdbff0e
Cr-Commit-Position: refs/heads/master@{#435581}

[tkent, 2016-12-01 06:35:19]

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

[commit-bot: I haz the power, 2016-12-01 06:35:33]

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

[tkent, 2016-12-01 06:52:05]

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

[commit-bot: I haz the power, 2016-12-01 06:52:09]

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

[tkent, 2016-12-01 06:52:24]

Patchset #1 (id:1) has been deleted

[tkent, 2016-12-01 06:57:06]

tkent@chromium.org changed reviewers:
+ kochi@chromium.org

[tkent, 2016-12-01 06:57:08]

kochi@, would you review this please?

[kochi, 2016-12-01 07:02:26]

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

https://codereview.chromium.org/2546473004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/wtf/text/AtomicString.h:38: // instances shares their
string storage if the strings are identical. Comparing
s/shares/can share/

https://codereview.chromium.org/2546473004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/wtf/text/AtomicString.h:42: // AtomicString instances
are not tread-safe. An AtomicString instance created
s/tread/thread/

[tkent, 2016-12-01 07:09:18]

thank you for reviewing.

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

https://codereview.chromium.org/2546473004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/wtf/text/AtomicString.h:38: // instances shares their
string storage if the strings are identical. Comparing
On 2016/12/01 at 07:02:26, kochi wrote:
> s/shares/can share/

Done.

https://codereview.chromium.org/2546473004/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/wtf/text/AtomicString.h:42: // AtomicString instances
are not tread-safe. An AtomicString instance created
On 2016/12/01 at 07:02:26, kochi wrote:
> s/tread/thread/

Done.

[tkent, 2016-12-01 07:09:26]

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

[commit-bot: I haz the power, 2016-12-01 07:09:41]

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

[kochi, 2016-12-01 07:16:16]

lgtm

[kochi, 2016-12-01 07:16:57]

https://codereview.chromium.org/2546473004/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/wtf/text/AtomicString.h (right):

https://codereview.chromium.org/2546473004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/wtf/text/AtomicString.h:38: // instances can shares
their string storage if the strings are
nit: can share (without s)

[tkent, 2016-12-01 08:24:30]

https://codereview.chromium.org/2546473004/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/wtf/text/AtomicString.h (right):

https://codereview.chromium.org/2546473004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/wtf/text/AtomicString.h:38: // instances can shares
their string storage if the strings are
On 2016/12/01 at 07:16:56, kochi wrote:
> nit: can share (without s)

oops.  Done.

[tkent, 2016-12-01 08:24:39]

Description was changed from

==========
Add comments to wtf/AtomciString.h.

This CL has no behavior changes.
==========

to

==========
Add comments to wtf/AtomciString.h.

This CL has no behavior changes.

NOTRY=true
==========

[tkent, 2016-12-01 08:24:42]

The CQ bit was checked by tkent@chromium.org

[tkent, 2016-12-01 08:24:43]

The patchset sent to the CQ was uploaded after l-g-t-m from kochi@chromium.org
Link to the patchset: https://codereview.chromium.org/2546473004/#ps60001
(title: "s")

[commit-bot: I haz the power, 2016-12-01 08:24:59]

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-12-01 08:30:16]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1480580682794090,
"parent_rev": "3437eaa3a9ffce22c93c8a6a4f2dbb1d5007a062", "commit_rev":
"fc5a462b9743706adc85d28a8996f50b5e391f5c"}

[commit-bot: I haz the power, 2016-12-01 08:30:47]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-12-01 08:33:22]

Description was changed from

==========
Add comments to wtf/AtomciString.h.

This CL has no behavior changes.

NOTRY=true
==========

to

==========
Add comments to wtf/AtomciString.h.

This CL has no behavior changes.

NOTRY=true

Committed: https://crrev.com/8c8664eebc5f132119492447844a66e49bdbff0e
Cr-Commit-Position: refs/heads/master@{#435581}
==========

[commit-bot: I haz the power, 2016-12-01 08:33:23]

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

[kochi, 2016-12-01 08:43:49]

Noticed that the title has mispelling (Atomci) :-(

[tkent, 2016-12-01 22:57:31]

On 2016/12/01 at 08:43:49, kochi wrote:
> Noticed that the title has mispelling (Atomci) :-(

OMG! I shouldn't have made a CL while I was drunk.