Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(48)

Unified Diff: components/dom_distiller/core/task_tracker.cc

Issue 178303004: Add incremental updates for multipage distillation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix compile by adding a header. Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/dom_distiller/core/task_tracker.cc
diff --git a/components/dom_distiller/core/task_tracker.cc b/components/dom_distiller/core/task_tracker.cc
index 1afd4825ac75dc96777a41aafa80f4f2e9cc39da..4d2afd76eae1b68cba1a12ead7b2cef1f80a6e74 100644
--- a/components/dom_distiller/core/task_tracker.cc
+++ b/components/dom_distiller/core/task_tracker.cc
@@ -41,7 +41,9 @@ void TaskTracker::StartDistiller(DistillerFactory* factory) {
distiller_ = factory->CreateDistiller();
distiller_->DistillPage(url,
- base::Bind(&TaskTracker::OnDistilledDataReady,
+ base::Bind(&TaskTracker::OnDistilledArticleReady,
+ weak_ptr_factory_.GetWeakPtr()),
+ base::Bind(&TaskTracker::OnArticleDistillationUpdated,
weak_ptr_factory_.GetWeakPtr()));
}
@@ -139,7 +141,7 @@ void TaskTracker::NotifyViewer(ViewRequestDelegate* delegate) {
delegate->OnArticleReady(distilled_article_.get());
}
-void TaskTracker::OnDistilledDataReady(
+void TaskTracker::OnDistilledArticleReady(
scoped_ptr<DistilledArticleProto> distilled_article) {
distilled_article_ = distilled_article.Pass();
bool distillation_successful = false;
@@ -164,4 +166,11 @@ void TaskTracker::OnDistilledDataReady(
DoSaveCallbacks(distillation_successful);
}
+void TaskTracker::OnArticleDistillationUpdated(
+ const ArticleDistillationUpdate& article_update) {
+ for (size_t i = 0; i < viewers_.size(); ++i) {
+ viewers_[i]->OnArticleUpdated(article_update);
+ }
+}
+
} // namespace dom_distiller

Powered by Google App Engine
This is Rietveld 408576698