Support blacklisting domains for Reader Mode

components/dom_distiller/content/renderer/distillability_agent.cc

 // Copyright 2015 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 "base/metrics/histogram.h"
 
 #include "components/dom_distiller/content/common/distiller_messages.h"
 #include "components/dom_distiller/content/renderer/distillability_agent.h"
 #include "components/dom_distiller/core/distillable_page_detector.h"
 #include "components/dom_distiller/core/experiments.h"
@@ skipping 32 matching lines @@
 }
 
 // Returns whether this update is the last one for the page.
 bool IsLast(bool is_loaded) {
   if (GetDistillerHeuristicsType() == DistillerHeuristicsType::ADABOOST_MODEL)
     return 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) {
   WebDistillabilityFeatures features = doc.distillabilityFeatures();
   GURL parsed_url(doc.url());
   if (!parsed_url.is_valid()) {
     return false;
   }
   bool distillable = detector->Classify(CalculateDerivedFeatures(
     features.openGraph,
     parsed_url,
     features.elementCount,
     features.anchorCount,
     features.formCount,
     features.mozScore,
     features.mozScoreAllSqrt,
     features.mozScoreAllLinear
   ));
+  bool blacklisted = IsBlacklisted(parsed_url);
 
   int bucket = static_cast<unsigned>(features.isMobileFriendly) |
       (static_cast<unsigned>(distillable) << 1);
   if (is_last) {
     UMA_HISTOGRAM_ENUMERATION("DomDistiller.PageDistillableAfterLoading",
         bucket, 4);
   } else {
     UMA_HISTOGRAM_ENUMERATION("DomDistiller.PageDistillableAfterParsing",
         bucket, 4);
   }
-  return distillable && (!features.isMobileFriendly);
+
+  return distillable && (!features.isMobileFriendly) && (!blacklisted);
 }
 
 bool IsDistillablePage(WebDocument& doc, bool is_last) {
   switch (GetDistillerHeuristicsType()) {
     case DistillerHeuristicsType::ALWAYS_TRUE:
       return true;
     case DistillerHeuristicsType::OG_ARTICLE:
       return doc.distillabilityFeatures().openGraph;
     case DistillerHeuristicsType::ADABOOST_MODEL:
       return IsDistillablePageAdaboost(
@@ skipping 30 matching lines @@
 
   bool is_last = IsLast(is_loaded);
   Send(new FrameHostMsg_Distillability(routing_id(),
       IsDistillablePage(doc, is_last), is_last));
 }
 
 
 DistillabilityAgent::~DistillabilityAgent() {}
 
 }  // namespace dom_distiller