Remove FontRenderParams NOTIMPLEMENTED()s.

Remove FontRenderParams NOTIMPLEMENTED()s.

Don't spam the logs on Windows, Mac, or Android when
GetFontRenderParams() is called with a non-empty query;
RenderTextHarfBuzz sends non-empty queries and is being
enabled for more platforms now.

Continue to log an error if a family name is requested on
these platforms, though.

BUG=424971
Committed: https://crrev.com/1f6cb5cdf9edd7680c8bef6722482d090354627f
Cr-Commit-Position: refs/heads/master@{#300339}

[Daniel Erat, 2014-10-20 14:04:17]

derat@chromium.org changed reviewers:
+ ckocagil@chromium.org, mgiuca@chromium.org, msw@chromium.org

[msw, 2014-10-20 18:30:18]

lgtm with an optional nit

https://codereview.chromium.org/645073004/diff/1/ui/gfx/font_render_params_ma...
File ui/gfx/font_render_params_mac.cc (right):

https://codereview.chromium.org/645073004/diff/1/ui/gfx/font_render_params_ma...
ui/gfx/font_render_params_mac.cc:33: // TODO: Query the OS for font render
settings instead of returning defaults.
optional nit: make this comment match the other two:
// Customized font rendering settings are not supported, only defaults.

[Daniel Erat, 2014-10-20 20:00:47]

https://codereview.chromium.org/645073004/diff/1/ui/gfx/font_render_params_ma...
File ui/gfx/font_render_params_mac.cc (right):

https://codereview.chromium.org/645073004/diff/1/ui/gfx/font_render_params_ma...
ui/gfx/font_render_params_mac.cc:33: // TODO: Query the OS for font render
settings instead of returning defaults.
On 2014/10/20 18:30:18, msw wrote:
> optional nit: make this comment match the other two:
> // Customized font rendering settings are not supported, only defaults.

i actually don't know the background here, or the degree to which font rendering
is configurable in os x. cem, i think that you added this TODO -- is it still
relevant?

[Daniel Erat, 2014-10-20 20:00:50]

The CQ bit was checked by derat@chromium.org

[commit-bot: I haz the power, 2014-10-20 20:01:32]

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

[commit-bot: I haz the power, 2014-10-20 21:48:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-10-20 21:48:49]

Patchset 1 (id:??) landed as
https://crrev.com/1f6cb5cdf9edd7680c8bef6722482d090354627f
Cr-Commit-Position: refs/heads/master@{#300339}