Update documentation for SkTypeface constructors.

Update documentation for SkFontMgr methods.

The appropriate way to get the default system font is to pass |nullptr| as the
familyName. This was not clear from the interface, which may be why ui/gfx/ has
the wrong behavior.

BUG=chromium:564266
Committed: https://skia.googlesource.com/skia/+/2b1516fe35155e05331ca6d4f09be916dc1fd778

[erikchen, 2015-12-03 18:36:09]

Description was changed from

==========
Update documentation for SkTypeface constructors.

The appropriate way to get the default system font is to pass |nullptr| as the
familyName. This was not clear from the interface, which may be why ui/gfx/ has
the wrong behavior.

BUG=chromium:564266
==========

to

==========
Update documentation for SkFontMgr methods.

The appropriate way to get the default system font is to pass |nullptr| as the
familyName. This was not clear from the interface, which may be why ui/gfx/ has
the wrong behavior.

BUG=chromium:564266
==========

[erikchen, 2015-12-03 18:36:33]

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

[erikchen, 2015-12-03 18:36:34]

bungeman: Please review.

[reed1, 2015-12-03 19:41:37]

reed@google.com changed reviewers:
+ reed@google.com

[reed1, 2015-12-03 19:41:38]

nice addition. lgtm.

nit: Specifically, we match for the system default family, and still apply the
specified Style to find the actual typeface.

[bungeman-skia, 2015-12-03 19:50:01]

bungeman@google.com changed reviewers:
+ bungeman@google.com

[bungeman-skia, 2015-12-03 19:50:01]

This lgtm, but we also will need to make sure we have tests for this (I don't
think we currently do) to make sure none of the ports are currently assuming
these are non-null. In particular I'm not sure about the style set one, early
implementations may not be conforming. I'm ok with the doc change now, I'll
write up the tests and see if they fail shortly.

[bungeman-skia, 2015-12-03 19:56:38]

The CQ bit was checked by bungeman@google.com

[commit-bot: I haz the power, 2015-12-03 19:56:46]

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

[commit-bot: I haz the power, 2015-12-03 20:12:16]

Description was changed from

==========
Update documentation for SkFontMgr methods.

The appropriate way to get the default system font is to pass |nullptr| as the
familyName. This was not clear from the interface, which may be why ui/gfx/ has
the wrong behavior.

BUG=chromium:564266
==========

to

==========
Update documentation for SkFontMgr methods.

The appropriate way to get the default system font is to pass |nullptr| as the
familyName. This was not clear from the interface, which may be why ui/gfx/ has
the wrong behavior.

BUG=chromium:564266

Committed:
https://skia.googlesource.com/skia/+/2b1516fe35155e05331ca6d4f09be916dc1fd778
==========

[commit-bot: I haz the power, 2015-12-03 20:12:17]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/2b1516fe35155e05331ca6d4f09be916dc1fd778