Make NinePieceImageData use RefCountedCopyable.

Make NinePieceImageData use RefCountedCopyable.

NinePieceImageData is RefCounted and copyable. This patch makes it
inherit from RefCountedCopyable so that it can use the compiler
generated copy constructor.

BUG=628043
Review-Url: https://codereview.chromium.org/2808703002
Cr-Commit-Position: refs/heads/master@{#465133}
Committed: https://chromium.googlesource.com/chromium/src/+/20e570ab5950d95673078e4edd1653faf97812b0

[shend, 2017-04-10 05:17:29]

shend@chromium.org changed reviewers:
+ nainar@chromium.org

[shend, 2017-04-10 05:17:30]

Hi Naina, PTAL

[nainar, 2017-04-10 17:19:03]

lgtm

[shend, 2017-04-12 04:10:39]

shend@chromium.org changed reviewers:
+ meade@chromium.org

[shend, 2017-04-12 04:10:40]

Hi Eddy, PTAL

[shend, 2017-04-18 00:01:04]

shend@chromium.org changed reviewers:
+ suzyh@chromium.org
- meade@chromium.org

[shend, 2017-04-18 00:01:05]

Redirecting to Suzy, PTAL

[suzyh_UTC10 (ex-contributor), 2017-04-18 00:38:01]

On 2017/04/18 at 00:01:05, shend wrote:
> Redirecting to Suzy, PTAL

lgtm

I'd recommend adding documentation for RefCountedCopyable to
platform/heap/BlinkGCAPIReference.md and/or other suitable locations to provide
guidelines on when (not) to use it.

[shend, 2017-04-18 01:14:30]

On 2017/04/18 at 00:38:01, suzyh wrote:
> On 2017/04/18 at 00:01:05, shend wrote:
> > Redirecting to Suzy, PTAL
> 
> lgtm
> 
> I'd recommend adding documentation for RefCountedCopyable to
platform/heap/BlinkGCAPIReference.md and/or other suitable locations to provide
guidelines on when (not) to use it.

Good idea! I'll do it in a separate patch.

[shend, 2017-04-18 01:51:06]

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

[commit-bot: I haz the power, 2017-04-18 01:51:41]

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, 2017-04-18 03:41:50]

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

[commit-bot: I haz the power, 2017-04-18 03:41:51]

Dry run: This issue passed the CQ dry run.

[shend, 2017-04-18 03:55:03]

The CQ bit was checked by shend@chromium.org

[commit-bot: I haz the power, 2017-04-18 03:55:21]

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, 2017-04-18 03:58:45]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492487703317960, "parent_rev":
"1c304d5beae8ffe9b9317353609abb31537656ac", "commit_rev":
"20e570ab5950d95673078e4edd1653faf97812b0"}

[commit-bot: I haz the power, 2017-04-18 03:59:31]

Description was changed from

==========
Make NinePieceImageData use RefCountedCopyable.

NinePieceImageData is RefCounted and copyable. This patch makes it
inherit from RefCountedCopyable so that it can use the compiler
generated copy constructor.

BUG=628043
==========

to

==========
Make NinePieceImageData use RefCountedCopyable.

NinePieceImageData is RefCounted and copyable. This patch makes it
inherit from RefCountedCopyable so that it can use the compiler
generated copy constructor.

BUG=628043

Review-Url: https://codereview.chromium.org/2808703002
Cr-Commit-Position: refs/heads/master@{#465133}
Committed:
https://chromium.googlesource.com/chromium/src/+/20e570ab5950d95673078e4edd16...
==========

[commit-bot: I haz the power, 2017-04-18 03:59:32]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/20e570ab5950d95673078e4edd16...