Remove FontRenderParams NOTIMPLEMENTED()s.

ui/gfx/font_render_params_mac.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/gfx/font_render_params.h"
 
 #include "base/logging.h"
 #include "base/macros.h"
 
 namespace gfx {
@@ skipping 10 matching lines @@
   params.subpixel_positioning = true;
   params.hinting = FontRenderParams::HINTING_MEDIUM;
 
   return params;
 }
 
 }  // namespace
 
 FontRenderParams GetFontRenderParams(const FontRenderParamsQuery& query,
                                      std::string* 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 two:
// Customized font rendering settings are not supported, only defaults.

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

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?

-  if (!query.is_empty() || family_out)
-    NOTIMPLEMENTED();
   CR_DEFINE_STATIC_LOCAL(const gfx::FontRenderParams, params, (LoadDefaults()));
   return params;
 }
 
 }  // namespace gfx