Update base::scoped_refptr<> docs

Update base::scoped_refptr<> docs

chromium-style enforces a private or protected destructor for classes
that derive from RefCounted<> which in turn makes it necessary to add a
friend declaration for RefCounted<>.

BUG=none

Committed: https://crrev.com/55b05a8ae594e2e8d49c7e6da64bbea51df7a76f
Cr-Commit-Position: refs/heads/master@{#435516}

[Thiemo Nagel, 2016-11-29 11:06:30]

tnagel@chromium.org changed reviewers:
+ danakj@chromium.org

[Thiemo Nagel, 2016-11-29 11:06:32]

Hi Dana,

could you please take a look at this small update?

Thank you,
Thiemo

[danakj, 2016-11-30 00:00:08]

Thanks, just style fix for comment (even comments inside comments)

https://codereview.chromium.org/2533243002/diff/1/base/memory/ref_counted.h
File base/memory/ref_counted.h (right):

https://codereview.chromium.org/2533243002/diff/1/base/memory/ref_counted.h#n...
base/memory/ref_counted.h:228: //     friend class RefCounted<MyFoo>;  // allow
destruction by RefCounted<>
Capitalize and period for each comment.

[Thiemo Nagel, 2016-11-30 10:41:24]

Thank you!  Could you please take another look?

https://codereview.chromium.org/2533243002/diff/1/base/memory/ref_counted.h
File base/memory/ref_counted.h (right):

https://codereview.chromium.org/2533243002/diff/1/base/memory/ref_counted.h#n...
base/memory/ref_counted.h:228: //     friend class RefCounted<MyFoo>;  // allow
destruction by RefCounted<>
On 2016/11/30 00:00:08, danakj wrote:
> Capitalize and period for each comment.

Done.

[danakj, 2016-11-30 23:17:38]

LGTM

[danakj, 2016-11-30 23:17:41]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2016-11-30 23:18:01]

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 01:19:53]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480547861904140,
"parent_rev": "cff6272fc6ace2164fc887b022db940395b908cb", "commit_rev":
"1ff75213b354d0e27ad940796ce338e53c61383b"}

[commit-bot: I haz the power, 2016-12-01 01:20:56]

Committed patchset #2 (id:20001)

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

Description was changed from

==========
Update base::scoped_refptr<> docs

chromium-style enforces a private or protected destructor for classes
that derive from RefCounted<> which in turn makes it necessary to add a
friend declaration for RefCounted<>.

BUG=none
==========

to

==========
Update base::scoped_refptr<> docs

chromium-style enforces a private or protected destructor for classes
that derive from RefCounted<> which in turn makes it necessary to add a
friend declaration for RefCounted<>.

BUG=none

Committed: https://crrev.com/55b05a8ae594e2e8d49c7e6da64bbea51df7a76f
Cr-Commit-Position: refs/heads/master@{#435516}
==========

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

Patchset 2 (id:??) landed as
https://crrev.com/55b05a8ae594e2e8d49c7e6da64bbea51df7a76f
Cr-Commit-Position: refs/heads/master@{#435516}

[Thiemo Nagel, 2016-12-01 09:30:00]

On 2016/11/30 23:17:38, danakj wrote:
> LGTM

Thanks for ticking the box!
Thiemo