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

Unified Diff: components/translate/core/browser/translate_ranker_impl.cc

Issue 2839433002: [translate] Fix shutdown race for translate ranker model loader. (Closed)
Patch Set: experiment: delay model load to first translate activity Created 3 years, 8 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/translate/core/browser/translate_ranker_impl.cc
diff --git a/components/translate/core/browser/translate_ranker_impl.cc b/components/translate/core/browser/translate_ranker_impl.cc
index 906c498f964e219b6a57fecbed801ee529d65ae7..b25706f858230526de57fb164fdd8e8f94e77639 100644
--- a/components/translate/core/browser/translate_ranker_impl.cc
+++ b/components/translate/core/browser/translate_ranker_impl.cc
@@ -156,7 +156,6 @@ TranslateRankerImpl::TranslateRankerImpl(const base::FilePath& model_path,
base::Bind(&TranslateRankerImpl::OnModelAvailable,
weak_ptr_factory_.GetWeakPtr()),
model_path, model_url, kUmaPrefix);
- model_loader_->Start();
}
}

Powered by Google App Engine
This is Rietveld 408576698