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

Unified Diff: chrome/browser/ui/android/infobars/translate_compact_infobar.cc

Issue 2874393002: Automatically trigger Never-Translate in new translation infobar (Closed)
Patch Set: fix Created 3 years, 7 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: chrome/browser/ui/android/infobars/translate_compact_infobar.cc
diff --git a/chrome/browser/ui/android/infobars/translate_compact_infobar.cc b/chrome/browser/ui/android/infobars/translate_compact_infobar.cc
index 94a729fa268f915958d2e6a02ac3a6c1236e8ccf..8e650301ed6de8f20487c418743cfd74bee21cdc 100644
--- a/chrome/browser/ui/android/infobars/translate_compact_infobar.cc
+++ b/chrome/browser/ui/android/infobars/translate_compact_infobar.cc
@@ -65,8 +65,7 @@ void TranslateCompactInfoBar::ProcessButton(int action) {
translate::TranslateInfoBarDelegate* delegate = GetDelegate();
if (action == InfoBarAndroid::ACTION_TRANSLATE) {
delegate->Translate();
- if (!delegate->ShouldAlwaysTranslate() &&
- delegate->ShouldAutoAlwaysTranslate()) {
+ if (!delegate->ShouldAlwaysTranslate() && ShouldAutoAlwaysTranslate()) {
JNIEnv* env = base::android::AttachCurrentThread();
Java_TranslateCompactInfoBar_setAutoAlwaysTranslate(env,
GetJavaInfoBar());
@@ -147,7 +146,14 @@ void TranslateCompactInfoBar::OnPageTranslated(
bool TranslateCompactInfoBar::ShouldAutoAlwaysTranslate() {
translate::TranslateInfoBarDelegate* delegate = GetDelegate();
- return delegate->ShouldAutoAlwaysTranslate();
+ return (delegate->GetTranslationAcceptedCount() == kAcceptCountThreshold);
+}
+
+jboolean TranslateCompactInfoBar::ShouldAutoNeverTranslate(
+ JNIEnv* env,
+ const base::android::JavaParamRef<jobject>& obj) {
+ translate::TranslateInfoBarDelegate* delegate = GetDelegate();
+ return (delegate->GetTranslationDeniedCount() == kDeniedCountThreshold);
}
translate::TranslateInfoBarDelegate* TranslateCompactInfoBar::GetDelegate() {

Powered by Google App Engine
This is Rietveld 408576698