Configure font font names in GFX unittests

ui/gfx/font_names_testing.cc

+// Copyright 2016 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_names_testing.h"
+
+#include "build/build_config.h"
+
+namespace gfx {
+
+#if defined(OS_LINUX)
+const char kSymbolFontName[] = "DejaVu Sans";
+#else
+const char kSymbolFontName[] = "Symbol";
+#endif
+
+#if defined(OS_LINUX)
+const char kCJKFontName[] = "Kochi Mincho";
+#elif defined(OS_MACOSX)
+const char kCJKFontName[] = "PingFang SC";
+#else
+const char kCJKFontName[] = "SimSun";
+#endif
+
+} // end namespace gfx

[Alexei Svitkine (slow), 2016/09/06 19:48:34]

This comment should just be "// namespace". You also need two spaces before the
// per the style guide.

[msw, 2016/09/06 20:06:06]

Actually, shouldn't it still say gfx? Like: "}  // namespace gfx"

[Alexei Svitkine (slow), 2016/09/06 21:06:31]

You are correct, indeed it should be "// namespace gfx". Sorry for the
confusion.