Chromium Code Reviews| Index: chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/HostedToolbar.java |
| diff --git a/chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/HostedToolbar.java b/chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/HostedToolbar.java |
| index bba90f0896bd96419ddf121dbc138a8a4bc42cb4..1987d798a52db97d94962d853eaebb6da1d53a65 100644 |
| --- a/chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/HostedToolbar.java |
| +++ b/chrome/android/java_staging/src/org/chromium/chrome/browser/toolbar/HostedToolbar.java |
| @@ -116,7 +116,7 @@ public class HostedToolbar extends ToolbarLayout implements LocationBar { |
| @Override |
| public boolean shouldEmphasizeHttpsScheme() { |
| - return !mUseDarkColors; |
| + return mUseDarkColors; |
|
pasko
2015/05/26 17:30:07
I am totally not an expert in this code, so please
Yusuf
2015/05/26 17:34:46
You are right. We should also be looking at the se
|
| } |
| @Override |
| @@ -170,6 +170,7 @@ public class HostedToolbar extends ToolbarLayout implements LocationBar { |
| @Override |
| public void updateLoadingState(boolean updateUrl) { |
| + if (updateUrl) setUrlToPageUrl(); |
| updateSecurityIcon(getSecurityLevel()); |
| } |