Make AdaBoost the default heuristic for ReaderModeUI finch

components/dom_distiller/core/experiments.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 "components/dom_distiller/core/experiments.h"
 
 #include "base/command_line.h"
 #include "base/metrics/field_trial.h"
 #include "components/dom_distiller/core/dom_distiller_switches.h"
 
@@ skipping 20 matching lines @@
     }
     NOTREACHED() << "Invalid value for " << switches::kReaderModeHeuristics;
   } else {
     if (group_name == "AdaBoost") {
       return DistillerHeuristicsType::ADABOOST_MODEL;
     }
     if (group_name == "OGArticle") {
       return DistillerHeuristicsType::OG_ARTICLE;
     }
   }
-  return DistillerHeuristicsType::NONE;
+  return DistillerHeuristicsType::ADABOOST_MODEL;

[mdjones, 2016/08/05 16:59:43]

Is this going to skew our groups?

[wychen, 2016/08/06 07:03:22]

This is meant to be landed after we decide to roll to 100%.
Sorry for the confusion.

With the disabled group (patch set 3), as long as we name all the non-exp groups
Disabled*, the behavior should be the same.

[wychen, 2016/08/06 21:55:09]

OK, this is also wrong. Some users don't get finch signals, so this would need
to wait for 100% stable approval.

 }
 
 bool ShouldShowFeedbackForm() {
   const std::string group_name =
       base::FieldTrialList::FindFullName("ReaderModeUIFeedback");
   const std::string switch_value =
       base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
           switches::kReaderModeFeedback);
   if (switch_value != "") {
     if (switch_value == switches::reader_mode_feedback::kOn) {
       return true;
     }
     if (switch_value == switches::reader_mode_feedback::kOff) {
       return false;
     }
     NOTREACHED() << "Invalid value for " << switches::kReaderModeFeedback;
   } else {
     if (group_name == "DoNotShow") {
       return false;
     }
     if (group_name == "Show") {
       return true;
     }
   }
   return false;
 }
 }