Index: chrome/android/java/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java |
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java b/chrome/android/java/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java |
index 1d8fdd9706e3d79a27bc666935e6210eeaf936ea..e07027fb68e0ae5704858fc64a912d90a595d9bd 100644 |
--- a/chrome/android/java/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java |
+++ b/chrome/android/java/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java |
@@ -28,7 +28,6 @@ import android.view.MotionEvent; |
import android.view.View; |
import android.widget.FrameLayout; |
import android.widget.ImageButton; |
-import android.widget.ImageView; |
import android.widget.TextView; |
import org.chromium.base.ApiCompatibilityUtils; |
@@ -52,9 +51,11 @@ import org.chromium.chrome.browser.tab.Tab; |
import org.chromium.chrome.browser.toolbar.ActionModeController.ActionBarDelegate; |
import org.chromium.chrome.browser.util.ColorUtils; |
import org.chromium.chrome.browser.widget.TintedDrawable; |
+import org.chromium.chrome.browser.widget.TintedImageButton; |
import org.chromium.components.dom_distiller.core.DomDistillerService; |
import org.chromium.components.dom_distiller.core.DomDistillerUrlUtils; |
import org.chromium.components.security_state.ConnectionSecurityLevel; |
+import org.chromium.ui.base.DeviceFormFactor; |
import org.chromium.ui.base.WindowAndroid; |
import org.chromium.ui.interpolators.BakedBezierInterpolator; |
import org.chromium.ui.widget.Toast; |
@@ -93,7 +94,7 @@ public class CustomTabToolbar extends ToolbarLayout implements LocationBar, |
private View mTitleUrlContainer; |
private UrlBar mUrlBar; |
private TextView mTitleBar; |
- private ImageView mSecurityButton; |
+ private TintedImageButton mSecurityButton; |
private ImageButton mCustomActionButton; |
private int mSecurityIconType; |
private ImageButton mCloseButton; |
@@ -137,7 +138,7 @@ public class CustomTabToolbar extends ToolbarLayout implements LocationBar, |
mLocationBarFrameLayout = findViewById(R.id.location_bar_frame_layout); |
mTitleUrlContainer = findViewById(R.id.title_url_container); |
mTitleUrlContainer.setOnLongClickListener(this); |
- mSecurityButton = (ImageButton) findViewById(R.id.security_button); |
+ mSecurityButton = (TintedImageButton) findViewById(R.id.security_button); |
mSecurityIconType = ConnectionSecurityLevel.NONE; |
mCustomActionButton = (ImageButton) findViewById(R.id.action_button); |
mCustomActionButton.setOnLongClickListener(this); |
@@ -232,11 +233,8 @@ public class CustomTabToolbar extends ToolbarLayout implements LocationBar, |
@Override |
public boolean shouldEmphasizeHttpsScheme() { |
int securityLevel = getSecurityLevel(); |
- if (securityLevel == ConnectionSecurityLevel.SECURITY_ERROR |
- || securityLevel == ConnectionSecurityLevel.SECURITY_POLICY_WARNING) { |
- return true; |
- } |
- return false; |
+ return securityLevel == ConnectionSecurityLevel.SECURITY_ERROR |
+ || securityLevel == ConnectionSecurityLevel.SECURITY_POLICY_WARNING; |
} |
@Override |
@@ -482,13 +480,15 @@ public class CustomTabToolbar extends ToolbarLayout implements LocationBar, |
if (securityLevel == ConnectionSecurityLevel.NONE) { |
mAnimDelegate.hideSecurityButton(); |
} else { |
- int id = LocationBarLayout.getSecurityIconResource( |
- securityLevel, !shouldEmphasizeHttpsScheme()); |
- // ImageView#setImageResource is no-op if given resource is the current one. |
+ boolean isSmallDevice = !DeviceFormFactor.isTablet(getContext()); |
+ int id = LocationBarLayout.getSecurityIconResource(securityLevel, isSmallDevice); |
if (id == 0) { |
mSecurityButton.setImageDrawable(null); |
} else { |
+ // ImageView#setImageResource is no-op if given resource is the current one. |
mSecurityButton.setImageResource(id); |
+ mSecurityButton.setTint(LocationBarLayout.getColorStateList( |
+ securityLevel, getToolbarDataProvider(), getResources())); |
} |
mAnimDelegate.showSecurityButton(); |
} |