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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/infobar/TranslateAlwaysPanel.java

Issue 378043002: Visual makeover for infobars. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: don't show buttons with empty text Created 6 years, 5 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/android/java/src/org/chromium/chrome/browser/infobar/TranslateAlwaysPanel.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/infobar/TranslateAlwaysPanel.java b/chrome/android/java/src/org/chromium/chrome/browser/infobar/TranslateAlwaysPanel.java
index e0d413b8df61787b16a19be8980c284795718408..a3de96451ab03e6edb277004068982385949f51b 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/infobar/TranslateAlwaysPanel.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/infobar/TranslateAlwaysPanel.java
@@ -4,7 +4,6 @@
package org.chromium.chrome.browser.infobar;
import android.content.Context;
-import android.widget.TextView;
import org.chromium.chrome.R;
@@ -23,16 +22,15 @@ public class TranslateAlwaysPanel implements TranslateSubPanel {
@Override
public void createContent(Context context, InfoBarLayout layout) {
- TextView panelMessage = (TextView) layout.findViewById(R.id.infobar_message);
- panelMessage.setText(context.getString(
+ layout.setMessage(context.getString(
R.string.translate_infobar_translation_done, mOptions.targetLanguage()));
if (!mOptions.triggeredFromMenu()) {
- TranslateCheckBox checkBox = new TranslateCheckBox(mOptions, mListener);
- checkBox.createContent(context, layout);
+ TranslateCheckBox checkBox = new TranslateCheckBox(context, mOptions, mListener);
+ layout.setCustomContent(checkBox);
}
- layout.addButtons(context.getString(R.string.translate_button_done),
+ layout.setButtons(context.getString(R.string.translate_button_done),
context.getString(R.string.translate_show_original));
}

Powered by Google App Engine
This is Rietveld 408576698