Support blacklisting domains for Reader Mode

components/dom_distiller/content/renderer/distillability_agent.cc

Index: components/dom_distiller/content/renderer/distillability_agent.cc
diff --git a/components/dom_distiller/content/renderer/distillability_agent.cc b/components/dom_distiller/content/renderer/distillability_agent.cc
index 99592ef0b6fef51c3ca712a6de556bd29d3c8e33..e50ef666475d2d15a61e905f376a10b708f8df1f 100644
--- a/components/dom_distiller/content/renderer/distillability_agent.cc
+++ b/components/dom_distiller/content/renderer/distillability_agent.cc
@@ -50,6 +50,19 @@ bool IsLast(bool is_loaded) {
   return true;
 }
 
+const char* const kBlacklist[] = {

[mdjones, 2016/02/18 17:13:27]

Being a constant, this could probably be at the top of the file/namespace. I
don't feel too strongly about this though.

+  "www.reddit.com"
+};
+
+bool IsBlacklisted(const GURL& url) {
+  for (size_t i = 0; i < arraysize(kBlacklist); ++i) {
+    if (base::LowerCaseEqualsASCII(url.host(), kBlacklist[i])) {
+      return true;
+    }
+  }
+  return false;
+}
+
 bool IsDistillablePageAdaboost(WebDocument& doc,
                                const DistillablePageDetector* detector,
                                bool is_last) {
@@ -68,6 +81,7 @@ bool IsDistillablePageAdaboost(WebDocument& doc,
     features.mozScoreAllSqrt,
     features.mozScoreAllLinear
   ));
+  bool blacklisted = IsBlacklisted(parsed_url);
 
   int bucket = static_cast<unsigned>(features.isMobileFriendly) |
       (static_cast<unsigned>(distillable) << 1);
@@ -78,7 +92,8 @@ bool IsDistillablePageAdaboost(WebDocument& doc,
     UMA_HISTOGRAM_ENUMERATION("DomDistiller.PageDistillableAfterParsing",
         bucket, 4);
   }
-  return distillable && (!features.isMobileFriendly);
+
+  return distillable && (!features.isMobileFriendly) && (!blacklisted);
 }
 
 bool IsDistillablePage(WebDocument& doc, bool is_last) {