RefPtr -> SkAutoTUnref on SkStream to remove SkRefCnt.

RefPtr -> SkAutoTUnref on SkStream to remove SkRefCnt.

SkStream is currently SkRefCnt, but this has never been useful, and is
often a problem. As a result, SkStream will soon no longer be reference
counted. RefPtr places a number of constraints on the class being
reference counted, so use SkAutoTUnref in the mean time to stage the
reference counting removal.

The plan is to change this one instance to SkAutoTUnref, then make
SkStream no longer reference counted but contain a 'dummy' unref().
The use of SkAutoTUnref here will then be removed and SkStream's
'dummy' unref() will be removed.

The proposed changes to SkStream can be seen at
https://codereview.chromium.org/849103004/ .

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

[bungeman-chromium, 2015-01-20 17:08:07]

bungeman@chromium.org changed reviewers:
+ eae@chromium.org, reed@chromium.org, scroggo@chromium.org

[scroggo, 2015-01-20 17:16:14]

On 2015/01/20 17:08:07, bungeman2 wrote:

LGTM.

[bungeman-chromium, 2015-01-20 17:22:03]

Summary of how this works for other reviewers.

Currently we never take ownership of the reference, but we always take ownership
of the data. With this change, we will effectively always be taking both. When
SkStream drops inheritance from SkRefCnt, it will (temporarily) add an 'unref()'
which does nothing.

[bungeman-chromium, 2015-01-20 18:12:37]

bungeman@chromium.org changed reviewers:
+ schenney@chromium.org

[eae, 2015-01-20 18:40:36]

LGTM

[bungeman-chromium, 2015-01-20 18:41:26]

The CQ bit was checked by bungeman@chromium.org

[commit-bot: I haz the power, 2015-01-20 18:42:00]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/825903004/1

[commit-bot: I haz the power, 2015-01-20 20:41:13]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=188693