Configure DomDistiller debug output.

components/dom_distiller/standalone/content_extractor.cc

 // 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.
 
 #include <sstream>
 
 #include "base/command_line.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/message_loop/message_loop.h"
 #include "base/path_service.h"
 #include "base/run_loop.h"
+#include "base/strings/string_number_conversions.h"
 #include "components/dom_distiller/content/distiller_page_web_contents.h"
 #include "components/dom_distiller/core/distiller.h"
 #include "components/dom_distiller/core/dom_distiller_database.h"
 #include "components/dom_distiller/core/dom_distiller_service.h"
 #include "components/dom_distiller/core/dom_distiller_store.h"
 #include "components/dom_distiller/core/proto/distilled_article.pb.h"
 #include "components/dom_distiller/core/proto/distilled_page.pb.h"
 #include "components/dom_distiller/core/task_tracker.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
@@ skipping 15 matching lines @@
 // Indicates that DNS resolution should be disabled for this test.
 const char* kDisableDnsSwitch = "disable-dns";
 
 // Will write the distilled output to the given file instead of to stdout.
 const char* kOutputFile = "output-file";
 
 // Indicates to output a serialized protocol buffer instead of human-readable
 // output.
 const char* kShouldOutputBinary = "output-binary";
 
 const char* kExtractTextOnly = "extract-text-only";

[nyquist, 2014/05/29 17:50:41]

Nit: Totally unrelated, but could you add a comment to this one too? Maybe we
can do it in the future.

[Yaron, 2014/05/29 20:16:30]

Done.

 
+// Indicates to include debug output.
+const char* kDebugLevel = "debug-level";
+
 scoped_ptr<DomDistillerService> CreateDomDistillerService(
     content::BrowserContext* context,
     const base::FilePath& db_path) {
   scoped_refptr<base::SequencedTaskRunner> background_task_runner =
       content::BrowserThread::GetBlockingPool()->GetSequencedTaskRunner(
           content::BrowserThread::GetBlockingPool()->GetSequenceToken());
 
   // TODO(cjhopman): use an in-memory database instead of an on-disk one with
   // temporary directory.
   scoped_ptr<DomDistillerDatabase> db(
       new DomDistillerDatabase(background_task_runner));
   scoped_ptr<DomDistillerStore> dom_distiller_store(new DomDistillerStore(
       db.PassAs<DomDistillerDatabaseInterface>(), db_path));
 
   scoped_ptr<DistillerPageFactory> distiller_page_factory(
       new DistillerPageWebContentsFactory(context));
   scoped_ptr<DistillerURLFetcherFactory> distiller_url_fetcher_factory(
       new DistillerURLFetcherFactory(context->GetRequestContext()));
 
   dom_distiller::proto::DomDistillerOptions options;
   if (base::CommandLine::ForCurrentProcess()->HasSwitch(kExtractTextOnly)) {
     options.set_extract_text_only(true);
   }
+  int debug_level = 0;
+  if (base::CommandLine::ForCurrentProcess()->HasSwitch(kDebugLevel) &&
+      base::StringToInt(
+          base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
+              kDebugLevel),
+          &debug_level)) {
+    options.set_debug_level(debug_level);
+  }
   scoped_ptr<DistillerFactory> distiller_factory(
       new DistillerFactoryImpl(distiller_url_fetcher_factory.Pass(), options));
 
   return scoped_ptr<DomDistillerService>(new DomDistillerService(
       dom_distiller_store.PassAs<DomDistillerStoreInterface>(),
       distiller_factory.Pass(),
       distiller_page_factory.Pass()));
 }
 
 void AddComponentsResources() {
@@ skipping 139 matching lines @@
   scoped_ptr<DomDistillerService> service_;
   scoped_ptr<ContentExtractionRequest> request_;
 };
 
 IN_PROC_BROWSER_TEST_F(ContentExtractor, MANUAL_ExtractUrl) {
   Start();
   base::RunLoop().Run();
 }
 
 }  // namespace dom_distiller