Add flag "distillability-dev" for distillability development

third_party/WebKit/Source/core/dom/DocumentStatisticsCollector.cpp

 // 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 "DocumentStatisticsCollector.h"
 
 #include "core/HTMLNames.h"
 #include "core/InputTypeNames.h"
 #include "core/dom/ElementTraversal.h"
 #include "core/dom/NodeComputedStyle.h"
 #include "core/dom/Text.h"
 #include "core/frame/FrameHost.h"
 #include "core/html/HTMLHeadElement.h"
 #include "core/html/HTMLInputElement.h"
 #include "core/html/HTMLMetaElement.h"
+#include "core/inspector/ConsoleMessage.h"
 #include "platform/Histogram.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebDistillability.h"
 
 namespace blink {
 
 using namespace HTMLNames;
 
 namespace {
 
@@ skipping 180 matching lines @@
 
 bool isMobileFriendly(Document& document)
 {
     if (FrameHost* frameHost = document.frameHost())
         return frameHost->visualViewport().shouldDisableDesktopWorkarounds();
     return false;
 }
 
 } // namespace
 
-WebDistillabilityFeatures DocumentStatisticsCollector::collectStatistics(Document& document)
+WebDistillabilityFeatures DocumentStatisticsCollector::collectStatistics(Document& document, bool dumpInfo)
 {
     TRACE_EVENT0("blink", "DocumentStatisticsCollector::collectStatistics");
 
     WebDistillabilityFeatures features = WebDistillabilityFeatures();
 
     if (!document.frame() || !document.frame()->isMainFrame())
         return features;
 
     DCHECK(document.hasFinishedParsing());
 
@@ skipping 12 matching lines @@
 
     // Traverse the DOM tree and collect statistics.
     collectFeatures(*body, features);
     features.openGraph = hasOpenGraphArticle(*head);
 
     double elapsedTime = monotonicallyIncreasingTime() - startTime;
 
     DEFINE_STATIC_LOCAL(CustomCountHistogram, distillabilityHistogram, ("WebCore.DistillabilityUs", 1, 1000000, 50));
     distillabilityHistogram.count(static_cast<int>(1e6 * elapsedTime));
 
+    if (dumpInfo) {
+        StringBuilder message;
+        message.append("distillability_features = {\"isMobileFriendly\": ");
+        message.appendNumber(features.isMobileFriendly);
+        message.append(", \"openGraph\": ");
+        message.appendNumber(features.openGraph);
+        message.append(", \"elementCount\": ");
+        message.appendNumber(features.elementCount);
+        message.append(", \"anchorCount\": ");
+        message.appendNumber(features.anchorCount);
+        message.append(", \"formCount\": ");
+        message.appendNumber(features.formCount);
+        message.append(", \"textInputCount\": ");
+        message.appendNumber(features.textInputCount);
+        message.append(", \"passwordInputCount\": ");
+        message.appendNumber(features.passwordInputCount);
+        message.append(", \"mozScore\": ");
+        message.appendNumber(features.mozScore);
+        message.append(", \"mozScoreAllSqrt\": ");
+        message.appendNumber(features.mozScoreAllSqrt);
+        message.append(", \"mozScoreAllLinear\": ");
+        message.appendNumber(features.mozScoreAllLinear);
+        message.append("}");
+        ConsoleMessage* consoleMessage = ConsoleMessage::create(ConsoleAPIMessageSource, DebugMessageLevel, message.toString());

[mdjones, 2016/05/18 19:06:59]

Is this automatically cleaned up (does ConsoleMessage or document own it)?

[wychen, 2016/05/18 19:19:01]

ConsoleMessage is derived from GarbageCollectedFinalized<ConsoleMessage>, so
it's automatically cleaned up.

+        document.addConsoleMessage(consoleMessage);
+    }
+
     return features;
 }
 
 } // namespace blink