Commit 8af3b9a6 authored by Tina Wang's avatar Tina Wang Committed by Commit Bot

[SendTabToSelf] Put icon to the most left

Put send tab to self icon to the left of other page action icons.

Bug: 978520
Change-Id: I155ea9213ebe3405fa88f95e3c0b788cccd4c421
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1676614
Commit-Queue: Tina Wang <tinazwang@chromium.org>
Reviewed-by: default avatarTommy Li <tommycli@chromium.org>
Reviewed-by: default avatarTanya Gupta <tgupta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#672648}
parent ffdc6c1f
......@@ -223,6 +223,10 @@ void LocationBarView::Init() {
// |browser_| may be null when LocationBarView is used for non-Browser windows
// such as PresentationReceiverWindowView, which do not support page actions.
if (browser_) {
// The send tab to self icon is intentionally the first one added so it is
// the left most icon.
if (send_tab_to_self::IsSendingEnabled())
params.types_enabled.push_back(PageActionIconType::kSendTabToSelf);
if (!base::FeatureList::IsEnabled(
autofill::features::kAutofillEnableToolbarStatusChip)) {
params.types_enabled.push_back(PageActionIconType::kManagePasswords);
......@@ -233,8 +237,6 @@ void LocationBarView::Init() {
params.types_enabled.push_back(PageActionIconType::kFind);
params.types_enabled.push_back(PageActionIconType::kTranslate);
params.types_enabled.push_back(PageActionIconType::kZoom);
if (send_tab_to_self::IsSendingEnabled())
params.types_enabled.push_back(PageActionIconType::kSendTabToSelf);
}
params.icon_size = GetLayoutConstant(LOCATION_BAR_ICON_SIZE);
params.icon_color = icon_color;
......
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