Allow multiple calls to SetDefaultSkiaFactory on Windows.

Allow multiple calls to SetDefaultSkiaFactory on Windows.

The ppapi code currently calls SetDefaultSkiaFactory twice on Win8+.
Previous code simply leaked an SkFontMgr and the current code asserts
(which is how this was discovered). This CL allows SetDefaultSkiaFactory
to be called more than once, properly handling the reference counting
and asserting in debug builds that SetDefaultSkiaFactory is not called
after SkFontMgr::Factory is called.

BUG=chromium:682652
Review-Url: https://codereview.chromium.org/2644103003
Cr-Commit-Position: refs/heads/master@{#444865}
Committed: https://chromium.googlesource.com/chromium/src/+/5ecd69eb2b90a83d67cce615e564dcc70a2d14bb

[bungeman-chromium, 2017-01-19 21:03:04]

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

[commit-bot: I haz the power, 2017-01-19 21:03:31]

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

[bungeman-chromium, 2017-01-19 21:04:42]

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

[f(malita), 2017-01-19 21:55:00]

lgtm

[bungeman-chromium, 2017-01-19 22:21:17]

The CQ bit was unchecked by bungeman@chromium.org

[bungeman-chromium, 2017-01-19 22:21:21]

The CQ bit was checked by bungeman@chromium.org

[commit-bot: I haz the power, 2017-01-19 22:21:31]

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-01-19 22:28:47]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484864481986750, "parent_rev":
"fd62178936d1d95a08421a7295b1727bdf647b9a", "commit_rev":
"5ecd69eb2b90a83d67cce615e564dcc70a2d14bb"}

[commit-bot: I haz the power, 2017-01-19 22:29:17]

Description was changed from

==========
Allow multiple calls to SetDefaultSkiaFactory on Windows.

The ppapi code currently calls SetDefaultSkiaFactory twice on Win8+.
Previous code simply leaked an SkFontMgr and the current code asserts
(which is how this was discovered). This CL allows SetDefaultSkiaFactory
to be called more than once, properly handling the reference counting
and asserting in debug builds that SetDefaultSkiaFactory is not called
after SkFontMgr::Factory is called.

BUG=chromium:682652
==========

to

==========
Allow multiple calls to SetDefaultSkiaFactory on Windows.

The ppapi code currently calls SetDefaultSkiaFactory twice on Win8+.
Previous code simply leaked an SkFontMgr and the current code asserts
(which is how this was discovered). This CL allows SetDefaultSkiaFactory
to be called more than once, properly handling the reference counting
and asserting in debug builds that SetDefaultSkiaFactory is not called
after SkFontMgr::Factory is called.

BUG=chromium:682652

Review-Url: https://codereview.chromium.org/2644103003
Cr-Commit-Position: refs/heads/master@{#444865}
Committed:
https://chromium.googlesource.com/chromium/src/+/5ecd69eb2b90a83d67cce615e564...
==========

[commit-bot: I haz the power, 2017-01-19 22:29:18]

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