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

Side by Side Diff: components/translate/core/browser/mock_translate_ranker.cc

Issue 2697703004: Allow TranslateRanker to override decisions taken by heuristics. (Closed)
Patch Set: Merge & Cleanup commented code. 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 unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/translate/core/browser/mock_translate_ranker.h" 5 #include "components/translate/core/browser/mock_translate_ranker.h"
6 6
7 #include "components/metrics/proto/translate_event.pb.h" 7 #include "components/metrics/proto/translate_event.pb.h"
8 #include "url/gurl.h" 8 #include "url/gurl.h"
9 9
10 namespace translate { 10 namespace translate {
11 namespace testing { 11 namespace testing {
12 12
13 MockTranslateRanker::MockTranslateRanker() {} 13 MockTranslateRanker::MockTranslateRanker() {}
14 14
15 MockTranslateRanker::~MockTranslateRanker() {} 15 MockTranslateRanker::~MockTranslateRanker() {}
16 16
17 bool MockTranslateRanker::IsLoggingEnabled() { 17 uint32_t MockTranslateRanker::GetModelVersion() const {
18 return is_logging_enabled_;
19 }
20
21 bool MockTranslateRanker::IsQueryEnabled() {
22 return is_query_enabled_;
23 }
24
25 bool MockTranslateRanker::IsEnforcementEnabled() {
26 return is_enforcement_enabled_;
27 }
28
29 int MockTranslateRanker::GetModelVersion() const {
30 return model_version_; 18 return model_version_;
31 } 19 }
32 20
33 bool MockTranslateRanker::ShouldOfferTranslation( 21 bool MockTranslateRanker::ShouldOfferTranslation(
34 const TranslatePrefs& /* translate_prefs */, 22 const TranslatePrefs& /* translate_prefs */,
35 const std::string& /* src_lang */, 23 const std::string& /* src_lang */,
36 const std::string& /* dst_lang */) { 24 const std::string& /* dst_lang */,
25 metrics::TranslateEventProto* /*translate_event */) {
37 return should_offer_translation_; 26 return should_offer_translation_;
38 } 27 }
39 28
40 void MockTranslateRanker::AddTranslateEvent(
41 const metrics::TranslateEventProto& translate_event,
42 const GURL& /* url */) {
43 event_cache_.push_back(translate_event);
44 }
45
46 void MockTranslateRanker::FlushTranslateEvents( 29 void MockTranslateRanker::FlushTranslateEvents(
47 std::vector<metrics::TranslateEventProto>* events) { 30 std::vector<metrics::TranslateEventProto>* events) {
48 events->swap(event_cache_); 31 events->swap(event_cache_);
49 event_cache_.clear(); 32 event_cache_.clear();
50 } 33 }
51 34
52 } // namespace testing 35 } // namespace testing
53 } // namespace translate 36 } // namespace translate
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698