Commit abe7731e authored by Tomasz Wiszkowski's avatar Tomasz Wiszkowski Committed by Commit Bot

Align size of Omnibox and Suggestion icons.

Bug: 992024, 991039
Change-Id: I3359e20a67c5a40d10a704c92525e6c7a1cc9f69
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1976799Reviewed-by: default avatarYusuf Ozuysal <yusufo@chromium.org>
Reviewed-by: default avatarTheresa  <twellington@chromium.org>
Commit-Queue: Ender <ender@google.com>
Cr-Commit-Position: refs/heads/master@{#726523}
parent ef4e520b
...@@ -37,7 +37,7 @@ ...@@ -37,7 +37,7 @@
android:minWidth="8dp" android:minWidth="8dp"
android:adjustViewBounds="true" android:adjustViewBounds="true"
android:importantForAccessibility="no" android:importantForAccessibility="no"
android:src="@drawable/ic_omnibox_page"/> app:srcCompat="@drawable/ic_globe_24dp"/>
<TextView <TextView
android:id="@+id/tab_title" android:id="@+id/tab_title"
android:layout_width="match_parent" android:layout_width="match_parent"
......
...@@ -4,6 +4,7 @@ ...@@ -4,6 +4,7 @@
found in the LICENSE file. --> found in the LICENSE file. -->
<org.chromium.chrome.browser.tasks.tab_management.MessageCardView <org.chromium.chrome.browser.tasks.tab_management.MessageCardView
xmlns:android="http://schemas.android.com/apk/res/android" xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:app="http://schemas.android.com/apk/res-auto"
android:id="@+id/tab_grid_message_item" android:id="@+id/tab_grid_message_item"
android:layout_width="match_parent" android:layout_width="match_parent"
android:layout_height="wrap_content" android:layout_height="wrap_content"
...@@ -17,7 +18,7 @@ ...@@ -17,7 +18,7 @@
android:minWidth="8dp" android:minWidth="8dp"
android:adjustViewBounds="true" android:adjustViewBounds="true"
android:importantForAccessibility="no" android:importantForAccessibility="no"
android:src="@drawable/ic_omnibox_page"/> app:srcCompat="@drawable/ic_globe_24dp"/>
<org.chromium.chrome.browser.ui.widget.text.TemplatePreservingTextView <org.chromium.chrome.browser.ui.widget.text.TemplatePreservingTextView
android:id="@+id/description" android:id="@+id/description"
android:layout_width="0dp" android:layout_width="0dp"
......
...@@ -9,10 +9,12 @@ import android.content.res.Resources; ...@@ -9,10 +9,12 @@ import android.content.res.Resources;
import android.graphics.Bitmap; import android.graphics.Bitmap;
import android.text.TextUtils; import android.text.TextUtils;
import android.view.View; import android.view.View;
import androidx.annotation.ColorRes; import androidx.annotation.ColorRes;
import androidx.annotation.DrawableRes; import androidx.annotation.DrawableRes;
import androidx.annotation.StringRes; import androidx.annotation.StringRes;
import androidx.annotation.VisibleForTesting; import androidx.annotation.VisibleForTesting;
import org.chromium.base.Callback; import org.chromium.base.Callback;
import org.chromium.base.MathUtils; import org.chromium.base.MathUtils;
import org.chromium.base.library_loader.LibraryProcessType; import org.chromium.base.library_loader.LibraryProcessType;
...@@ -459,8 +461,8 @@ class StatusMediator { ...@@ -459,8 +461,8 @@ class StatusMediator {
mIsSecurityButtonShown = false; mIsSecurityButtonShown = false;
if (mUrlHasFocus) { if (mUrlHasFocus) {
if (mShowStatusIconWhenUrlFocused) { if (mShowStatusIconWhenUrlFocused) {
icon = mFirstSuggestionIsSearchQuery ? R.drawable.omnibox_search icon = mFirstSuggestionIsSearchQuery ? R.drawable.ic_suggestion_magnifier
: R.drawable.ic_omnibox_page; : R.drawable.ic_globe_24dp;
tint = mNavigationIconTintRes; tint = mNavigationIconTintRes;
} }
} else if (mSecurityIconRes != 0) { } else if (mSecurityIconRes != 0) {
......
...@@ -369,7 +369,7 @@ public class LocationBarModel implements ToolbarDataProvider, ToolbarCommonPrope ...@@ -369,7 +369,7 @@ public class LocationBarModel implements ToolbarDataProvider, ToolbarCommonPrope
// If we're showing a query in the omnibox, and the security level is high enough to show // If we're showing a query in the omnibox, and the security level is high enough to show
// the search icon, return that instead of the security icon. // the search icon, return that instead of the security icon.
if (getDisplaySearchTerms() != null) { if (getDisplaySearchTerms() != null) {
return R.drawable.omnibox_search; return R.drawable.ic_suggestion_magnifier;
} }
return getSecurityIconResource(getSecurityLevel(), !isTablet, isOfflinePage(), isPreview()); return getSecurityIconResource(getSecurityLevel(), !isTablet, isOfflinePage(), isPreview());
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment