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

Unified Diff: chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java

Issue 1169213006: Make ConnectionSecurityHelper a namespace instead of a class (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 6 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_staging/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java
diff --git a/chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java b/chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java
index 7f66fcf691874edd094da5a94da16be0da64c9f4..4cabefb8b605e8cb706c40990a58f423bd73c1db 100644
--- a/chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java
+++ b/chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/CustomTabToolbar.java
@@ -44,7 +44,7 @@ import org.chromium.chrome.browser.omnibox.LocationBarLayout;
import org.chromium.chrome.browser.omnibox.UrlBar;
import org.chromium.chrome.browser.omnibox.UrlFocusChangeListener;
import org.chromium.chrome.browser.profiles.Profile;
-import org.chromium.chrome.browser.ssl.ConnectionSecurityHelperSecurityLevel;
+import org.chromium.chrome.browser.ssl.ConnectionSecurityStatusSecurityLevel;
import org.chromium.chrome.browser.tab.ChromeTab;
import org.chromium.chrome.browser.widget.TintedImageButton;
import org.chromium.components.dom_distiller.core.DomDistillerService;
@@ -84,7 +84,7 @@ public class CustomTabToolbar extends ToolbarLayout implements LocationBar {
mTitleBar = (TextView) findViewById(R.id.title_bar);
mUrlInfoContainer = findViewById(R.id.url_info_container);
mSecurityButton = (ImageButton) findViewById(R.id.security_button);
- mSecurityIconType = ConnectionSecurityHelperSecurityLevel.NONE;
+ mSecurityIconType = ConnectionSecurityStatusSecurityLevel.NONE;
mCustomActionButton = (ImageButton) findViewById(R.id.action_button);
mReturnButton = (TintedImageButton) findViewById(R.id.back_button);
mSecurityButtonShowAnimator = ObjectAnimator.ofFloat(mSecurityButton, ALPHA, 1);
@@ -154,9 +154,9 @@ public class CustomTabToolbar extends ToolbarLayout implements LocationBar {
@Override
public boolean shouldEmphasizeHttpsScheme() {
int securityLevel = getSecurityLevel();
- if (securityLevel == ConnectionSecurityHelperSecurityLevel.SECURITY_ERROR
- || securityLevel == ConnectionSecurityHelperSecurityLevel.SECURITY_WARNING
- || securityLevel == ConnectionSecurityHelperSecurityLevel.SECURITY_POLICY_WARNING) {
+ if (securityLevel == ConnectionSecurityStatusSecurityLevel.SECURITY_ERROR
+ || securityLevel == ConnectionSecurityStatusSecurityLevel.SECURITY_WARNING
+ || securityLevel == ConnectionSecurityStatusSecurityLevel.SECURITY_POLICY_WARNING) {
return true;
}
return false;
@@ -297,7 +297,7 @@ public class CustomTabToolbar extends ToolbarLayout implements LocationBar {
}
private int getSecurityLevel() {
- if (getCurrentTab() == null) return ConnectionSecurityHelperSecurityLevel.NONE;
+ if (getCurrentTab() == null) return ConnectionSecurityStatusSecurityLevel.NONE;
return getCurrentTab().getSecurityLevel();
}
@@ -310,7 +310,7 @@ public class CustomTabToolbar extends ToolbarLayout implements LocationBar {
if (mSecurityIconType == securityLevel) return;
mSecurityIconType = securityLevel;
- if (securityLevel == ConnectionSecurityHelperSecurityLevel.NONE) {
+ if (securityLevel == ConnectionSecurityStatusSecurityLevel.NONE) {
// TODO(yusufo): Add an animator for hiding as well.
mSecurityButton.setVisibility(GONE);
} else {

Powered by Google App Engine
This is Rietveld 408576698