Attempted fix for refcount in sideloaded font path

Attempted fix for refcount in sideloaded font path

Followup to buggy code added in http://src.chromium.org/viewvc/blink?view=rev&rev=181407 .

This code path is only used in layout tests.

TBR=eae@chromium.org
R=bungeman@chromium.org
BUG=394902
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181587

[bungeman-chromium, 2014-09-08 17:49:27]

lgtm

[scottmg, 2014-09-08 17:51:44]

The CQ bit was checked by scottmg@chromium.org

[commit-bot: I haz the power, 2014-09-08 17:52:01]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/scottmg@chromium.org/551063002/1

[scottmg, 2014-09-08 17:52:16]

scottmg@chromium.org changed reviewers:
+ eae@chromium.org

[commit-bot: I haz the power, 2014-09-08 22:03:27]

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

[commit-bot: I haz the power, 2014-09-08 22:03:28]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/26078)

[scottmg, 2014-09-08 22:09:57]

The CQ bit was checked by scottmg@chromium.org

[commit-bot: I haz the power, 2014-09-08 22:10:59]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/scottmg@chromium.org/551063002/1

[commit-bot: I haz the power, 2014-09-09 00:07:34]

Committed patchset #1 (id:1) as 181587

[Stephen White, 2014-09-09 12:05:08]

On 2014/09/09 00:07:34, I haz the power (commit-bot) wrote:
> Committed patchset #1 (id:1) as 181587

Drive-by peanut: we try to adoptRef() the bare pointers returned by Skia
constructors or factory methods directly into RefPtrs at the place where the new
Skia object is created (ie., the call to the constructor or create() factory),
and then use RefPtr and PassRefPtr from there to indicate ownership transfer. I
don't know if that's possible in your case, since the code crosses the
Chrome/Blink API, but I thought I'd mention it.