Remove DistilledPageInfo and related structs.

components/dom_distiller/core/distiller_page.cc

 // Copyright 2013 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/distiller_page.h"
 
 #include "base/bind.h"
 #include "base/json/json_writer.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 32 matching lines @@
   DCHECK_NE(std::string::npos, options_offset);
   DCHECK_EQ(std::string::npos,
             script.find(kOptionsPlaceholder, options_offset + 1));
   script =
       script.replace(options_offset, strlen(kOptionsPlaceholder), options_json);
   return script;
 }
 
 }
 
-DistilledPageInfo::DistilledPageInfo() {}
-
-DistilledPageInfo::~DistilledPageInfo() {}
-
-DistilledPageInfo::MarkupArticle::MarkupArticle() {}
-
-DistilledPageInfo::MarkupArticle::~MarkupArticle() {}
-
-DistilledPageInfo::MarkupImage::MarkupImage() {}
-
-DistilledPageInfo::MarkupImage::~MarkupImage() {}
-
-DistilledPageInfo::MarkupInfo::MarkupInfo() {}
-
-DistilledPageInfo::MarkupInfo::~MarkupInfo() {}
-
 DistillerPageFactory::~DistillerPageFactory() {}
 
 DistillerPage::DistillerPage() : ready_(true) {}
 
 DistillerPage::~DistillerPage() {}
 
 void DistillerPage::DistillPage(
     const GURL& gurl,
     const dom_distiller::proto::DomDistillerOptions options,
     const DistillerPageCallback& callback) {
   DCHECK(ready_);
   // It is only possible to distill one page at a time. |ready_| is reset when
   // the callback to OnDistillationDone happens.
   ready_ = false;
   distiller_page_callback_ = callback;
   DistillPageImpl(gurl, GetDistillerScriptWithOptions(options));
 }
 
 void DistillerPage::OnDistillationDone(const GURL& page_url,
                                        const base::Value* value) {
   DCHECK(!ready_);
   ready_ = true;
 
-  scoped_ptr<DistilledPageInfo> page_info(new DistilledPageInfo());
-  bool found_content = !value->IsType(base::Value::TYPE_NULL);
-  if (found_content) {
-    dom_distiller::proto::DomDistillerResult distiller_result =
-        dom_distiller::proto::json::DomDistillerResult::ReadFromValue(value);
+  scoped_ptr<dom_distiller::proto::DomDistillerResult> distiller_result;
 
-    page_info->title = distiller_result.title();
-    page_info->html = distiller_result.distilled_content().html();
-    page_info->next_page_url = distiller_result.pagination_info().next_page();
-    page_info->prev_page_url = distiller_result.pagination_info().prev_page();
-    for (int i = 0; i < distiller_result.image_urls_size(); ++i) {
-      const std::string image_url = distiller_result.image_urls(i);
-      if (GURL(image_url).is_valid()) {
-        page_info->image_urls.push_back(image_url);
-      }
-    }
-    const dom_distiller::proto::MarkupInfo& src_markup_info =
-        distiller_result.markup_info();
-    DistilledPageInfo::MarkupInfo& dst_markup_info = page_info->markup_info;
-    dst_markup_info.title = src_markup_info.title();
-    dst_markup_info.type = src_markup_info.type();
-    dst_markup_info.url = src_markup_info.url();
-    dst_markup_info.description = src_markup_info.description();
-    dst_markup_info.publisher = src_markup_info.publisher();
-    dst_markup_info.copyright = src_markup_info.copyright();
-    dst_markup_info.author = src_markup_info.author();
-
-    const dom_distiller::proto::MarkupArticle& src_article =
-        src_markup_info.article();
-    DistilledPageInfo::MarkupArticle& dst_article = dst_markup_info.article;
-    dst_article.published_time = src_article.published_time();
-    dst_article.modified_time = src_article.modified_time();
-    dst_article.expiration_time = src_article.expiration_time();
-    dst_article.section = src_article.section();
-    for (int i = 0; i < src_article.authors_size(); ++i) {
-      dst_article.authors.push_back(src_article.authors(i));
-    }
-
-    for (int i = 0; i < src_markup_info.images_size(); ++i) {
-      const dom_distiller::proto::MarkupImage& src_image =
-          src_markup_info.images(i);
-      DistilledPageInfo::MarkupImage dst_image;
-      dst_image.url = src_image.url();
-      dst_image.secure_url = src_image.secure_url();
-      dst_image.type = src_image.type();
-      dst_image.caption = src_image.caption();
-      dst_image.width = src_image.width();
-      dst_image.height = src_image.height();
-      dst_markup_info.images.push_back(dst_image);
-    }
-    if (distiller_result.has_timing_info()) {
+  bool found_content;
+  if (value->IsType(base::Value::TYPE_NULL)) {
+    found_content = false;
+  } else {
+    found_content = true;
+    distiller_result.reset(new dom_distiller::proto::DomDistillerResult(
+        dom_distiller::proto::json::DomDistillerResult::ReadFromValue(value)));
+    if (distiller_result->has_timing_info()) {
       const dom_distiller::proto::TimingInfo& timing =
-          distiller_result.timing_info();
+          distiller_result->timing_info();
       if (timing.has_markup_parsing_time()) {
         UMA_HISTOGRAM_TIMES(
             "DomDistiller.Time.MarkupParsing",
             base::TimeDelta::FromMillisecondsD(timing.markup_parsing_time()));
       }
       if (timing.has_document_construction_time()) {
         UMA_HISTOGRAM_TIMES(
             "DomDistiller.Time.DocumentConstruction",
             base::TimeDelta::FromMillisecondsD(
                 timing.document_construction_time()));
@@ skipping 12 matching lines @@
       if (timing.has_total_time()) {
         UMA_HISTOGRAM_TIMES(
             "DomDistiller.Time.DistillationTotal",
             base::TimeDelta::FromMillisecondsD(timing.total_time()));
       }
     }
   }
 
   base::MessageLoop::current()->PostTask(
       FROM_HERE,
-      base::Bind(
-          distiller_page_callback_, base::Passed(&page_info), found_content));
+      base::Bind(distiller_page_callback_,
+                 base::Passed(&distiller_result),
+                 found_content));
 }
 
 }  // namespace dom_distiller