Chromium Code Reviews| Index: ui/gfx/font_render_params_mac.cc |
| diff --git a/ui/gfx/font_render_params_mac.cc b/ui/gfx/font_render_params_mac.cc |
| index 0062f3b1aae2fcdcbc3eb53d970449a74cdcbe9c..83fd341e507ae290ccb55c29b1dbc476ca08723b 100644 |
| --- a/ui/gfx/font_render_params_mac.cc |
| +++ b/ui/gfx/font_render_params_mac.cc |
| @@ -28,9 +28,9 @@ FontRenderParams LoadDefaults() { |
| FontRenderParams GetFontRenderParams(const FontRenderParamsQuery& query, |
| std::string* family_out) { |
| - // TODO: Query the OS for font render settings instead of returning defaults. |
| - if (!query.is_empty() || family_out) |
| + if (family_out) |
| NOTIMPLEMENTED(); |
| + // TODO: Query the OS for font render settings instead of returning defaults. |
|
msw
2014/10/20 18:30:18
optional nit: make this comment match the other tw
Daniel Erat
2014/10/20 20:00:47
i actually don't know the background here, or the
|
| CR_DEFINE_STATIC_LOCAL(const gfx::FontRenderParams, params, (LoadDefaults())); |
| return params; |
| } |