Font Family Preferences for Distilled Pages

components/dom_distiller/content/resources/dom_distiller_viewer.js

 // 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.
 
 function addToPage(html) {
   var div = document.createElement('div');
   div.innerHTML = html;
   document.getElementById('content').appendChild(div);
 }
 
 function showLoadingIndicator(isLastPage) {
   document.getElementById('loadingIndicator').className =
       isLastPage ? 'hidden' : 'visible';
 }
 
+// Maps JS Font Family to CSS class and then changes body class name.
+// CSS classes must agree with distilledpage.css.
+function useFontFamily(fontFamily){

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

Space between ) and {

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

Done.

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

Done.

+  var cssClass;
+  if (fontFamily == "serif") {
+    cssClass = "serif";
+  } else if (fontFamily == "monospace") {
+    cssClass = "monospace";
+  } else {
+    cssClass = "sans-serif";
+  }
+  // Relies on the classname order of the body being Theme class, then Font
+  // Family class.
+  var themeClass = document.body.className.split(" ")[0];
+  cssClass = themeClass + " " + cssClass;
+  document.body.className = cssClass;
+}
+
 // Maps JS theme to CSS class and then changes body class name.
 // CSS classes must agree with distilledpage.css.
 function useTheme(theme) {
   var cssClass;
   if (theme == "sepia") {
     cssClass = "sepia";
   } else if (theme == "dark") {
     cssClass = "dark";
   } else {
     cssClass = "light";
   }
+  // Relies on the classname order of the body being Theme class, then Font

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

Any way to do these where we don't rely on order?

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

explored other options with nyquist@
conclusion: leave as is
On 2014/08/10 02:43:45, robliao wrote:

+  // Family class.
+  var fontFamilyClass = document.body.className.split(" ")[1];
+  cssClass = cssClass + " " + fontFamilyClass;
   document.body.className = cssClass;
 }