Font Family Preferences for Distilled Pages

components/dom_distiller/core/viewer_unittest.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 "components/dom_distiller/core/viewer.h"
 
 #include "components/dom_distiller/core/distilled_page_prefs.h"
 #include "components/dom_distiller/core/dom_distiller_service.h"
 #include "components/dom_distiller/core/dom_distiller_test_util.h"
 #include "components/dom_distiller/core/task_tracker.h"
@@ skipping 133 matching lines @@
                 DistilledPagePrefs::DARK)),
             0);
   EXPECT_EQ(kLightJs.compare(viewer::GetDistilledPageThemeJs(
                 DistilledPagePrefs::LIGHT)),
             0);
   EXPECT_EQ(kSepiaJs.compare(viewer::GetDistilledPageThemeJs(
                 DistilledPagePrefs::SEPIA)),
             0);
 }
 
+TEST_F(DomDistillerViewerTest, TestGetDistilledPageFontFamilyJsOutput) {
+  std::string kSerifJsFontFamily = "useFontFamily('serif');";
+  std::string kMonospaceJsFontFamily = "useFontFamily('monospace');";
+  std::string kSansSerifJsFontFamily = "useFontFamily('sans-serif');";
+  EXPECT_EQ(kSerifJsFontFamily.compare(viewer::GetDistilledPageFontFamilyJs(

[robliao, 2014/08/10 02:43:46]

Go ahead and linebreak after the first (.

[sunangel, 2014/08/11 21:51:33]

Done and did it for below for consistency.
On 2014/08/10 02:43:46, robliao wrote:

+                DistilledPagePrefs::SERIF)),
+            0);
+  EXPECT_EQ(kMonospaceJsFontFamily.compare(viewer::GetDistilledPageFontFamilyJs(
+                DistilledPagePrefs::MONOSPACE)),
+            0);
+  EXPECT_EQ(kSansSerifJsFontFamily.compare(viewer::GetDistilledPageFontFamilyJs(
+                DistilledPagePrefs::SANS_SERIF)),
+            0);
+}
+
 }  // namespace dom_distiller