Commit a3d0f25d authored by Mohamed Amir Yosef's avatar Mohamed Amir Yosef Committed by Chromium LUCI CQ

[Autofill] Use LocationBarBubbleDelegateView enum as reason ...

... in AutofillBubbleHandler.

Before this patch:
Each bubble used to call the enum value using its own class name.
(specifically SaveCardBubbleViews and LocalCardMigrationBubbleViews)

After this patch:
LocationBarBubbleDelegateView is used instead when referring to
USER_GESTURE and AUTOMATIC enum values.

Change-Id: I4bfb79713443f608cee6d245af8c2064793e6d3b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2644624
Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org>
Commit-Queue: Evan Stade <estade@chromium.org>
Auto-Submit: Mohamed Amir Yosef <mamir@chromium.org>
Reviewed-by: default avatarEvan Stade <estade@chromium.org>
Cr-Commit-Position: refs/heads/master@{#846284}
parent 11a7835f
......@@ -85,8 +85,8 @@ SaveCardBubbleView* AutofillBubbleHandlerImpl::ShowSaveCreditCardBubble(
bubble->SetHighlightedButton(icon_view);
views::BubbleDialogDelegateView::CreateBubble(bubble);
bubble->Show(is_user_gesture ? SaveCardBubbleViews::USER_GESTURE
: SaveCardBubbleViews::AUTOMATIC);
bubble->Show(is_user_gesture ? LocationBarBubbleDelegateView::USER_GESTURE
: LocationBarBubbleDelegateView::AUTOMATIC);
return bubble;
}
......@@ -107,8 +107,8 @@ AutofillBubbleHandlerImpl::ShowLocalCardMigrationBubble(
bubble->SetHighlightedButton(icon_view);
views::BubbleDialogDelegateView::CreateBubble(bubble);
bubble->Show(is_user_gesture ? LocalCardMigrationBubbleViews::USER_GESTURE
: LocalCardMigrationBubbleViews::AUTOMATIC);
bubble->Show(is_user_gesture ? LocationBarBubbleDelegateView::USER_GESTURE
: LocationBarBubbleDelegateView::AUTOMATIC);
return bubble;
}
......
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